diff options
author | Patrick Steinhardt <ps@pks.im> | 2015-12-01 16:19:29 +0300 |
---|---|---|
committer | Patrick Steinhardt <ps@pks.im> | 2015-12-01 16:25:15 +0300 |
commit | 9487585ddc5d7cbb8b9d085e030a01ff805bc41b (patch) | |
tree | 39691fcf493c6571fdf30a02610c02c1a837ab45 | |
parent | 337b2b08f46ea77d61fa66657ad62d8702bc233a (diff) |
tree: mark cloned tree entries as un-pooled
When duplicating a `struct git_tree_entry` with
`git_tree_entry_dup` the resulting structure is not allocated
inside a memory pool. As we do a 1:1 copy of the original struct,
though, we also copy the `pooled` field, which is set to `true`
for pooled entries. This results in a huge memory leak as we
never free tree entries that were duplicated from a pooled
tree entry.
Fix this by marking the newly duplicated entry as un-pooled.
-rw-r--r-- | src/tree.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/tree.c b/src/tree.c index 0a32868cd..0e3738afa 100644 --- a/src/tree.c +++ b/src/tree.c @@ -253,6 +253,8 @@ int git_tree_entry_dup(git_tree_entry **dest, const git_tree_entry *source) memcpy(copy, source, total_size); + copy->pooled = 0; + *dest = copy; return 0; } |