Re: [PATCH 0/2] misc kernel leak patches

Eric Sandeen
2013-10-02T20:54:07Z

On 10/2/13 7:51 AM, Mark Tinguely wrote:
> Here are a couple patches to free leaked memory.
>
> First patch is v2 of a Coverity found leak in xfs_dir2_node_removename().
>
> Second patch arose from an eariler leak found by Coverity. In looking
> at that patch, Eric noted that it appeared that the transaction's
> item list is also leaked if recovery fails. I tested this patch by forcing
> errors into the recovery code to verify that the items are now being released.
>
> I will send patches for xfsprogs equivalent.

Good plan w.r.t. kernel first, progs 2nd, I think. Makes it less likely
to accidentally send a free(); or similar upstream. :)

-Eric