Backport: Fix from rovv: when adding a detached signature, do not automatically assum...

svn:r17249

diff --git a/ChangeLog b/ChangeLog
index fa958748077b1be91d65c8cd39b966097db75ec7..41b4432afa50dcb297174cfe85034f62275bd567 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,4 +1,8 @@
 Changes in version 0.2.0.32 - 2008-??-??
+  o Major bugfixes:
+    - Fix a DOS opportunity during the voting signature collection process
+      at directory authorities. Spotted by rovv. Bugfix on 0.2.0.x.
+
   o Minor bugfixes:
     - Fix several infrequent memory leaks spotted by Coverity.
     - When testing for libevent functions, set the LDFLAGS variable

diff --git a/src/or/dirvote.c b/src/or/dirvote.c
index a34b7f180392b1e67b815aa47a3a4ae0ba648a8e..d3c1ce7bbf1ee226a924e92f9224e943dbcd270b 100644 (file)
--- a/src/or/dirvote.c
+++ b/src/or/dirvote.c
@@ -1089,8 +1089,8 @@ networkstatus_add_detached_signatures(networkstatus_t *target,
         memcpy(target_voter->signing_key_digest, src_voter->signing_key_digest,
                DIGEST_LEN);
         target_voter->signature_len = src_voter->signature_len;
-        target_voter->good_signature = 1;
-        target_voter->bad_signature = 0;
+        target_voter->good_signature = src_voter->good_signature;
+        target_voter->bad_signature = src_voter->bad_signature;
       } else {
         log_info(LD_DIR, "Not adding signature from %s", voter_identity);
       }