Re: [fuse-devel] [PATCH 00/25] move handling of setuid/gid bitsfrom VFS into individual setattr functions (RESEND)

From: Jeff Layton
Date: Thu Aug 09 2007 - 07:56:27 EST


On Wed, 8 Aug 2007 22:05:13 +0200 (CEST)
Jan Engelhardt <jengelh@xxxxxxxxxxxxxxx> wrote:

>
> On Aug 8 2007 09:48, Andrew Morton wrote:
> >> > On Mon, 6 Aug 2007 09:54:03 -0400
> >> > Jeff Layton <jlayton@xxxxxxxxxx> wrote:
> >> >
> >> > Is there any way in which we can prevent these problems? Say
> >> >
> >> > - rename something so that unconverted filesystems will reliably fail to
> >> > compile?
> >> >
> >>
> >> I suppose we could rename the .setattr inode operation to something
> >> else, but then we'll be stuck with it for at least a while. That seems
> >> sort of kludgey too...
> >
> >Sure. We're changing the required behaviour of .setattr. Changing its
> >name is a fine and reasonably reliable way to communicate that fact.
>
> Maybe ->chattr/->chgattr?
>
>

That seems like a good replacement name. :-)

Now that I think on this further though, maybe Trond's suggestion to
change how the return code works is the best one. That would
(hopefully) catch this problem at runtime, so if someone is using a
precompiled but unconverted module then that would be detected too.

--
Jeff Layton <jlayton@xxxxxxxxxx>
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/