"GOT", but the "O" is a cute, smiling pufferfish. Index | Thread | Search

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

Download raw body.

Thread
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@