[tor-commits] [tor/master] Avoid memory leak in error messages in control.c (not in any tor)

nickm at torproject.org nickm at torproject.org
Thu Apr 23 13:28:20 UTC 2015


commit f5fa6ac534d9262701382cab3152826cc56a9e5d
Author: Nick Mathewson <nickm at torproject.org>
Date:   Thu Apr 23 09:26:31 2015 -0400

    Avoid memory leak in error messages in control.c (not in any tor)
---
 src/or/control.c |    7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/src/or/control.c b/src/or/control.c
index bbc4115..547145b 100644
--- a/src/or/control.c
+++ b/src/or/control.c
@@ -1716,13 +1716,11 @@ getinfo_helper_dir(control_connection_t *control_conn,
     SMARTLIST_FOREACH(sl, char *, c, tor_free(c));
     smartlist_free(sl);
   } else if (!strcmpstart(question, "hs/client/desc/id/")) {
-    char *msg;
     rend_cache_entry_t *e = NULL;
 
     question += strlen("hs/client/desc/id/");
     if (strlen(question) != REND_SERVICE_ID_LEN_BASE32) {
-      tor_asprintf(&msg, "\"%s\" invalid address.", question);
-      *errmsg = msg;
+      *errmsg = "Invalid address";
       return -1;
     }
 
@@ -1730,8 +1728,7 @@ getinfo_helper_dir(control_connection_t *control_conn,
       /* Descriptor found in cache */
       *answer = tor_strdup(e->desc);
     } else {
-      tor_asprintf(&msg, "\"%s\" not found in cache.", question);
-      *errmsg = msg;
+      *errmsg = "Not found in cache";
       return -1;
     }
   } else if (!strcmpstart(question, "md/id/")) {





More information about the tor-commits mailing list