[PATCH v7 12/19] show-branch: free() allocated "head" before return
- From
Ævar Arnfjörð Bjarmason <avarab@gmail.com>
- Date
- Feb 6, 2023, 23:07 UTC
- Message-ID
- <patch-v7-12.19-b8536257029-20230206T230142Z-avarab@gmail.com>
- In-Reply-To
- <cover-v7-00.19-00000000000-20230206T230141Z-avarab@gmail.com>
Stop leaking the "head" variable, which we've been leaking since it was originally added in [1], and in its current form since [2]
1. ed378ec7e85 (Make ref resolution saner, 2006-09-11) 2. d9e557a320b (show-branch: store resolved head in heap buffer, 2017-02-14).
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com> --- builtin/show-branch.c | 1 + 1 file changed, 1 insertion(+)
diff --git a/builtin/show-branch.c b/builtin/show-branch.c index c013abaf942..358ac3e519a 100644 --- a/builtin/show-branch.c +++ b/builtin/show-branch.c @@ -956,5 +956,6 @@ int cmd_show_branch(int ac, const char **av, const char *prefix) if (shown_merge_point && --extra < 0) break; } + free(head); return 0; }
-- 2.39.1.1425.gac85d95d48c