commit: fix memory leak in `reduce_heads()`
authorMartin Ågren <redacted>
Fri, 22 Sep 2017 23:34:50 +0000 (01:34 +0200)
committerJunio C Hamano <redacted>
Sun, 24 Sep 2017 01:05:51 +0000 (10:05 +0900)
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 <redacted>
Reviewed-by: Jeff King <redacted>
Signed-off-by: Junio C Hamano <redacted>
commit.c

index d3150d627071316accafb7d9c69b5988050f4a8e..f73976bcc9bafe6de3b7d91e12aff1308b0c9210 100644 (file)
--- 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;
 }
 
git clone https://git.99rst.org/PROJECT