On Tue, Apr 14, 2015 at 11:54:55AM -0500, Eric Sandeen wrote: > On 3/3/15 4:38 AM, Jan Kara wrote: > > Currently XFS calls file_remove_privs() without holding i_mutex. This is > > wrong because that function can end up messing with file permissions and > > security xattrs for which we need i_mutex held. > > > > Fix the problem by grabbing iolock exclusively when we will need to > > change anything in permissions / xattrs. > > > > Signed-off-by: Jan Kara <jack@xxxxxxx> > > This seems like it stands alone, modulo the file_remove_privs function > renaming... should it just be pulled into XFS by Dave? I guess that would > require the renaming patch to be rebased... The file_remove_privs() is introduced earlier in the series. With all the changes pending to xfs_file_aio_write_checks() from the upcoming VFS merge, the DIo write fixes I'll be pushing, this, and other problems Al Viro has noticed, I'm probably going to have to refactor this code for 4.1-rc2 to fix all the bugs we've introduced during integration of all the changes... Cheers, Dave. -- Dave Chinner david@xxxxxxxxxxxxx -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
