xfs
[Top] [All Lists]

Re: [fuse-devel] [PATCH 01/25] VFS: move attr_kill logic from notify_cha

To: Miklos Szeredi <miklos@xxxxxxxxxx>
Subject: Re: [fuse-devel] [PATCH 01/25] VFS: move attr_kill logic from notify_change into helper function
From: Trond Myklebust <trond.myklebust@xxxxxxxxxx>
Date: Mon, 06 Aug 2007 15:04:23 -0400
Cc: jlayton@xxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, linux-fsdevel@xxxxxxxxxxxxxxx, codalist@xxxxxxxxxxxxxxxxxxxxxxxx, cluster-devel@xxxxxxxxxx, jfs-discussion@xxxxxxxxxxxxxxxxxxxxx, mikulas@xxxxxxxxxxxxxxxxxxxxxxxx, zippel@xxxxxxxxxxxxxx, xfs@xxxxxxxxxxx, joel.becker@xxxxxxxxxx, wli@xxxxxxxxxxxxxx, reiserfs-devel@xxxxxxxxxxxxxxx, dhowells@xxxxxxxxxx, fuse-devel@xxxxxxxxxxxxxxxxxxxxx, jffs-dev@xxxxxxxx, user-mode-linux-user@xxxxxxxxxxxxxxxxxxxxx, v9fs-developer@xxxxxxxxxxxxxxxxxxxxx, linux-ext4@xxxxxxxxxxxxxxx, linux-cifs-client@xxxxxxxxxxxxxxx, ocfs2-devel@xxxxxxxxxxxxxx, bfennema@xxxxxxxxxxxxxxxxxxxxxx
In-reply-to: <E1II7JR-0005BN-00@dorka.pomaz.szeredi.hu>
References: <200708061354.l76Ds6sq002260@dantu.rdu.redhat.com> <E1II6cM-0004yj-00@dorka.pomaz.szeredi.hu> <20070806141333.0f54ab17.jlayton@redhat.com> <E1II7JR-0005BN-00@dorka.pomaz.szeredi.hu>
Sender: xfs-bounce@xxxxxxxxxxx
On Mon, 2007-08-06 at 20:28 +0200, Miklos Szeredi wrote:

> Your patch is changing the API in a very unsafe way, since there will
> be no error or warning on an unconverted fs.  And that could lead to
> security holes.
> 
> If we would rename the setattr method to setattr_new as well as
> changing it's behavior, that would be fine.  But I guess we do not
> want to do that.

Which "unconverted fses"? If we're talking out of tree stuff, then too
bad: it is _their_ responsibility to keep up with kernel changes.

Trond


<Prev in Thread] Current Thread [Next in Thread>