diff options
| author | Roger Dingledine <arma@torproject.org> | 2006-04-25 07:06:48 +0000 |
|---|---|---|
| committer | Roger Dingledine <arma@torproject.org> | 2006-04-25 07:06:48 +0000 |
| commit | 701341ef8a408c9369b29c3c6557c95a5aa53610 (patch) | |
| tree | 7b34c0ef2d3b05bbcd5f9a92c5b2e4d9c92ad338 /src/or/circuitbuild.c | |
| parent | 3c86402cc2003b490d2586658f401346008016f9 (diff) | |
| download | tor-701341ef8a408c9369b29c3c6557c95a5aa53610.tar tor-701341ef8a408c9369b29c3c6557c95a5aa53610.tar.gz | |
put one of the XXX's back in
svn:r6422
Diffstat (limited to 'src/or/circuitbuild.c')
| -rw-r--r-- | src/or/circuitbuild.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/src/or/circuitbuild.c b/src/or/circuitbuild.c index 9a68a636b..4f69d028e 100644 --- a/src/or/circuitbuild.c +++ b/src/or/circuitbuild.c @@ -1421,6 +1421,10 @@ count_acceptable_routers(smartlist_t *routers) if (r->is_valid == 0) { // log_debug(LD_CIRC,"Nope, the directory says %d is not valid.",i); goto next_i_loop; + /* XXX This clause makes us count incorrectly: if AllowInvalidRouters + * allows this node in some places, then we're getting an inaccurate + * count. For now, be conservative and don't count it. But later we + * should try to be smarter. */ } num++; // log_debug(LD_CIRC,"I like %d. num_acceptable_routers now %d.",i, num); |