From: Martin Ågren <martin.agren@gmail.com>
Date: Fri, 22 Sep 2017 23:34:50 +0000 (+0200)
Subject: commit: fix memory leak in `reduce_heads()`
X-Git-Tag: v2.15.0-rc0~29^2~4
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=cb7b29eb67772d08e2365ed07ede9d954d0344c1;p=thirdparty%2Fgit.git

commit: fix memory leak in `reduce_heads()`

We don't free the temporary scratch space we use with
`remove_redundant()`. Free it similar to how we do it in
`get_merge_bases_many_0()`.

Signed-off-by: Martin Ågren <martin.agren@gmail.com>
Reviewed-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
---

diff --git a/commit.c b/commit.c
index d3150d6270..f73976bcc9 100644
--- a/commit.c
+++ b/commit.c
@@ -1080,6 +1080,7 @@ struct commit_list *reduce_heads(struct commit_list *heads)
 	num_head = remove_redundant(array, num_head);
 	for (i = 0; i < num_head; i++)
 		tail = &commit_list_insert(array[i], tail)->next;
+	free(array);
 	return result;
 }