From: Junio C Hamano <gitster@pobox.com>
Date: Mon, 09 Oct 2023 17:28:13 GMT
Subject: Re: [PATCH] repack: free existing_cruft array after use
Message-ID: <xmqqttqz1z9u.fsf@gitster.g>
In-Reply-To: <ZSNWZ9+Q2eOpy91A@nand.local>

Taylor Blau <me@ttaylorr.com> writes:

> On Sat, Oct 07, 2023 at 01:20:31PM -0400, Jeff King wrote:
>> On Mon, Oct 02, 2023 at 08:44:32PM -0400, Taylor Blau wrote:
>> ...
>> Coverity (using the just-merged-to-next version of the workflow file!)
>> flagged a leak here. Since the topic (tb/repack-max-cruft-size) is in
>> 'next', I think we'd want this on top:
>
> Woohoo! I'm glad that this is already paying dividends.
> ...
> Thanks, I can't believe I missed this when writing this function. The
> fix looks obviously correct to me, so this has my:
>
>     Acked-by: Taylor Blau <me@ttaylorr.com>
>
> Thanks,

Will queue.  Thanks.