diff options
| author | Jeff King <peff@peff.net> | 2023-10-07 13:20:31 -0400 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2023-10-09 10:27:34 -0700 |
| commit | c1b754d0597be83439ecc8de2a59a90f35cd4040 (patch) | |
| tree | 33582fcd92cb5745c55304edf4877eaa39dbdb62 | |
| parent | 3c1e2c2113842b8462803ef8c9aca596eacfd3af (diff) | |
| download | git-c1b754d0597be83439ecc8de2a59a90f35cd4040.tar.xz | |
repack: free existing_cruft array after use
We allocate an array of packed_git pointers so that we can sort the list
of cruft packs, but we never free the array, causing a small leak. Note
that we don't need to free the packed_git structs themselves; they're
owned by the repository object.
Signed-off-by: Jeff King <peff@peff.net>
Acked-by: Taylor Blau <me@ttaylorr.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
| -rw-r--r-- | builtin/repack.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/builtin/repack.c b/builtin/repack.c index a1a893d952..69e8f302c0 100644 --- a/builtin/repack.c +++ b/builtin/repack.c @@ -955,6 +955,7 @@ static void collapse_small_cruft_packs(FILE *in, size_t max_size, existing->non_kept_packs.items[i].string); strbuf_release(&buf); + free(existing_cruft); } static int write_cruft_pack(const struct pack_objects_args *args, |