From: Nick Mathewson <nickm@torproject.org>
Date: Thu, 9 Apr 2015 15:57:55 +0000 (-0400)
Subject: Collect badness values in ntor_ref.py; don't just clobber.
X-Git-Tag: tor-0.2.7.1-alpha~72
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=f9327848babe8fd3b08eabb187d32b945f7567fc;p=thirdparty%2Ftor.git

Collect badness values in ntor_ref.py; don't just clobber.

Fixes bug 15591; patch from joelanders
---

diff --git a/changes/bug15591 b/changes/bug15591
new file mode 100644
index 0000000000..1398a027d7
--- /dev/null
+++ b/changes/bug15591
@@ -0,0 +1,4 @@
+  o Testing:
+    - Check for matching value in server response in ntor_ref.py.
+      Fixes bug 15591; bugfix on 0.2.4.8-alpha. Reported and fixed
+      by "joelanders".
diff --git a/src/test/ntor_ref.py b/src/test/ntor_ref.py
index e37637d92a..767da57a9c 100755
--- a/src/test/ntor_ref.py
+++ b/src/test/ntor_ref.py
@@ -283,7 +283,7 @@ def client_part2(seckey_x, msg, node_id, pubkey_B, keyBytes=72):
     my_auth = H_mac(auth_input)
 
     badness = my_auth != their_auth
-    badness = bad_result(yx) + bad_result(bx)
+    badness |= bad_result(yx) + bad_result(bx)
 
     if badness:
         return None