From: Martin Ågren Date: Sun, 5 Nov 2017 20:24:31 +0000 (+0100) Subject: bisect: fix memory leak when returning best element X-Git-Tag: v2.15.1~4^2 X-Git-Url: https://git.lorimer.id.au/gitweb.git/diff_plain/f4e45cb3eb6fad4570ff63eecb37bae8102992fc bisect: fix memory leak when returning best element When `find_bisection()` returns a single list entry, it leaks the other entries. Move the to-be-returned item to the front and free the remainder. Signed-off-by: Martin Ågren Signed-off-by: Junio C Hamano --- diff --git a/bisect.c b/bisect.c index b1941505b2..3756f127b0 100644 --- a/bisect.c +++ b/bisect.c @@ -399,8 +399,12 @@ void find_bisection(struct commit_list **commit_list, int *reaches, /* Do the real work of finding bisection commit. */ best = do_find_bisection(list, nr, weights, find_all); if (best) { - if (!find_all) + if (!find_all) { + list->item = best->item; + free_commit_list(list->next); + best = list; best->next = NULL; + } *reaches = weight(best); } free(weights);