On 9/27/13 2:36 PM, Roger Willcocks wrote: > > On Fri, 2013-09-27 at 08:01 -0500, Mark Tinguely wrote: >> plain text document attachment >> (xfs-fix-leak-in-xfs_dir2_node_removename.patch) >> Free the memory pointed to by state before returning on error from >> xfs_dir2_node_removename.c >> >> Signed-off-by: Mark Tinguely <tinguely@xxxxxxx> >> --- >> Found by Coverity (134681) in userspace, same patch applies there >> also. >> > > Is the first hunk right ? > > xfs_da_node_lookup_int called as > > error = xfs_da_node_lookup_int(state, &rval); > > and returns with > > *result = retval; > return(0); > > so, on return, error == 0 and rval == an error code. The next lines: > > if (error) > rval = error; > > won't change that. But previously if rval != EEXIST you returned rval. > With the change below, you return error, which is zero. Oh geez. o_O . . . Thanks Roger. 3rd review's the charm? ;) -Eric _______________________________________________ xfs mailing list xfs@xxxxxxxxxxx http://oss.sgi.com/mailman/listinfo/xfs