[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

[or-cvs] [tor/maint-0.2.2 2/2] Add a ! to directory_caches_dir_info() to fix a logic error



Author: Nick Mathewson <nickm@xxxxxxxxxxxxxx>
Date: Wed, 20 Oct 2010 13:49:38 -0400
Subject: Add a ! to directory_caches_dir_info() to fix a logic error
Commit: 2849a95691c002108666eb414ff497bf93349e7d

We want to fetch directory info more aggressively if we need it to
refuseunknownexits.  Thus, we'll want it if our exit policy is _NOT_
reject *.
---
 changes/caches_if_exit |    5 +++++
 src/or/dirserv.c       |    2 +-
 2 files changed, 6 insertions(+), 1 deletions(-)
 create mode 100644 changes/caches_if_exit

diff --git a/changes/caches_if_exit b/changes/caches_if_exit
new file mode 100644
index 0000000..0e66227
--- /dev/null
+++ b/changes/caches_if_exit
@@ -0,0 +1,5 @@
+  o Minor bugfixes:
+    - Fix a logic error where servers that _didn't_ act as exits would
+      try to keep their server lists more aggressively up to date than
+      exits, when it was supposed to be the other way around. Bugfix
+      on 0.2.2.17-alpha.
diff --git a/src/or/dirserv.c b/src/or/dirserv.c
index 8ae0342..8d0ec98 100644
--- a/src/or/dirserv.c
+++ b/src/or/dirserv.c
@@ -1213,7 +1213,7 @@ directory_caches_dir_info(or_options_t *options)
     return 0;
   /* We need an up-to-date view of network info if we're going to try to
    * block exit attempts from unknown relays. */
-  return router_my_exit_policy_is_reject_star() &&
+  return ! router_my_exit_policy_is_reject_star() &&
     should_refuse_unknown_exits(options);
 }
 
-- 
1.7.1