diff options
| author | Yuezhang Mo <Yuezhang.Mo@sony.com> | 2023-10-31 10:36:39 +0100 |
|---|---|---|
| committer | Namjae Jeon <linkinjeon@kernel.org> | 2023-11-03 14:24:11 +0100 |
| commit | 1373ca10ec04afba9199de1fab01fde91338a78b (patch) | |
| tree | 002c00fe274b4d33dd54eb6423b8a22dd107c562 /fs | |
| parent | exfat: fix setting uninitialized time to ctime/atime (diff) | |
| download | linux-1373ca10ec04afba9199de1fab01fde91338a78b.tar.xz linux-1373ca10ec04afba9199de1fab01fde91338a78b.zip | |
exfat: fix ctime is not updated
Commit 4c72a36edd54 ("exfat: convert to new timestamp accessors")
removed attr_copy() from exfat_set_attr().
It causes xfstests generic/221 to fail. In xfstests generic/221,
it tests ctime should be updated even if futimens() update atime
only. But in this case, ctime will not be updated if attr_copy()
is removed.
attr_copy() may also update other attributes, and removing it may
cause other bugs, so this commit restores to call attr_copy() in
exfat_set_attr().
Fixes: 4c72a36edd54 ("exfat: convert to new timestamp accessors")
Signed-off-by: Yuezhang Mo <Yuezhang.Mo@sony.com>
Reviewed-by: Andy Wu <Andy.Wu@sony.com>
Reviewed-by: Aoyama Wataru <wataru.aoyama@sony.com>
Reviewed-by: Sungjong Seo <sj1557.seo@samsung.com>
Signed-off-by: Namjae Jeon <linkinjeon@kernel.org>
Diffstat (limited to 'fs')
| -rw-r--r-- | fs/exfat/file.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/fs/exfat/file.c b/fs/exfat/file.c index 02c4e2937879..bfdfafe00993 100644 --- a/fs/exfat/file.c +++ b/fs/exfat/file.c @@ -295,6 +295,7 @@ int exfat_setattr(struct mnt_idmap *idmap, struct dentry *dentry, if (attr->ia_valid & ATTR_SIZE) inode_set_mtime_to_ts(inode, inode_set_ctime_current(inode)); + setattr_copy(&nop_mnt_idmap, inode, attr); exfat_truncate_inode_atime(inode); if (attr->ia_valid & ATTR_SIZE) { |