Browse Source

note a worrying double-free possibility in torgzip.
nick?


svn:r4537

Roger Dingledine 19 years ago
parent
commit
4457c181f5
1 changed files with 1 additions and 1 deletions
  1. 1 1
      src/common/torgzip.c

+ 1 - 1
src/common/torgzip.c

@@ -208,7 +208,7 @@ tor_gzip_uncompress(char **out, size_t *out_len,
   *out_len = stream->total_out;
   if (inflateEnd(stream)!=Z_OK) {
     log_fn(LOG_WARN, "Error freeing gzip structures");
-    goto err;
+    goto err; /* XXX this will try to inflateEnd again, right? is that bad? */
   }
   tor_free(stream);