]> git.ipfire.org Git - thirdparty/tor.git/commitdiff
Fix memory leak in tor-gencert.c
authorNick Mathewson <nickm@torproject.org>
Mon, 18 Sep 2017 00:42:28 +0000 (20:42 -0400)
committerNick Mathewson <nickm@torproject.org>
Mon, 18 Sep 2017 00:42:28 +0000 (20:42 -0400)
This was introduced in 4ff170d7b1cbe4074cb852, and is probably
unreachable, but coverity complained about it (CID 1417761). Bug not
in any released Tor, so no changes file.

src/tools/tor-gencert.c

index 600e2252d4aa4666e39a151ae2b039b2addf585f..fb7465c0eb8df741a15ce6ed5a13acf214a488d6 100644 (file)
@@ -464,8 +464,6 @@ generate_certificate(void)
   char expires[ISO_TIME_LEN+1];
   char id_digest[DIGEST_LEN];
   char fingerprint[FINGERPRINT_LEN+1];
-  char *ident = key_to_string(identity_key);
-  char *signing = key_to_string(signing_key);
   FILE *f;
   size_t signed_len;
   char digest[DIGEST_LEN];
@@ -478,6 +476,8 @@ generate_certificate(void)
   if (get_digest(identity_key, id_digest)) {
     return -1;
   }
+  char *ident = key_to_string(identity_key);
+  char *signing = key_to_string(signing_key);
 
   tor_localtime_r(&now, &tm);
   tm.tm_mon += months_lifetime;