diff options
author | Taylor Blau <me@ttaylorr.com> | 2021-10-26 17:01:26 -0400 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2021-10-28 15:32:14 -0700 |
commit | 655b8561d6b10f22f0e7350df9388110667001af (patch) | |
tree | fb687841f5a1471b248f54569307efcce0d18ff6 /pretty.c | |
parent | pack-bitmap.c: don't leak type-level bitmaps (diff) | |
download | tgif-655b8561d6b10f22f0e7350df9388110667001af.tar.xz |
pack-bitmap.c: more aggressively free in free_bitmap_index()
The function free_bitmap_index() is somewhat lax in what it frees. There
are two notable examples:
- While it does call kh_destroy_oid_map on the "bitmaps" map, which
maps commit OIDs to their corresponding bitmaps, the bitmaps
themselves are not freed. Note here that we recycle already-freed
ewah_bitmaps into a pool, but these are handled correctly by
ewah_pool_free().
- We never bother to free the extended index's "positions" map, which
we always allocate in load_bitmap().
Fix both of these.
Signed-off-by: Taylor Blau <me@ttaylorr.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'pretty.c')
0 files changed, 0 insertions, 0 deletions