diff options
author | Shawn O. Pearce <spearce@spearce.org> | 2010-04-17 13:07:34 -0700 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2010-04-17 13:55:45 -0700 |
commit | 03b6aeb27443f117a3d8375f01bc38baeeff65a5 (patch) | |
tree | 7dae2836758a59be2841541a4b542f50e945ff0a /http.c | |
parent | f78683f3a85ca16ae16e1180703d3280b272f45d (diff) | |
download | git-03b6aeb27443f117a3d8375f01bc38baeeff65a5.tar.gz |
http.c: Remove bad free of static block
The filename variable here is pointing to a block of memory that
was allocated by sha1_file.c and is also held in a static variable
scoped within the sha1_pack_name() function. Doing a free() here is
returning that memory to the allocator while we might still try to
reuse it on a subsequent sha1_pack_name() invocation. That's not
acceptable, so don't free it.
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'http.c')
-rw-r--r-- | http.c | 1 |
1 files changed, 0 insertions, 1 deletions
@@ -1082,7 +1082,6 @@ struct http_pack_request *new_http_pack_request( return preq; abort: - free(filename); free(preq->url); free(preq); return NULL; |