fat: truncate inode timestamp updates in setattr

setattr_copy can't truncate timestamps correctly for
msdos/vfat, so truncate and copy them ourselves.

Link: http://lkml.kernel.org/r/a2b4701b1125573fafaeaae6802050ca86d6f8cc.1538363961.git.sorenson@redhat.com
Signed-off-by: Frank Sorenson <sorenson@redhat.com>
Acked-by: OGAWA Hirofumi <hirofumi@mail.parknet.co.jp>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>

diff --git a/fs/fat/file.c b/fs/fat/file.c
index 4b54384054154e84b950a87608d7bc8da2e720a2..13935ee99e1e53038f458e257d48f11cc7426591 100644 (file)
--- a/fs/fat/file.c
+++ b/fs/fat/file.c
@@ -542,6 +542,18 @@ int fat_setattr(struct dentry *dentry, struct iattr *attr)
                up_write(&MSDOS_I(inode)->truncate_lock);
        }
 
+       /*
+        * setattr_copy can't truncate these appropriately, so we'll
+        * copy them ourselves
+        */
+       if (attr->ia_valid & ATTR_ATIME)
+               fat_truncate_time(inode, &attr->ia_atime, S_ATIME);
+       if (attr->ia_valid & ATTR_CTIME)
+               fat_truncate_time(inode, &attr->ia_ctime, S_CTIME);
+       if (attr->ia_valid & ATTR_MTIME)
+               fat_truncate_time(inode, &attr->ia_mtime, S_MTIME);
+       attr->ia_valid &= ~(ATTR_ATIME|ATTR_CTIME|ATTR_MTIME);
+
        setattr_copy(inode, attr);
        mark_inode_dirty(inode);
 out: