Subject: Re: [PATCH v2 0/2] negotiator/default: avoid stack overflow

From: Derrick Stolee <[email-masked]>
Date: Mon, 1 May 2023 21:49:14 -0400

On 5/1/2023 6:11 PM, Junio C Hamano wrote:
> Han Xin <[email-masked]> writes:
> 
>> This series avoid stack overflow in negotiator/default.c and memory leak
>> in negotiator/skipping.c.
>>
>> Changes since v2:
>> * Rewrite the commit link in the typical format.
>> * Fix the incorrect check for the COMMON bit introduced in v2.
> 
> I see Derrick pointed out a logic error during the review of v2 and
> this round corrects it.  Is everybody happy with this iteration and
> considers it safe to merge it to 'next'?

Sorry for the lack of confirmation on that. I do think the v3
patches are good (the cover letter says v2).

Thanks,
-Stolee