Re: [PATCH] Fix memory leak in get_reachable_subset

From: Junio C Hamano <gitster@pobox.com>
Date: Apr 24, 2023, 16:09 UTC
Message-ID: <xmqqbkjdgt36.fsf@gitster.g>
In-Reply-To: <20230421234409.1925489-1-mh@glandium.org>

Mike Hommey <mh@glandium.org> writes:

> ---
>  commit-reach.c | 2 ++
>  1 file changed, 2 insertions(+)

Missing sign-off?

[jc: adding the author of fcb2c076 (commit-reach: implement get_reachable_subset, 2018-11-02) to the CC list]

Show 12 quoted lines

> diff --git a/commit-reach.c b/commit-reach.c
> index 70bde8af05..f15d84566b 100644
> --- a/commit-reach.c
> +++ b/commit-reach.c
> @@ -944,6 +944,8 @@ struct commit_list *get_reachable_subset(struct commit **from, int nr_from,
>  		}
>  	}
>  
> +	clear_prio_queue(&queue);
> +
>  	clear_commit_marks_many(nr_to, to, PARENT1);
>  	clear_commit_marks_many(nr_from, from, PARENT2);

Looking good.