From: Nick Mathewson <nickm@torproject.org>
Date: Fri, 23 Feb 2007 23:11:08 +0000 (+0000)
Subject:  r11900@catbus:  nickm | 2007-02-23 18:11:04 -0500
X-Git-Tag: tor-0.1.2.8-beta~29
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=4e05ccb289aaf5b8afc24888dedd4bd9bda285ef;p=thirdparty%2Ftor.git

 r11900@catbus:  nickm | 2007-02-23 18:11:04 -0500
 Fix null-pointer access on update_router_descriptor_cache_downloads.


svn:r9626
---

diff --git a/src/or/routerlist.c b/src/or/routerlist.c
index f11f1670ea..e9ac4534e3 100644
--- a/src/or/routerlist.c
+++ b/src/or/routerlist.c
@@ -4029,13 +4029,12 @@ update_router_descriptor_cache_downloads(time_t now)
   SMARTLIST_FOREACH(networkstatus_list, networkstatus_t *, ns,
     {
       smartlist_t *dl;
+      dl = downloadable[ns_sl_idx] = smartlist_create();
+      download_from[ns_sl_idx] = smartlist_create();
       if (ns->published_on + MAX_NETWORKSTATUS_AGE-10*60 > now) {
         /* Don't download if the networkstatus is almost ancient. */
         continue;
       }
-      dl = smartlist_create();
-      downloadable[ns_sl_idx] = dl;
-      download_from[ns_sl_idx] = smartlist_create();
       SMARTLIST_FOREACH(ns->entries, routerstatus_t * , rs,
         {
           if (!rs->need_to_mirror)
@@ -4076,7 +4075,6 @@ update_router_descriptor_cache_downloads(time_t now)
     smartlist_t *dl = downloadable[which_ns];
     int idx;
     char *d;
-    tor_assert(dl);
     if (!smartlist_len(dl))
       continue;
     idx = crypto_rand_int(smartlist_len(dl));