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>

diff --git a/builtin/repack.c b/builtin/repack.c
index a1a893d9524fd175bfe0793f5e1e0e0aea3a3947..69e8f302c0079658955011c1d1db7ad13a9b0579 100644 (file)
--- 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,