diff options
| author |  Jonathan Tan <jonathantanmy@google.com> | 2020-05-04 12:13:24 -0700 |
|---|---|---|
| committer |  Junio C Hamano <gitster@pobox.com> | 2020-05-04 14:08:38 -0700 |
| commit | fbda77c6c00a7cec65c4f266b09252df73a9deff (patch) | |
| tree | f36de609ab516a9651fec7f7fdc94d04eb438546 /t/helper/test-mktemp.c | |
| parent | bloom: ignore renames when computing changed paths (diff) | |
| download | tgif-fbda77c6c00a7cec65c4f266b09252df73a9deff.tar.xz | |
commit-graph: avoid memory leaks
A fuzzer running on the entry point provided by fuzz-commit-graph.c
revealed a memory leak when parse_commit_graph() creates a struct
bloom_filter_settings and then returns early due to error. Fix that
error by always freeing that struct first (if it exists) before
returning early due to error.
While making that change, I also noticed another possible memory leak -
when the BLOOMDATA chunk is provided but not BLOOMINDEXES. Also fix that
error.
Signed-off-by: Jonathan Tan <jonathantanmy@google.com>
Reviewed-by: Derrick Stolee <dstolee@microsoft.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/helper/test-mktemp.c')
0 files changed, 0 insertions, 0 deletions