[ANNOUNCE] xfs; for-next branch updated to 31bb236

Dave Chinner david at fromorbit.com
Wed Mar 2 15:09:31 CST 2016


On Wed, Mar 02, 2016 at 01:45:59AM -0800, Christoph Hellwig wrote:
> > Mateusz Guzik (1):
> >       [2e83b79] xfs: fix two memory leaks in xfs_attr_list.c error paths
> 
> This didn't seem to address the comment about using a goto for
> error handling?

I looked at it, and there isn't a case for a goto. The allocated
memory is in a variable local to the if branch where the leak was
found. Using a jump-based error handling stack means making
all those variables global, and then having to make the freeing
of it conditional. Simply didn't seem worth it to me.

Cheers,

Dave.
-- 
Dave Chinner
david at fromorbit.com



More information about the xfs mailing list