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;
> return file_remove_suid(file);
Yup, will fix.