f2fs: fix to set inode->i_mode correctly for posix_acl_update_mode
authorWeichao Guo <guoweichao@oppo.com>
Mon, 14 Dec 2020 03:54:53 +0000 (11:54 +0800)
committerJaegeuk Kim <jaegeuk@kernel.org>
Wed, 27 Jan 2021 23:20:00 +0000 (15:20 -0800)
commita28d9aa1a2c7c774c38f2da1a662434bc29cb98e
treeb7b2ff3999ccae493d005af77be084de7081433f
parent36218b81f094648d929994399eb6eb5c97b991e5
f2fs: fix to set inode->i_mode correctly for posix_acl_update_mode

We should update the ~S_IRWXUGO part of inode->i_mode in __setattr_copy,
because posix_acl_update_mode updates mode based on inode->i_mode,
which finally overwrites the ~S_IRWXUGO part of i_acl_mode with old i_mode.

Testcase to reproduce this bug:
0. adduser abc
1. mkfs.f2fs /dev/sdd
2. mount -t f2fs /dev/sdd /mnt/f2fs
3. mkdir /mnt/f2fs/test
4. setfacl -m u:abc:r /mnt/f2fs/test
5. chmod +s /mnt/f2fs/test

Signed-off-by: Weichao Guo <guoweichao@oppo.com>
Signed-off-by: Bin Shu <shubin@oppo.com>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
fs/f2fs/file.c