On Fri, 8 Feb 2008 13:49:01 -0500 (EST)
david m. richter [EMAIL PROTECTED] wrote:
On Fri, 8 Feb 2008, J. Bruce Fields wrote:
On Fri, Feb 08, 2008 at 07:15:02AM -0500, Jeff Layton wrote:
On Thu, 7 Feb 2008 18:26:18 -0500
J. Bruce Fields [EMAIL PROTECTED] wrote:
On Sun, Jan 20
care of the lockd hang. If
lockd is stuck trying to rpc_ping itself then it probably would hang
like this, wouldn't it?
--
Jeff Layton [EMAIL PROTECTED]
-
To unsubscribe from this list: send the line unsubscribe linux-fsdevel in
the body of a message to [EMAIL PROTECTED]
More majordomo info
On Fri, 8 Feb 2008 16:12:28 -0500
J. Bruce Fields [EMAIL PROTECTED] wrote:
On Fri, Feb 08, 2008 at 03:54:14PM -0500, Jeff Layton wrote:
Interesting. It's not clear me why the underlying filesystem would make
any difference there. Though now that I look, it looks like fl_grant
really only
changes.
4) makes cifs_writepages accurately distinguish between EIO and ENOSPC
errors when writing out pages.
Some simple testing indicates that the patch works as expected and that
it fixes the reproducer for the known problem.
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/cifs/cifsfs.c
, since that should represent the
amount of unread data in the response.
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
Acked-by: Guenter Kukkukk [EMAIL PROTECTED]
---
fs/smbfs/sock.c |5 ++---
1 files changed, 2 insertions(+), 3 deletions(-)
diff --git a/fs/smbfs/sock.c b/fs/smbfs/sock.c
index
on a regular file doesn't mean anything on
Linux. Having it set is harmless, but I could see problems if someone
were for instance, to tar up some files on a CIFS share and untar them
on an OS where it does have meaning.
--
Jeff Layton [EMAIL PROTECTED]
-
To unsubscribe from this list: send the line
Fix some warnings with SMBFS_DEBUG_* builds. This patch makes it so
that builds with -Werror don't fail.
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/smbfs/file.c |7 ---
fs/smbfs/inode.c |2 +-
fs/smbfs/proc.c |2 +-
fs/smbfs/smbiod.c |2 +-
4 files changed, 7
that should represent the
amount of unread data in the response.
I think this is correct, but an ACK or NACK from someone more familiar
with this code would be appreciated...
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/smbfs/sock.c |5 ++---
1 files changed, 2 insertions(+), 3
is being submitted for inclusion into -mm.
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
-
To unsubscribe from this list: send the line unsubscribe linux-fsdevel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Make sure ecryptfs doesn't trip the BUG() in notify_change. This also
allows the lower filesystem to interpret ATTR_KILL_S*ID in its own way.
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/ecryptfs/inode.c |8
1 files changed, 8 insertions(+), 0 deletions(-)
diff --git a/fs
It's theoretically possible for a single SETATTR call to come in that
sets the mode and the uid/gid. In that case, don't set the ATTR_KILL_S*ID
bits since that would trip the BUG() in notify_change. Just fix up the
mode to have the same effect.
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs
Don't allow unionfs_setattr to trip the BUG() in notify_change. Clear
ATTR_MODE if the either ATTR_KILL_S*ID is set. This also allows the
lower filesystem to interpret these bits in its own way.
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/unionfs/inode.c |7 +++
1 files changed
change was only for the purposes of clearing these bits.
This means that we now have an implicit assumption that notify_change is
never called with ATTR_MODE and either ATTR_KILL_S*ID bit set. Nothing
currently enforces that, so this patch also adds a BUG() if that occurs.
Signed-off-by: Jeff Layton
If the ATTR_KILL_S*ID bits are set then any mode change is only for
clearing the setuid/setgid bits. For NFS, skip the mode change and
let the server handle it.
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/nfs/inode.c |4
1 files changed, 4 insertions(+), 0 deletions(-)
diff
If the ATTR_KILL_S*ID bits are set then any mode change is only for
clearing the setuid/setgid bits. For CIFS, skip the mode change and
let the server handle it.
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/cifs/inode.c |5 +
1 files changed, 5 insertions(+), 0 deletions(-)
diff
reiserfs_setattr can call notify_change recursively using the same
iattr struct. This could cause it to trip the BUG() in notify_change.
Fix reiserfs to clear those bits near the beginning of the function.
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/reiserfs/inode.c |6 +-
1
On Mon, 17 Sep 2007 07:29:05 -0400
Jeff Layton [EMAIL PROTECTED] wrote:
This patchset is the latest one for fixing the clearing of setuid/setgid
bits in networked filesystems. It should apply cleanly to 2.6.23-rc4-mm1.
This is basically the same patchset as take 5. The main differences
,
and the comment in notify_change is now a bit more descriptive. I've also
moved the description of the main rationale for the patch into patch 5.
Andrew, would it be possible to go ahead and get this committed to -mm?
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
-
To unsubscribe from this list: send
This patch makes sure ecryptfs doesn't trip the BUG() in notify_change.
It also allows the lower filesystem to interpret ATTR_KILL_S*ID in its
own way.
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/ecryptfs/inode.c |8
1 files changed, 8 insertions(+), 0 deletions(-)
diff
*ID. Just fix up the mode to have the same effect. Also, move
locking the i_mutex to lower in the function. It's not needed until it
checks the i_size.
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/unionfs/inode.c | 10 ++
1 files changed, 6 insertions(+), 4 deletions(-)
diff --git
reiserfs_setattr can call notify_change recursively using the same
iattr struct. This could cause it to trip the BUG() in notify_change.
Fix reiserfs to clear those bits near the beginning of the function.
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/reiserfs/inode.c |6 +-
1
If the ATTR_KILL_S*ID bits are set then any mode change is only for
clearing the setuid/setgid bits. For NFS, skip the mode change and
let the server handle it.
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/nfs/inode.c |4
1 files changed, 4 insertions(+), 0 deletions(-)
diff
change was only for the purposes of clearing these bits.
This means that we now have an implicit assumption that notify_change is
never called with ATTR_MODE and either ATTR_KILL_S*ID bit set. Nothing
currently enforces that, so this patch also adds a BUG() if that occurs.
Signed-off-by: Jeff Layton
If the ATTR_KILL_S*ID bits are set then any mode change is only for
clearing the setuid/setgid bits. For CIFS, skip the mode change and
let the server handle it.
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/cifs/inode.c |5 +
1 files changed, 5 insertions(+), 0 deletions(-)
diff
It's theoretically possible for a single SETATTR call to come in that
sets the mode and the uid/gid. In that case, don't set the ATTR_KILL_S*ID
bits since that would trip the BUG() in notify_change. Just fix up the
mode to have the same effect.
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs
On Fri, 14 Sep 2007 23:09:24 +1000
Greg Banks [EMAIL PROTECTED] wrote:
On Fri, Sep 14, 2007 at 07:02:58AM -0400, Jeff Layton wrote:
On Fri, 14 Sep 2007 20:25:45 +1000
Greg Banks [EMAIL PROTECTED] wrote:
I'm curious about the reasons behind this change. You mention
credential issues
On Sat, 15 Sep 2007 00:40:33 +1000
Greg Banks [EMAIL PROTECTED] wrote:
On Fri, Sep 14, 2007 at 09:38:46AM -0400, Jeff Layton wrote:
On Fri, 14 Sep 2007 23:09:24 +1000
Greg Banks [EMAIL PROTECTED] wrote:
On Fri, Sep 14, 2007 at 07:02:58AM -0400, Jeff Layton wrote:
On Fri, 14 Sep
On Sat, 15 Sep 2007 01:43:45 +1000
Greg Banks [EMAIL PROTECTED] wrote:
On Fri, Sep 14, 2007 at 10:58:38AM -0400, Jeff Layton wrote:
On Sat, 15 Sep 2007 00:40:33 +1000
Greg Banks [EMAIL PROTECTED] wrote:
Ok, you convinced me.
Right. When I was first looking at this, I considered
When making a directory with POSIX mkdir calls, cifs_mkdir does not
respect the umask. This moves the AND'ing of the mode with the umask
to higher in the function so that the POSIX mkdir creates with the
correct mode.
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/cifs/inode.c |2 +-
1
. The only differences are that
the patches have been reordered to make the tree cleanly bisectable,
and the comment in notify_change is now a bit more descriptive.
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
-
To unsubscribe from this list: send the line unsubscribe linux-fsdevel in
the body
Make sure ecryptfs doesn't trip the BUG() in notify_change. This also
allows the lower filesystem to interpret these bits in their own way.
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/ecryptfs/inode.c |8
1 files changed, 8 insertions(+), 0 deletions(-)
diff --git a/fs
It's theoretically possible for a single SETATTR call to come in that
sets the mode and the uid/gid. In that case, assume the mode is
correct and don't set the ATTR_KILL_S*ID bits. Doing so would trip the
BUG() in notify_change.
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/nfsd/vfs.c
reiserfs_setattr can call notify_change recursively using the same
iattr struct. This could cause it to trip the BUG() in notify_change.
Fix reiserfs to clear those bits near the beginning of the function.
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/reiserfs/inode.c |6 +-
1
with both ATTR_MODE
and either of the ATTR_KILL_S*ID bits set, since the filesystem would
have no way to know what part of the mode change was intentional. If
it is called this way, consider it a BUG().
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/attr.c | 26 --
1 files
to lower in the function. It's not needed until it
checks the i_size.
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/unionfs/inode.c | 10 ++
1 files changed, 6 insertions(+), 4 deletions(-)
diff --git a/fs/unionfs/inode.c b/fs/unionfs/inode.c
index 59bb418..f2e7f25 100644
--- a/fs
If the ATTR_KILL_S*ID bits are set then any mode change is only for
clearing the setuid/setgid bits. For CIFS, skip the mode change and
let the server handle it.
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/cifs/inode.c |5 +
1 files changed, 5 insertions(+), 0 deletions(-)
diff
If the ATTR_KILL_S*ID bits are set then any mode change is only for
clearing the setuid/setgid bits. For NFS skip the mode change and
let the server handle it.
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/nfs/inode.c |4
1 files changed, 4 insertions(+), 0 deletions(-)
diff
the same patchset as take 4 with a few extra patches to
fix up the callers of notify_change, and some minor parenthetical
cleanups.
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
-
To unsubscribe from this list: send the line unsubscribe linux-fsdevel in
the body of a message to [EMAIL PROTECTED
with both ATTR_MODE
and either of the ATTR_KILL_S*ID bits set, since the filesystem would
have no way to know what part of the mode change was intentional. If
it is called this way, consider it a BUG().
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/attr.c | 23 +--
1 files
If the ATTR_KILL_S*ID bits are set then any mode change is only for
clearing the setuid/setgid bits. For NFS skip the mode change and
let the server handle it.
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/nfs/inode.c |4
1 files changed, 4 insertions(+), 0 deletions(-)
diff
If the ATTR_KILL_S*ID bits are set then any mode change is only for
clearing the setuid/setgid bits. For NFS, skip the mode change and
let the server handle it.
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/cifs/inode.c |5 +
1 files changed, 5 insertions(+), 0 deletions(-)
diff
Make sure ecryptfs doesn't trip the BUG() in notify_change. This also
allows the lower filesystem to interpret these bits in their own way.
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/ecryptfs/inode.c |8
1 files changed, 8 insertions(+), 0 deletions(-)
diff --git a/fs
It's theoretically possible for a single SETATTR call to come in that
sets the mode and the uid/gid. In that case, assume the mode is
correct and don't set the ATTR_KILL_S*ID bits. Doing so would trip the
BUG() in notify_change.
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/nfsd/vfs.c
reiserfs_setattr can call notify_change recursively using the same
iattr struct. This could cause it to trip the BUG() in notify_change.
Fix reiserfs to clear those bits near the beginning of the function.
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/reiserfs/inode.c |6 +-
1
to lower in the function. It's not needed until it
checks the i_size.
(Jeff Sipek indicated that he was planning to change some of this code,
so this patch may need changes if it goes in after his patchset)
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/unionfs/inode.c | 10 ++
1
into a mode change if the setuid or setgid bits are
currently set in the inode's mode.
To enable this, the patch does the standard processing of these bits on
a set of local variables, and then has these local variables supercede
the values in the iattr when a killattr op isn't defined.
Signed-off-by: Jeff
-off-by: Jeff Layton [EMAIL PROTECTED]
-
To unsubscribe from this list: send the line unsubscribe linux-fsdevel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Add the nfs_killattr inode operation. For NFS, we generally want the server
to handle clearing these bits. So the function is really just a noop.
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/nfs/dir.c |3 +++
fs/nfs/file.c |2 ++
fs/nfs/inode.c | 10
..make it just clear the ATTR_KILL_S*ID bits since we want the
server to handle it.
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/cifs/cifsfs.c |9 +
1 files changed, 9 insertions(+), 0 deletions(-)
diff --git a/fs/cifs/cifsfs.c b/fs/cifs/cifsfs.c
index cabb6a5..c357c55 100644
This updates the VFS docs under the Documentation/ directory to describe
the new killattr inode operation.
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
Documentation/filesystems/Locking |2 ++
Documentation/filesystems/vfs.txt |6 ++
2 files changed, 8 insertions(+), 0 deletions
On Tue, 21 Aug 2007 15:35:08 +1000
Timothy Shimmin [EMAIL PROTECTED] wrote:
Jeff Layton wrote:
This should fix all of the filesystems in the mainline kernels to handle
ATTR_KILL_SUID and ATTR_KILL_SGID correctly. For most of them, this is
just a matter of making sure that they call
On Tue, 21 Aug 2007 17:21:28 -0400
Josef Sipek [EMAIL PROTECTED] wrote:
On Tue, Aug 21, 2007 at 07:35:51AM -0400, Jeff Layton wrote:
On Tue, 21 Aug 2007 15:35:08 +1000
Timothy Shimmin [EMAIL PROTECTED] wrote:
Jeff Layton wrote:
This should fix all of the filesystems in the mainline
This is the only filesystem in -mm but not in mainline that seems to
be affected by this change...
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/reiser4/plugin/inode_ops.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/fs/reiser4/plugin/inode_ops.c b/fs/reiser4
that
notify_change doesn't try to do a second setattr. NFS already clears
these bits, and this just updates a comment to make it clear
that doing this is more than an optimization.
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/cifs/inode.c |5 +
fs/nfs/inode.c |6 +-
2 files changed, 10
.
Comments and suggestions appreciated. Also, please let me know if
I've missed any filesystems that need to be converted...
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
-
To unsubscribe from this list: send the line unsubscribe linux-fsdevel in
the body of a message to [EMAIL PROTECTED]
More
to catch filesystems that
don't handle these bits correctly without breaking them outright.
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/attr.c | 91 ---
include/linux/fs.h |1 +
2 files changed, 72 insertions(+), 20 deletions
. Would that be sufficient to take care of everyone's
concerns?
--
Jeff Layton [EMAIL PROTECTED]
-
To unsubscribe from this list: send the line unsubscribe linux-fsdevel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
with this patch. Is that enough
of an issue to warrant us taking extra steps to deal with it?
Another alternative might be to rename notify_change(). I don't really
like gratuitously breaking the API, though, and that function is
referenced in a lot of documentation. Changing it might be confusing...
--
Jeff
how best to split these up. A coworker
mentioned that Andrew had tossed him back a single patch that
touched several mainline filesystems and asked him to break it
up. I took that to mean that the patches should generally be split
out, but I guess I took that too far ;-)
--
Jeff Layton [EMAIL
to suggestions for
something different.
We should at least add a generic_ prefix to indicate it's a generic
helper valid for most filesystem (and the kerneldoc comment can explain
the details)
Both good suggestions. I'll plan to incorporate them in the next
respin of the set.
Thanks,
--
Jeff
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/affs/inode.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/fs/affs/inode.c b/fs/affs/inode.c
index 4609a6c..1fa2937 100644
--- a/fs/affs/inode.c
+++ b/fs/affs/inode.c
@@ -218,6 +218,7 @@ affs_notify_change(struct
Not sure if this is necessary or desirable for AFS, but adding it for
consistency.
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/afs/inode.c |3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/fs/afs/inode.c b/fs/afs/inode.c
index d196840..d0ad5e6 100644
--- a/fs/afs
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/cifs/inode.c |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/fs/cifs/inode.c b/fs/cifs/inode.c
index dd41677..6fee1fa 100644
--- a/fs/cifs/inode.c
+++ b/fs/cifs/inode.c
@@ -1429,6 +1429,8 @@ int cifs_setattr(struct
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/9p/vfs_inode.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/fs/9p/vfs_inode.c b/fs/9p/vfs_inode.c
index e5c45ee..00fcd4e 100644
--- a/fs/9p/vfs_inode.c
+++ b/fs/9p/vfs_inode.c
@@ -756,6 +756,7 @@ static int
.. and only save off ia_valid once it returns
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/ext2/inode.c |1 +
fs/ext3/inode.c |5 -
fs/ext4/inode.c |5 -
3 files changed, 9 insertions(+), 2 deletions(-)
diff --git a/fs/ext2/inode.c b/fs/ext2/inode.c
index 0079b2c
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/coda/inode.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/fs/coda/inode.c b/fs/coda/inode.c
index 342f4e0..c06d031 100644
--- a/fs/coda/inode.c
+++ b/fs/coda/inode.c
@@ -239,6 +239,7 @@ int coda_setattr(struct dentry
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/hpfs/inode.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/fs/hpfs/inode.c b/fs/hpfs/inode.c
index 85d3e1d..d2b26eb 100644
--- a/fs/hpfs/inode.c
+++ b/fs/hpfs/inode.c
@@ -265,6 +265,7 @@ int hpfs_notify_change(struct
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/hostfs/hostfs_kern.c |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/fs/hostfs/hostfs_kern.c b/fs/hostfs/hostfs_kern.c
index c778620..ea75204 100644
--- a/fs/hostfs/hostfs_kern.c
+++ b/fs/hostfs/hostfs_kern.c
@@ -833,6
that to decide whether to return early, and to pass
a (hopefully) appropriate bitmask to fsnotify_change.
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/attr.c | 54 +--
include/linux/fs.h |1 +
2 files changed, 36 insertions(+), 19
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/gfs2/ops_inode.c |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/fs/gfs2/ops_inode.c b/fs/gfs2/ops_inode.c
index 911c115..8401fc7 100644
--- a/fs/gfs2/ops_inode.c
+++ b/fs/gfs2/ops_inode.c
@@ -994,6 +994,8 @@ static
..and only save off ia_valid after it returns
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/reiserfs/inode.c |6 +-
1 files changed, 5 insertions(+), 1 deletions(-)
diff --git a/fs/reiserfs/inode.c b/fs/reiserfs/inode.c
index ddde489..55b2aea 100644
--- a/fs/reiserfs/inode.c
+++ b
..and only save off ia_valid after it returns
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/sysfs/inode.c |5 -
1 files changed, 4 insertions(+), 1 deletions(-)
diff --git a/fs/sysfs/inode.c b/fs/sysfs/inode.c
index 10d1b52..38f2ba9 100644
--- a/fs/sysfs/inode.c
+++ b/fs/sysfs
..and only save off ia_valid after it returns
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/xfs/linux-2.6/xfs_iops.c |5 -
1 files changed, 4 insertions(+), 1 deletions(-)
diff --git a/fs/xfs/linux-2.6/xfs_iops.c b/fs/xfs/linux-2.6/xfs_iops.c
index 0b5fa12..67fba53 100644
--- a/fs
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
arch/powerpc/platforms/cell/spufs/inode.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/arch/powerpc/platforms/cell/spufs/inode.c
b/arch/powerpc/platforms/cell/spufs/inode.c
index b3d0dd1..d0f6b62 100644
--- a/arch
..seems unlikely that the sb mode would include a setuid bit, but I suppose
it's possible.
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/smbfs/inode.c |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/fs/smbfs/inode.c b/fs/smbfs/inode.c
index 73d1450..f42d191 100644
..and only set ia_valid after it returns
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/hugetlbfs/inode.c |5 -
1 files changed, 4 insertions(+), 1 deletions(-)
diff --git a/fs/hugetlbfs/inode.c b/fs/hugetlbfs/inode.c
index c848a19..3566abb 100644
--- a/fs/hugetlbfs/inode.c
+++ b
Not sure if this is really needed, but I don't think it will hurt anything
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/proc/base.c|3 +++
fs/proc/generic.c |3 +++
fs/proc/proc_sysctl.c |3 +++
3 files changed, 9 insertions(+), 0 deletions(-)
diff --git a/fs/proc
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/ocfs2/file.c |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/fs/ocfs2/file.c b/fs/ocfs2/file.c
index c4034f6..d05d472 100644
--- a/fs/ocfs2/file.c
+++ b/fs/ocfs2/file.c
@@ -972,6 +972,8 @@ int ocfs2_setattr(struct
and call
the helper when I wasn't sure. I've tried to CC the maintainers
for the individual filesystems as well where I could find them,
please let me know if there are others who should be informed.
Comments and suggestions appreciated...
Signed-off-by: Jeff Layton [EMAIL PROTECTED
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/jfs/acl.c |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/fs/jfs/acl.c b/fs/jfs/acl.c
index 4d84bdc..34ca314 100644
--- a/fs/jfs/acl.c
+++ b/fs/jfs/acl.c
@@ -227,6 +227,8 @@ int jfs_setattr(struct dentry *dentry
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/jffs2/fs.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/fs/jffs2/fs.c b/fs/jffs2/fs.c
index 1d3b7a9..5218f04 100644
--- a/fs/jffs2/fs.c
+++ b/fs/jffs2/fs.c
@@ -37,6 +37,7 @@ static int jffs2_do_setattr (struct inode
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/cifs/inode.c |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/fs/cifs/inode.c b/fs/cifs/inode.c
index dd41677..6fee1fa 100644
--- a/fs/cifs/inode.c
+++ b/fs/cifs/inode.c
@@ -1429,6 +1429,8 @@ int cifs_setattr(struct
who should be informed.
Comments and suggestions appreciated...
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
-
To unsubscribe from this list: send the line unsubscribe linux-fsdevel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
.. and only save off ia_valid once it returns
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/ext2/inode.c |1 +
fs/ext3/inode.c |5 -
fs/ext4/inode.c |5 -
3 files changed, 9 insertions(+), 2 deletions(-)
diff --git a/fs/ext2/inode.c b/fs/ext2/inode.c
index 0079b2c
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/coda/inode.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/fs/coda/inode.c b/fs/coda/inode.c
index 342f4e0..c06d031 100644
--- a/fs/coda/inode.c
+++ b/fs/coda/inode.c
@@ -239,6 +239,7 @@ int coda_setattr(struct dentry
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/gfs2/ops_inode.c |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/fs/gfs2/ops_inode.c b/fs/gfs2/ops_inode.c
index 911c115..8401fc7 100644
--- a/fs/gfs2/ops_inode.c
+++ b/fs/gfs2/ops_inode.c
@@ -994,6 +994,8 @@ static
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/fuse/dir.c |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/fs/fuse/dir.c b/fs/fuse/dir.c
index bd5a772..60c8d5e 100644
--- a/fs/fuse/dir.c
+++ b/fs/fuse/dir.c
@@ -1016,6 +1016,8 @@ static int fuse_setattr(struct dentry
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/hostfs/hostfs_kern.c |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/fs/hostfs/hostfs_kern.c b/fs/hostfs/hostfs_kern.c
index c778620..ea75204 100644
--- a/fs/hostfs/hostfs_kern.c
+++ b/fs/hostfs/hostfs_kern.c
@@ -833,6
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/hpfs/inode.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/fs/hpfs/inode.c b/fs/hpfs/inode.c
index 85d3e1d..d2b26eb 100644
--- a/fs/hpfs/inode.c
+++ b/fs/hpfs/inode.c
@@ -265,6 +265,7 @@ int hpfs_notify_change(struct
..and only set ia_valid after it returns
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/hugetlbfs/inode.c |5 -
1 files changed, 4 insertions(+), 1 deletions(-)
diff --git a/fs/hugetlbfs/inode.c b/fs/hugetlbfs/inode.c
index c848a19..3566abb 100644
--- a/fs/hugetlbfs/inode.c
+++ b
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/jffs2/fs.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/fs/jffs2/fs.c b/fs/jffs2/fs.c
index 1d3b7a9..5218f04 100644
--- a/fs/jffs2/fs.c
+++ b/fs/jffs2/fs.c
@@ -37,6 +37,7 @@ static int jffs2_do_setattr (struct inode
Not sure if this is really needed, but I don't think it will hurt anything
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/proc/base.c|3 +++
fs/proc/generic.c |3 +++
fs/proc/proc_sysctl.c |3 +++
3 files changed, 9 insertions(+), 0 deletions(-)
diff --git a/fs/proc
..and only set old_ia_valid after it returns
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/ramfs/file-nommu.c |5 -
1 files changed, 4 insertions(+), 1 deletions(-)
diff --git a/fs/ramfs/file-nommu.c b/fs/ramfs/file-nommu.c
index cad2b7a..c354883 100644
--- a/fs/ramfs/file
..and only save off ia_valid after it returns
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/reiserfs/inode.c |6 +-
1 files changed, 5 insertions(+), 1 deletions(-)
diff --git a/fs/reiserfs/inode.c b/fs/reiserfs/inode.c
index ddde489..55b2aea 100644
--- a/fs/reiserfs/inode.c
+++ b
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
arch/powerpc/platforms/cell/spufs/inode.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/arch/powerpc/platforms/cell/spufs/inode.c
b/arch/powerpc/platforms/cell/spufs/inode.c
index b3d0dd1..d0f6b62 100644
--- a/arch
Signed-off-by: Jeff Layton [EMAIL PROTECTED]
---
fs/9p/vfs_inode.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/fs/9p/vfs_inode.c b/fs/9p/vfs_inode.c
index e5c45ee..00fcd4e 100644
--- a/fs/9p/vfs_inode.c
+++ b/fs/9p/vfs_inode.c
@@ -756,6 +756,7 @@ static int
that all .setattr ops
call the helper explicitly.
I'm working on a patchset now and hope to have something together in a
few days.
--
Jeff Layton [EMAIL PROTECTED]
-
To unsubscribe from this list: send the line unsubscribe linux-fsdevel in
the body of a message to [EMAIL PROTECTED]
More majordomo info
with -EBUSY in
that situation. A new mount option was also introduced (nosharedcache)
that makes it fall back on the previous behavior (not sharing
superblocks between mounts of the same export).
--
Jeff Layton [EMAIL PROTECTED]
-
To unsubscribe from this list: send the line unsubscribe linux-fsdevel
On Wed, 27 Jun 2007 18:15:55 -0400
Trond Myklebust [EMAIL PROTECTED] wrote:
On Tue, 2007-05-29 at 12:47 -0400, Jeff Layton wrote:
I've been looking at issue of clearing setuid/setgid bits when a file
is written to on NFS. Here's the problem in a nutshell:
We have 2 users. test1
On Wed, 6 Jun 2007 09:55:50 +0100
Christoph Hellwig [EMAIL PROTECTED] wrote:
On Tue, Jun 05, 2007 at 03:23:40PM -0400, Jeff Layton wrote:
I recently sent a similar, smaller patch for this problem. After some
discussion with Steve and Shaggy, I think I better understand why cifsd
allows
1 - 100 of 150 matches
Mail list logo