[tor-commits] [tor/maint-0.2.8] Do not pass NULL to log(%s) in dir_server_new.

nickm at torproject.org nickm at torproject.org
Mon Aug 22 05:23:48 UTC 2016


commit 0ba05313d56e6737c73d003c84eeebdfd0d40398
Author: Nick Mathewson <nickm at torproject.org>
Date:   Mon Aug 22 01:22:19 2016 -0400

    Do not pass NULL to log(%s) in dir_server_new.
    
    This bug had existed since 0.2.4.7-alpha, but now that we have
    FallbackDirs by default, it actually matters.
    
    Fixes bug 19947; bugfix on 0.2.4.7-alpha or maybe 0.2.8.1-alpha.
    
    Rubiate wrote the patch; teor wrote the changes file.
---
 changes/bug19947    | 4 ++++
 src/or/routerlist.c | 4 ++--
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/changes/bug19947 b/changes/bug19947
new file mode 100644
index 0000000..b9dce8b
--- /dev/null
+++ b/changes/bug19947
@@ -0,0 +1,4 @@
+ o Minor bugfixes (fallback directories):
+    - Avoid logging a NULL string pointer when loading fallback directory information.
+      Fixes bug 19947; bugfix on 0.2.4.7-alpha and 0.2.8.1-alpha.
+      Report and patch by "rubiate".
diff --git a/src/or/routerlist.c b/src/or/routerlist.c
index 620c32d..c358872 100644
--- a/src/or/routerlist.c
+++ b/src/or/routerlist.c
@@ -4337,10 +4337,10 @@ dir_server_new(int is_authority,
 
   if (nickname)
     tor_asprintf(&ent->description, "directory server \"%s\" at %s:%d",
-                 nickname, hostname, (int)dir_port);
+                 nickname, hostname_, (int)dir_port);
   else
     tor_asprintf(&ent->description, "directory server at %s:%d",
-                 hostname, (int)dir_port);
+                 hostname_, (int)dir_port);
 
   ent->fake_status.addr = ent->addr;
   tor_addr_copy(&ent->fake_status.ipv6_addr, &ent->ipv6_addr);



More information about the tor-commits mailing list