diff options
| author | Miklos Szeredi <[email protected]> | 2008-07-01 13:01:26 +0000 |
|---|---|---|
| committer | Al Viro <[email protected]> | 2008-07-27 00:53:25 +0000 |
| commit | 9767d74957450da6365c363d69e3d02d605d7375 (patch) | |
| tree | 913afe491d13b42b0cfcb540569080bba4725dfd /net/unix/af_unix.c | |
| parent | [PATCH] vfs: use kstrdup() and check failing allocation (diff) | |
| download | kernel-9767d74957450da6365c363d69e3d02d605d7375.tar.gz kernel-9767d74957450da6365c363d69e3d02d605d7375.zip | |
[patch 1/4] vfs: utimes: move owner check into inode_change_ok()
Add a new ia_valid flag: ATTR_TIMES_SET, to handle the
UTIMES_OMIT/UTIMES_NOW and UTIMES_NOW/UTIMES_OMIT cases. In these
cases neither ATTR_MTIME_SET nor ATTR_ATIME_SET is in the flags, yet
the POSIX draft specifies that permission checking is performed the
same way as if one or both of the times was explicitly set to a
timestamp.
See the path "vfs: utimensat(): fix error checking for
{UTIME_NOW,UTIME_OMIT} case" by Michael Kerrisk for the patch
introducing this behavior.
This is a cleanup, as well as allowing filesystems (NFS/fuse/...) to
perform their own permission checking instead of the default.
CC: Ulrich Drepper <[email protected]>
CC: Michael Kerrisk <[email protected]>
Signed-off-by: Miklos Szeredi <[email protected]>
Signed-off-by: Al Viro <[email protected]>
Diffstat (limited to 'net/unix/af_unix.c')
0 files changed, 0 insertions, 0 deletions