On 5/1/2023 6:11 PM, Junio C Hamano wrote: > Han Xin <hanxin.hx@xxxxxxxxxxxxx> 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
