Browse Source

Merge remote-tracking branch 'public/bug19648'

Nick Mathewson 6 years ago
parent
commit
86eb63deb4
2 changed files with 8 additions and 2 deletions
  1. 5 0
      changes/bug19648
  2. 3 2
      src/or/routerlist.c

+ 5 - 0
changes/bug19648

@@ -0,0 +1,5 @@
+  o Minor bugfixes (logging):
+    - When logging the number of descriptors we intend to download per
+      directory request, do not log a number higher than then the
+      number of descriptors we're fetching in total. Fixes bug 19648;
+      bugfix on 0.1.1.8-alpha.

+ 3 - 2
src/or/routerlist.c

@@ -5055,8 +5055,9 @@ launch_descriptor_downloads(int purpose,
   if (n_per_request > max_dl_per_req)
     n_per_request = max_dl_per_req;
 
-  if (n_per_request < MIN_DL_PER_REQUEST)
-    n_per_request = MIN_DL_PER_REQUEST;
+  if (n_per_request < MIN_DL_PER_REQUEST) {
+    n_per_request = MIN(MIN_DL_PER_REQUEST, n_downloadable);
+  }
 
   if (n_downloadable > n_per_request)
     req_plural = rtr_plural = "s";