Re: [PATCH] remote: plug memory leaks at early returns

From: Junio C Hamano <gitster@pobox.com>
Date: Aug 26, 2024, 15:11 UTC
Message-ID: <xmqqmskznwsp.fsf@gitster.g>
In-Reply-To: <ZswUWj_dg7-kM8b0@tanuki>

Patrick Steinhardt <ps@pks.im> writes:

Show 17 quoted lines

> On Fri, Aug 23, 2024 at 02:13:09PM -0700, Junio C Hamano wrote:
>> René Scharfe <l.s.r@web.de> writes:
>> 
>> > Signed-off-by: René Scharfe <l.s.r@web.de>
>> > ---
>> >  builtin/remote.c | 23 +++++++++++++++--------
>> >  1 file changed, 15 insertions(+), 8 deletions(-)
>> 
>> Looks straight-forward.  Does this allow us to mark any test script
>> as leak-free?  I understand that Patrick has another round of
>> leakfixes topic that is not yet published, and I'd prefer to see us
>> not step each other's toes.
>
> No, this doesn't conflict with anything I have. And even if it did, I'd
> be happy to drop some patches from my local series :)
>
> The changes themselves also look good to me, thanks!

Thanks.  Will mark for "next".