r17610@catbus: nickm | 2008-01-14 13:20:49 -0500

Fix a bogus free() call on a base64 failure in router_append_dirobj_signature().


svn:r13129

ChangeLog

@@ -43,6 +43,7 @@ Changes in version 0.2.0.16-alpha - 2008-01-??
     - Do not try to download missing certificates until we have tried
       to check our fallback consensus.  Fixes bug 583.
     - Make bridges round geoip info up, not down.
+    - Avoid a spurious free on base64 failure.  Bugfix on 0.1.2.
 
   o Minor features (controller):
     - Get NS events working again.  (Patch from tup)

src/or/routerparse.c

@@ -521,7 +521,6 @@ router_append_dirobj_signature(char *buf, size_t buf_len, const char *digest,
   i = strlen(buf);
   if (base64_encode(buf+i, buf_len-i, signature, 128) < 0) {
     log_warn(LD_BUG,"couldn't base64-encode signature");
-    tor_free(buf);
     goto err;
   }