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 _______________________________________________ xfs mailing list xfs@xxxxxxxxxxx http://oss.sgi.com/mailman/listinfo/xfs