On Fri, Jan 07, 2011 at 03:53:57AM -0500, Christoph Hellwig wrote: > On Tue, Jan 04, 2011 at 03:48:37PM +1100, Dave Chinner wrote: > > From: Dave Chinner <dchinner@xxxxxxxxxx> > > > > The buffered IO and direct Io write paths share a common set of > > checks and limiting code prior to issuing the write. Factor that > > into a common helper function. > > Wouldn't it be better to create the helper before duplicating the code, > aka move it before the buffer/direct splits in the series? Maybe, but at this I'd prefer not to move it as it would mean having to rework several other patches in the series as well. Moving it doesn't change the end result at all, so unless you feel really strongly about this I'm going to say no this time because we're now under time pressure to get this tested and finished for 2.6.38. > > + /* > > + * If we're writing the file then make sure to clear the setuid and > > + * setgid bits if the process is not being run by root. This keeps > > + * people from modifying setuid and setgid binaries. > > + */ > > + error = file_remove_suid(file); > > + return error; > > just > > return file_remove_suid(file); Yup, will fix. Cheers, Dave. -- Dave Chinner david@xxxxxxxxxxxxx _______________________________________________ xfs mailing list xfs@xxxxxxxxxxx http://oss.sgi.com/mailman/listinfo/xfs
