c/mirror_zfs
1
0
Fork 0
mirror of https://git.proxmox.com/git/mirror_zfs.git synced 2024-11-18 10:21:01 +03:00
Code Issues Packages Projects Releases Wiki Activity

Drop redundant POSIX ACL check in zpl_init_acl()

Browse Source 
ZFS_ACLTYPE_POSIXACL has already been tested in zpl_init_acl(),
so no need to test again on POSIX ACL access.

Reviewed by: Brian Behlendorf <behlendorf1@llnl.gov>
Reviewed-by: Tony Hutter <hutter2@llnl.gov>
Signed-off-by: Tomohiro Kusumi <kusumi.tomohiro@gmail.com>
Closes #9009
This commit is contained in:
Tomohiro Kusumi 2019-07-16 08:26:52 +09:00 committed by Tony Hutter
parent d751b12a9d
commit 73e50a7d5d
1 changed files with 4 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
module/zfs/zpl_xattr.c
View File

@ -1130,12 +1130,9 @@ zpl_init_acl(struct inode *ip, struct inode *dir)
return (0); return (0);
if (!S_ISLNK(ip->i_mode)) { if (!S_ISLNK(ip->i_mode)) {
if (ITOZSB(ip)->z_acl_type == ZFS_ACLTYPE_POSIXACL) { acl = zpl_get_acl(dir, ACL_TYPE_DEFAULT);
acl = zpl_get_acl(dir, ACL_TYPE_DEFAULT); if (IS_ERR(acl))
if (IS_ERR(acl)) return (PTR_ERR(acl));
return (PTR_ERR(acl));
}
if (!acl) { if (!acl) {
ip->i_mode &= ~current_umask(); ip->i_mode &= ~current_umask();
ip->i_ctime = current_time(ip); ip->i_ctime = current_time(ip);
@ -1144,7 +1141,7 @@ zpl_init_acl(struct inode *ip, struct inode *dir)
} }
} }
if ((ITOZSB(ip)->z_acl_type == ZFS_ACLTYPE_POSIXACL) && acl) { if (acl) {
umode_t mode; umode_t mode;
if (S_ISDIR(ip->i_mode)) { if (S_ISDIR(ip->i_mode)) {