From 0ba05313d56e6737c73d003c84eeebdfd0d40398 Mon Sep 17 00:00:00 2001 From: Nick Mathewson Date: Mon, 22 Aug 2016 01:22:19 -0400 Subject: [PATCH] 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(-) create mode 100644 changes/bug19947 diff --git a/changes/bug19947 b/changes/bug19947 new file mode 100644 index 000000000..b9dce8b75 --- /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 620c32d64..c3588726f 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);