[or-cvs] I *knew* something was wrong here. Flip some comparison op...

Nick Mathewson nickm at seul.org
Wed Oct 5 22:36:20 UTC 2005


Update of /home/or/cvsroot/tor/src/or
In directory moria:/tmp/cvs-serv17019/src/or

Modified Files:
	routerlist.c 
Log Message:
I *knew* something was wrong here.  Flip some comparison operators and make download-rate-limiting sane again.

Index: routerlist.c
===================================================================
RCS file: /home/or/cvsroot/tor/src/or/routerlist.c,v
retrieving revision 1.331
retrieving revision 1.332
diff -u -d -r1.331 -r1.332
--- routerlist.c	5 Oct 2005 20:45:18 -0000	1.331
+++ routerlist.c	5 Oct 2005 22:36:18 -0000	1.332
@@ -2715,15 +2715,16 @@
   } else {
     if (dirserv) {
       should_delay = (last_routerdesc_download_attempted +
-                      MAX_SERVER_INTERVAL_WITHOUT_REQUEST) < now;
+                      MAX_SERVER_INTERVAL_WITHOUT_REQUEST) > now;
     } else {
       should_delay = (last_routerdesc_download_attempted +
-                      MAX_CLIENT_INTERVAL_WITHOUT_REQUEST) < now;
+                      MAX_CLIENT_INTERVAL_WITHOUT_REQUEST) > now;
     }
     if (should_delay)
-      log_fn(LOG_INFO, "There are not many downloadable routerdescs; waiting till we have some more.");
+      log_fn(LOG_DEBUG, "There are not many downloadable routerdescs; waiting till we have some more.");
     else
-      log_fn(LOG_INFO, "There are not many downloadable routerdescs, but we've been waiting long enough. Downloading.");
+      log_fn(LOG_INFO, "There are not many downloadable routerdescs, but we've been waiting long enough (%d seconds). Downloading.",
+             (int)(now-last_routerdesc_download_attempted));
   }
 
   if (! should_delay) {



More information about the tor-commits mailing list