bugfix: router_exit_policy_rejects_all() was broken, so we were

sometimes picking middleman nodes as our last hop, which wasn't
very useful.

svn:r3075

diff --git a/src/or/rephist.c b/src/or/rephist.c
index 0cba10ec8a89f44a20571649dc2be937c5725dbb..2f1c976f086ca007f591d60f94fc145d7c764cee 100644 (file)
--- a/src/or/rephist.c
+++ b/src/or/rephist.c
@@ -294,7 +294,7 @@ void rep_hist_dump_stats(time_t now, int severity)
         upt, upt+downt, uptime*100.0);
 
     if (!strmap_isempty(or_history->link_history_map)) {
-      strlcpy(buffer, "    Good extend attempts: ", sizeof(buffer));
+      strlcpy(buffer, "    Extend attempts: ", sizeof(buffer));
       len = strlen(buffer);
       for (lhist_it = strmap_iter_init(or_history->link_history_map);
            !strmap_iter_done(lhist_it);

diff --git a/src/or/routerlist.c b/src/or/routerlist.c
index 91efa0da24c3665a912bb3fa885ffbf271936b5c..0d85c621d15b2acbbd71cb55fa22b30d8999630f 100644 (file)
--- a/src/or/routerlist.c
+++ b/src/or/routerlist.c
@@ -1020,7 +1020,7 @@ int router_exit_policy_all_routers_reject(uint32_t addr, uint16_t port) {
 /** Return true iff <b>router</b> does not permit exit streams.
  */
 int router_exit_policy_rejects_all(routerinfo_t *router) {
-  return router_compare_addr_to_addr_policy(0, 0, router->exit_policy)
+  return router_compare_addr_to_addr_policy(0, 1, router->exit_policy)
     == ADDR_POLICY_REJECTED;
 }