From: Nick Mathewson <nickm@torproject.org>
Date: Thu, 24 Nov 2011 04:04:10 +0000 (-0500)
Subject: Detect tor_addr_to_str failure in tor_dup_addr.
X-Git-Tag: tor-0.2.3.9-alpha~29^2
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=3ebe960f3f734322ea3ea471ca6abceb245ca322;p=thirdparty%2Ftor.git

Detect tor_addr_to_str failure in tor_dup_addr.

This avoids a possible strdup of an uninitialized buffer.

Fixes 4529; fix on 0.2.1.3-alpha; reported by troll_un.
---

diff --git a/changes/bug4529 b/changes/bug4529
new file mode 100644
index 0000000000..89d10b2f6b
--- /dev/null
+++ b/changes/bug4529
@@ -0,0 +1,5 @@
+  o Minor bufixes:
+    - If we had ever tried to call tor_addr_to_str on an address of
+      unknown type, we would have done a strdup on an uninitialized
+      buffer. Now we won't. Fixes bug 4529; bugfix on 0.2.1.3-alpha.
+      Reported by "troll_un".
diff --git a/src/common/address.c b/src/common/address.c
index 7fc7301051..46ccb1fe4a 100644
--- a/src/common/address.c
+++ b/src/common/address.c
@@ -945,8 +945,11 @@ char *
 tor_dup_addr(const tor_addr_t *addr)
 {
   char buf[TOR_ADDR_BUF_LEN];
-  tor_addr_to_str(buf, addr, sizeof(buf), 0);
-  return tor_strdup(buf);
+  if (tor_addr_to_str(buf, addr, sizeof(buf), 0)) {
+    return tor_strdup(buf);
+  } else {
+    return tor_strdup("<unknown address type>");
+  }
 }
 
 /** Return a string representing the address <b>addr</b>.  This string is