diff options
| author | Ævar Arnfjörð Bjarmason <avarab@gmail.com> | 2023-02-07 00:07:38 +0100 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2023-02-07 00:34:36 +0100 |
| commit | e8ed0a8ac5ecb4379018e78188ed3ff489c7cfc5 (patch) | |
| tree | 430ba36aa46a74354d7aed9851de6b378d497f9a /builtin | |
| parent | bundle.c: don't leak the "args" in the "struct child_process" (diff) | |
| download | git-e8ed0a8ac5ecb4379018e78188ed3ff489c7cfc5.tar.xz git-e8ed0a8ac5ecb4379018e78188ed3ff489c7cfc5.zip | |
commit-graph: use free_commit_graph() instead of UNLEAK()
In 0bfb48e6723 (builtin/commit-graph.c: UNLEAK variables, 2018-10-03)
this was made to UNLEAK(), but we can just as easily invoke the
free_commit_graph() function added in c3756d5b7fc (commit-graph: add
free_commit_graph, 2018-07-11) instead.
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin')
| -rw-r--r-- | builtin/commit-graph.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/builtin/commit-graph.c b/builtin/commit-graph.c index e8f77f535f..0102ac8540 100644 --- a/builtin/commit-graph.c +++ b/builtin/commit-graph.c @@ -67,6 +67,7 @@ static int graph_verify(int argc, const char **argv, const char *prefix) int fd; struct stat st; int flags = 0; + int ret; static struct option builtin_commit_graph_verify_options[] = { OPT_BOOL(0, "shallow", &opts.shallow, @@ -111,8 +112,9 @@ static int graph_verify(int argc, const char **argv, const char *prefix) if (!graph) return !!open_ok; - UNLEAK(graph); - return verify_commit_graph(the_repository, graph, flags); + ret = verify_commit_graph(the_repository, graph, flags); + free_commit_graph(graph); + return ret; } extern int read_replace_refs; |