[patch 1/4] vfs: utimes: move owner check into inode_change_ok()
authorMiklos Szeredi <mszeredi@suse.cz>
Tue, 1 Jul 2008 13:01:26 +0000 (15:01 +0200)
committerAl Viro <viro@zeniv.linux.org.uk>
Sun, 27 Jul 2008 00:53:25 +0000 (20:53 -0400)
commit9767d74957450da6365c363d69e3d02d605d7375
tree913afe491d13b42b0cfcb540569080bba4725dfd
parent88b387824fdaecb6ba0f471acf0aadf7d24739fd
[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 <drepper@redhat.com>
CC: Michael Kerrisk <mtk.manpages@gmail.com>
Signed-off-by: Miklos Szeredi <mszeredi@suse.cz>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
fs/attr.c
fs/utimes.c
include/linux/fs.h