[PATCH 0/6] Fix several obvious memory leaks

Jason A. Donenfeld Jason at zx2c4.com
Tue Mar 5 02:50:46 CET 2013


On Mon, Mar 4, 2013 at 7:53 PM, Jason A. Donenfeld <Jason at zx2c4.com> wrote:
> I'm putting these in the lf/memleak branch for now. It's important
> that I spend a bit of time checking these carefully, because the big
> risk we take with free(3)ing with a sledge-hammer is that we introduce
> use-after-frees.

That series ended up being much simpler than I thought it'd be. I like
splitting out the freeing of structs into smaller static functions, as
you've done; they're easily inlinable by the compiler, and it keeps
things cleaner. We still have a long ways to go, but this is a decent
way to get started. I've merged the lf/memleak branch to wip. Thanks a
bunch Lukas.




More information about the CGit mailing list