[or-cvs] [tor/master] Fix a rare bug in rend_fn tests when the randomly generated port is 0

nickm at torproject.org nickm at torproject.org
Sat Aug 7 18:31:16 UTC 2010


Author: Nick Mathewson <nickm at torproject.org>
Date: Sat, 7 Aug 2010 14:31:58 -0400
Subject: Fix a rare bug in rend_fn tests when the randomly generated port is 0
Commit: f060b18e6c4292be58de0e1b08f25b770cbddff8

Since the rend code doesn't like the port to be 0, we shouldn't generate
the port by declaring crypto_rand_int(65536); instead we should
say crypto_rand_int(65535)+1.

Diagnosed by Matt Edman; fixes bug 1808.
---
 src/test/test.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/src/test/test.c b/src/test/test.c
index 8f6564c..9948ecf 100644
--- a/src/test/test.c
+++ b/src/test/test.c
@@ -1029,7 +1029,7 @@ test_rend_fns(void)
                   intro->extend_info->identity_digest, DIGEST_LEN);
     /* Does not cover all IP addresses. */
     tor_addr_from_ipv4h(&intro->extend_info->addr, crypto_rand_int(65536));
-    intro->extend_info->port = crypto_rand_int(65536);
+    intro->extend_info->port = 1 + crypto_rand_int(65535);
     intro->intro_key = crypto_pk_dup_key(pk2);
     smartlist_add(generated->intro_nodes, intro);
   }
-- 
1.7.1



More information about the tor-commits mailing list