[tor-commits] [tor/master] Log statement to help track down bug4091

nickm at torproject.org nickm at torproject.org
Tue Mar 27 22:24:46 UTC 2012


commit 80b2756b537204ba070f1e3df4e5ed3f4e6815c0
Author: Nick Mathewson <nickm at torproject.org>
Date:   Tue Mar 27 18:28:39 2012 -0400

    Log statement to help track down bug4091
---
 changes/bug4091_debugging |    5 +++++
 src/common/address.c      |    4 +++-
 2 files changed, 8 insertions(+), 1 deletions(-)

diff --git a/changes/bug4091_debugging b/changes/bug4091_debugging
new file mode 100644
index 0000000..f785380
--- /dev/null
+++ b/changes/bug4091_debugging
@@ -0,0 +1,5 @@
+  o Minor features:
+    - Add more information to a log statement that might help track down
+      bug 4091. If you're seeing "Bug: tor_addr_is_internal() called with a
+      non-IP address" messages (or any Bug messages, for that matter!),
+      please let us know about it.
diff --git a/src/common/address.c b/src/common/address.c
index 17cda42..676c485 100644
--- a/src/common/address.c
+++ b/src/common/address.c
@@ -355,7 +355,9 @@ tor_addr_is_internal(const tor_addr_t *addr, int for_listening)
 
   /* unknown address family... assume it's not safe for external use */
   /* rather than tor_assert(0) */
-  log_warn(LD_BUG, "tor_addr_is_internal() called with a non-IP address.");
+  log_warn(LD_BUG, "tor_addr_is_internal() called with a non-IP address of "
+           "type %d", (int)v_family);
+  tor_fragile_assert();
   return 1;
 }
 



More information about the tor-commits mailing list