[PATCH/RFC 3/3] read-tree: stop leaking tree objects
- From
Jonathan Nieder <jrnieder@gmail.com>
- Date
- Aug 10, 2010, 03:33 UTC
- Message-ID
- <20100810033344.GD2386@burratino>
- In-Reply-To
- <20100810032647.GA2386@burratino>
The underlying problem is that the fill_tree_descriptor() API is easy to misuse, and this patch does not fix that.
Signed-off-by: Jonathan Nieder <jrnieder@gmail.com> --- unpack-trees.c | 7 ++++++- 1 files changed, 6 insertions(+), 1 deletions(-)
diff --git a/unpack-trees.c b/unpack-trees.c index 8cf0da3..f561d88 100644 --- a/unpack-trees.c +++ b/unpack-trees.c @@ -329,6 +329,7 @@ static int traverse_trees_recursive(int n, unsigned long dirmask, unsigned long { int i, ret, bottom; struct tree_desc t[MAX_UNPACK_TREES]; + void *buf[MAX_UNPACK_TREES]; struct traverse_info newinfo; struct name_entry *p; @@ -346,12 +347,16 @@ static int traverse_trees_recursive(int n, unsigned long dirmask, unsigned long const unsigned char *sha1 = NULL; if (dirmask & 1) sha1 = names[i].sha1; - fill_tree_descriptor(t+i, sha1); + buf[i] = fill_tree_descriptor(t+i, sha1); } bottom = switch_cache_bottom(&newinfo); ret = traverse_trees(n, t, &newinfo); restore_cache_bottom(&newinfo, bottom); + + for (i = 0; i < n; i++) + free(buf[i]); + return ret; }
-- 1.7.2.1.544.ga752d.dirty