On Tue, Oct 25, 2011 at 12:52:09PM -0500, Eric Sandeen wrote: > ext4_mark_iloc_dirty() says: > > * The caller must have previously called ext4_reserve_inode_write(). > * Give this, we know that the caller already has write access to iloc->bh. > > ext4_xattr_set_handle, however, just open-codes it. May as well use > the helper function for consistency. > > No bug here, just tidiness. > > (Note: on cleanup path, ext4_reserve_inode_write sets > the bh to NULL if it returns an error, and brelse() of > a null bh is handled gracefully). > > Signed-off-by: Eric Sandeen <sandeen@xxxxxxxxxx> Applied, thanks. - Ted -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html