On Tue 11-07-17 16:31:24, Jaegeuk Kim wrote: > This patch copies commit b7f8a09f80: > "btrfs: Don't clear SGID when inheriting ACLs" written by Jan. > > Fixes: 073931017b49d9458aa351605b43a7e34598caef > CC: [email protected] > Signed-off-by: Jan Kara <[email protected]> > Signed-off-by: Jaegeuk Kim <[email protected]>
Ah, somehow I've missed f2fs when going through filesystems that needed fixing up. Thanks for catching that. The patch looks good to me. It would just look a bit cleaner to me to move posix_acl_update_mode() call out of __f2fs_set_acl() into f2fs_set_acl() similarly as we do in other filesystems. But whatever, you are the maintainer ;) Feel free to add: Reviewed-by: Jan Kara <[email protected]> Honza > --- > > Change log from v1: > - fix for simple change > > fs/f2fs/acl.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/fs/f2fs/acl.c b/fs/f2fs/acl.c > index a140c5e3dc54..b4b8438c42ef 100644 > --- a/fs/f2fs/acl.c > +++ b/fs/f2fs/acl.c > @@ -211,7 +211,7 @@ static int __f2fs_set_acl(struct inode *inode, int type, > switch (type) { > case ACL_TYPE_ACCESS: > name_index = F2FS_XATTR_INDEX_POSIX_ACL_ACCESS; > - if (acl) { > + if (acl && !ipage) { > error = posix_acl_update_mode(inode, &inode->i_mode, > &acl); > if (error) > return error; > -- > 2.13.0.rc1.294.g07d810a77f-goog > > -- Jan Kara <[email protected]> SUSE Labs, CR ------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, Slashdot.org! http://sdm.link/slashdot _______________________________________________ Linux-f2fs-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel
