Fix a logic error that would automatically reject all but the first configured DNS...

svn:r17569

diff --git a/ChangeLog b/ChangeLog
index 1da30715742487b92ac5f2770ab5a214bd5ba306..6a361c2956bdfcce2d6cb14eeb8b346f076218b4 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,4 +1,9 @@
 Changes in version 0.2.1.9-alpha - 200?-??-??
+  o Major bugfixes:
+    - Fix a logic error that would automatically reject all but the first
+      configured DNS server.  Bugfix on 0.2.1.5-alpha.  Possible fix for part
+      of bug 813/868.  Bug spotted by coderman.
+
   o Minor features:
     - Give a better error message when an overzealous init script says,
       "sudo -u username tor --user username".  Makes Bug 882 easier

diff --git a/src/or/eventdns.c b/src/or/eventdns.c
index 88ca1bbe9178e9fecaf1032e24a79a5521c64de8..cd5629bd1ab88aa342fa376a4b66145603570588 100644 (file)
--- a/src/or/eventdns.c
+++ b/src/or/eventdns.c
@@ -2224,7 +2224,7 @@ _evdns_nameserver_add_impl(const struct sockaddr *address,
        int err = 0;
        if (server) {
                do {
-                       if (!sockaddr_eq(address, (struct sockaddr *)&server->address, 1)) {
+                       if (sockaddr_eq(address, (struct sockaddr *)&server->address, 1)) {
                                log(EVDNS_LOG_DEBUG, "Duplicate nameserver.");
                                return 3;
                        }