Browse Source

r9345@totoro: nickm | 2006-11-20 11:22:10 -0500
Fix a small and unlikely memory leak in dirserv.c. I think this is not the one arma is looking for.


svn:r8971

Nick Mathewson 19 years ago
parent
commit
b4ea6136a4
1 changed files with 3 additions and 1 deletions
  1. 3 1
      src/or/dirserv.c

+ 3 - 1
src/or/dirserv.c

@@ -91,17 +91,19 @@ add_fingerprint_to_dir(const char *nickname, const char *fp,
   tor_assert(fp);
   tor_assert(list);
 
-  fingerprint = tor_strdup(fp);
+  fingeprint = tor_strdup(fp);
   tor_strstrip(fingerprint, " ");
   if (base16_decode(d, DIGEST_LEN, fingerprint, strlen(fingerprint))) {
     log_warn(LD_DIRSERV, "Couldn't decode fingerprint \"%s\"",
              escaped(fp));
+    tor_free(fingerprint);
     return 0;
   }
 
   if (!strcasecmp(nickname, UNNAMED_ROUTER_NICKNAME)) {
     log_warn(LD_DIRSERV, "Tried to add a mapping for reserved nickname %s",
              UNNAMED_ROUTER_NICKNAME);
+    tor_free(fingerprint);
     return 0;
   }