Subject: Re: [PATCH v4 1/2] pack-bitmap: fix memory leak if load_bitmap() failed

From: Junio C Hamano <[email-masked]>
Date: Thu, 29 May 2025 08:33:29 -0700

"Taylor Blau via GitGitGadget" <[email-masked]> writes:

> From: Taylor Blau <[email-masked]>
>
> After going through the "failed" label, load_bitmap() will return -1,
> and its caller (either prepare_bitmap_walk() or prepare_bitmap_git())
> will then call free_bitmap_index().
> ...
> The solution is to remove the error handling code in load_bitmap(), because
> its caller will always call free_bitmap_index() in case of an error.
>
> Signed-off-by: Taylor Blau <[email-masked]>
> ---

As this is Lidong relaying <[email-masked]> that Taylor
sent to the list, shouldn't Lidong's sign-off be after Taylor's?