Patch from rovv: when we have no pending streams, choose exits with choose_good_exit_...

svn:r17091

diff --git a/ChangeLog b/ChangeLog
index 2a094444206961e6b447c7c4d7443dc9384bb0cb..e5a421a232fd221916485fa1e197ede4385b4fee 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -8,6 +8,9 @@ Changes in version 0.2.1.7-alpha - 2008-10-xx
     - Minor fix in the warning messages when you're having problems
       bootstrapping; also, be more forgiving of bootstrap problems when
       we're still making incremental progress on a given bootstrap phase.
+    - When we're choosing an exit node for a circuit, and we have no pending
+      streams, choose a good general exit rather than one that supports "all
+      the pending streams".  Bugfix on 0.1.1.x. (Fix by rovv.)
 
 
 Changes in version 0.2.1.6-alpha - 2008-09-30

diff --git a/src/or/circuitbuild.c b/src/or/circuitbuild.c
index 9470f8698601d6a4a24fb84acf01cf59945572da..02414c69adddc36be0630cf80a6ea8c457d8fe32 100644 (file)
--- a/src/or/circuitbuild.c
+++ b/src/or/circuitbuild.c
@@ -1265,6 +1265,11 @@ choose_good_exit_server_general(routerlist_t *dir, int need_uptime,
 //               router->nickname, i);
       }
     }); /* End looping over connections. */
+    if (n_pending_connections > 0 && n_supported[i] == 0) {
+      /* Leave best_support at -1 if that's where it is, so we can
+       * distinguish it later. */
+      continue;
+    }
     if (n_supported[i] > best_support) {
       /* If this router is better than previous ones, remember its index
        * and goodness, and start counting how many routers are this good. */