Remove needless strdup in addressmap_register_virtual_address()

Fixes bug 14195. Bugfix on 0.1.0.1-rc.

diff --git a/changes/bug14195 b/changes/bug14195
new file mode 100644 (file)
index 0000000..d2b82f3
--- /dev/null
+++ b/changes/bug14195
@@ -0,0 +1,3 @@
+  o Minor bugfixes (client):
+    - Fix a memory leak when using AutomapHostsOnResolve.
+      Fixes bug 14195; bugfix on 0.1.0.1-rc.

diff --git a/src/or/addressmap.c b/src/or/addressmap.c
index 4253532cecc7784c74cdb5af485948bd49d62710..d4b7acf27495a86052d471d5eed9123db7ca67b4 100644 (file)
--- a/src/or/addressmap.c
+++ b/src/or/addressmap.c
@@ -951,7 +951,7 @@ addressmap_register_virtual_address(int type, char *new_address)
         !strcasecmp(new_address, ent->new_address)) {
       tor_free(new_address);
       tor_assert(!vent_needs_to_be_added);
-      return tor_strdup(*addrp);
+      return *addrp;
     } else {
       log_warn(LD_BUG,
                "Internal confusion: I thought that '%s' was mapped to by "