X-Git-Url: http://git.cascardo.info/?a=blobdiff_plain;f=fs%2Futimes.c;h=22307cdf7014ba84887a036356deb54fe235d13d;hb=6f2e0d2c3bf0f8d322ab7516c57340c7189cca02;hp=85c40f4f373d56b3d5d41ed2fe7459759c8a3aeb;hpb=7ab85d4a85160ea2ffc96b1255443cbc83be180f;p=cascardo%2Flinux.git diff --git a/fs/utimes.c b/fs/utimes.c index 85c40f4f373d..22307cdf7014 100644 --- a/fs/utimes.c +++ b/fs/utimes.c @@ -81,26 +81,13 @@ static int utimes_common(struct path *path, struct timespec *times) newattrs.ia_valid |= ATTR_MTIME_SET; } /* - * Tell inode_change_ok(), that this is an explicit time + * Tell setattr_prepare(), that this is an explicit time * update, even if neither ATTR_ATIME_SET nor ATTR_MTIME_SET * were used. */ newattrs.ia_valid |= ATTR_TIMES_SET; } else { - /* - * If times is NULL (or both times are UTIME_NOW), - * then we need to check permissions, because - * inode_change_ok() won't do it. - */ - error = -EACCES; - if (IS_IMMUTABLE(inode)) - goto mnt_drop_write_and_out; - - if (!inode_owner_or_capable(inode)) { - error = inode_permission(inode, MAY_WRITE); - if (error) - goto mnt_drop_write_and_out; - } + newattrs.ia_valid |= ATTR_TOUCH; } retry_deleg: inode_lock(inode); @@ -112,7 +99,6 @@ retry_deleg: goto retry_deleg; } -mnt_drop_write_and_out: mnt_drop_write(path->mnt); out: return error;