Skip to content

Commit

Permalink
jfs: cache NULL when both default_acl and acl are NULL
Browse files Browse the repository at this point in the history
default_acl and acl of newly created inode will be initiated
as ACL_NOT_CACHED in vfs function inode_init_always() and later
will be updated by calling xxx_init_acl() in specific filesystems.
Howerver, when default_acl and acl are NULL then they keep the value
of ACL_NOT_CACHED, this patch tries to cache NULL for acl/default_acl
in this case.

Signed-off-by: Chengguang Xu <[email protected]>
Signed-off-by: Dave Kleikamp <[email protected]>
  • Loading branch information
Chengguang Xu authored and kleikamp committed Sep 6, 2018
1 parent be65e25 commit e8d4cee
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions fs/jfs/acl.c
Original file line number Diff line number Diff line change
Expand Up @@ -146,12 +146,16 @@ int jfs_init_acl(tid_t tid, struct inode *inode, struct inode *dir)
if (default_acl) {
rc = __jfs_set_acl(tid, inode, ACL_TYPE_DEFAULT, default_acl);
posix_acl_release(default_acl);
} else {
inode->i_default_acl = NULL;
}

if (acl) {
if (!rc)
rc = __jfs_set_acl(tid, inode, ACL_TYPE_ACCESS, acl);
posix_acl_release(acl);
} else {
inode->i_acl = NULL;
}

JFS_IP(inode)->mode2 = (JFS_IP(inode)->mode2 & 0xffff0000) |
Expand Down

0 comments on commit e8d4cee

Please sign in to comment.