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 /.gitmodules | |
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 '.gitmodules')
0 files changed, 0 insertions, 0 deletions