From: Stefan Sperling <stsp@stsp.name>
Subject: Re: leak in got_fetch_pack
To: Omar Polo <op@omarpolo.com>
Cc: gameoftrees@openbsd.org
Date: Mon, 14 Feb 2022 12:03:06 +0100

On Mon, Feb 14, 2022 at 09:57:08AM +0100, Omar Polo wrote:
> After this, the only other imsg_clear call in lib/ is repository.c:801
> (offset after patch is applied): i don't think it's needed, but it since
> it's a common cleanup step I thought it wouldn't hurt to keep it anyway.
> 
> 
> (it's really two diffs, one for the missing close and the other for the
> imsg_clear, err, cleanup, sending all together because it's easier.)

Thanks! Diff looks good and tests are passing here. ok stsp@