pack-redundant: fix memory leak when open_pack_index() fails
In add_pack(), we allocate l.remaining_objects with llist_init() before calling open_pack_index(). If open_pack_index() fails we return NULL without freeing the allocated list, leaking the memory. Fix by calling llist_free(l.remaining_objects) on the error path before returning. Signed-off-by: Sahitya Chandra <sahityajb@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Sahitya Chandra committed
Feb 21, 2026 at 16:08 UTC
7451864bfac0fc7ac829aceecd7f339b80dac732
1 file changed
+3
-1
builtin/pack-redundant.c
+3
-1
index e4ecf774ca..86749bb7e7 100644
--- a/builtin/pack-redundant.c
+++ b/builtin/pack-redundant.c
@@ -546,8 +546,10 @@ static struct pack_list * add_pack(struct packed_git *p)
l.pack = p;
llist_init(&l.remaining_objects);
- if (open_pack_index(p))
+ if (open_pack_index(p)) {
+ llist_free(l.remaining_objects);
return NULL;
+ }
base = p->index_data;
base += 256 * 4 + ((p->index_version < 2) ? 4 : 8);