tor

commit 1c47459e5a5ed6d404a8034502543e7e4a67be38
parent ed0bc85ed0ac765c91def249afa1390bd03cfe85
Author: Nick Mathewson <nickm@torproject.org>
Date:   Wed, 19 Dec 2018 15:36:08 -0500

Merge branch 'maint-0.3.5'

Diffstat:
A
changes/ticket28883
 | 
4
++++
M
src/test/test_rebind.py
 | 
6
++----

2 files changed, 6 insertions(+), 4 deletions(-)
diff --git a/changes/ticket28883 b/changes/ticket28883
@@ -0,0 +1,4 @@
+  o Minor bugfixes (testing):
+    - Make sure that test_rebind.py actually obeys its timeout, even
+      when it receives a large number of log messages. Fixes bug 28883;
+      bugfix on 0.3.5.4-alpha.
diff --git a/src/test/test_rebind.py b/src/test/test_rebind.py
@@ -11,7 +11,6 @@ import time
 
 LOG_TIMEOUT = 60.0
 LOG_WAIT = 0.1
-LOG_CHECK_LIMIT = LOG_TIMEOUT / LOG_WAIT
 
 def fail(msg):
     logging.error('FAIL')
@@ -26,8 +25,8 @@ def try_connecting_to_socksport():
     socks_socket.close()
 
 def wait_for_log(s):
-    log_checked = 0
-    while log_checked < LOG_CHECK_LIMIT:
+    cutoff = time.time() + LOG_TIMEOUT
+    while time.time() < cutoff:
         l = tor_process.stdout.readline()
         l = l.decode('utf8')
         if s in l:
@@ -38,7 +37,6 @@ def wait_for_log(s):
         # avoid busy-waiting
         if len(s) == 0:
             time.sleep(LOG_WAIT)
-        log_checked += 1
     fail('Could not find "{}" in logs after {} seconds'.format(s, LOG_TIMEOUT))
 
 def pick_random_port():