diff options
author | Patrick Steinhardt <ps@pks.im> | 2024-08-14 08:52:03 +0200 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2024-08-14 10:07:57 -0700 |
commit | c81dcf630cec64d0042f08f7210300671ed5b926 (patch) | |
tree | 1b9c4245155ba6f3d69f0406b369d88f9866448c /builtin/commit.c | |
parent | 9ddd5f755d78dd2a3119b63d5f78ca93e4fd3eba (diff) |
bulk-checkin: fix leaking state TODO
When flushing a bulk-checking to disk we also reset the `struct
bulk_checkin_packfile` state. But while we free some of its members,
others aren't being free'd, leading to memory leaks:
- The temporary packfile name is not getting freed.
- The `struct hashfile` only gets freed in case we end up calling
`finalize_hashfile()`. There are code paths though where that is not
the case, namely when nothing has been written. For this, we need to
make `free_hashfile()` public.
Fix those leaks.
Signed-off-by: Patrick Steinhardt <ps@pks.im>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin/commit.c')
0 files changed, 0 insertions, 0 deletions