Re: [PATCH v2 0/1] maintenance: fix two memory leaks

From: Junio C Hamano <gitster@pobox.com>
Date: May 11, 2021, 03:29 UTC
Message-ID: <xmqq5yzqosm0.fsf@gitster.g>
In-Reply-To: <20210510195909.621534-1-lenaic@lhuard.fr>

Lénaïc Huard <lenaic@lhuard.fr> writes:

Show 15 quoted lines

> Hi!
>
> Thank you for the code review.
>
> I’ve moved the `strbuf_release(&line);` closer to the last point where
> `line` is used.
>
> And I’ve included Junio’s patch to address the missing
> `free(local_path)` when `hold_lock_file_for_update()<0`.
>
> Lénaïc Huard (1):
>   maintenance: fix two memory leaks
>
>  builtin/gc.c | 10 ++++++++--
>  1 file changed, 8 insertions(+), 2 deletions(-)

Thanks, will queue.