diff options
author | Ingo Molnar <mingo@elte.hu> | 2005-04-14 12:43:44 +0200 |
---|---|---|
committer | Petr Baudis <xpasky@machine.sinus.cz> | 2005-05-11 23:08:20 +0200 |
commit | c599caca3ab7d5d66d7cadbbf589c2e4b5587d3e (patch) | |
tree | 3307e3b228f1134c554cca619a7ef6e5643a4701 /ls-tree.c | |
parent | aceedfe32e766b169d480c0ec24777d1b09bb118 (diff) | |
download | git-c599caca3ab7d5d66d7cadbbf589c2e4b5587d3e.tar.gz git-c599caca3ab7d5d66d7cadbbf589c2e4b5587d3e.tar.xz |
[patch] git: cleanup in ls-tree.c
cleanup: this patch adds a free() to ls-tree.c.
(Technically it's not a memory leak yet because the buffer is allocated
once by the function and then the utility exits - but it's a tad cleaner
to not leave such assumptions in the code, so that if someone reuses the
function (or extends the utility to include a loop) the uncleanliness
doesnt develop into a real memory leak.)
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Forward-ported.
Signed-off-by: Petr Baudis <pasky@ucw.cz>
Diffstat (limited to 'ls-tree.c')
-rw-r--r-- | ls-tree.c | 1 |
1 files changed, 1 insertions, 0 deletions
@@ -77,6 +77,7 @@ static int list(unsigned char *sha1) if (!buffer) die("unable to read sha1 file"); list_recursive(buffer, "tree", size, NULL); + free(buffer); return 0; } |