reiserfs: minimal fix for ACL caching
authorAl Viro <viro@zeniv.linux.org.uk>
Tue, 9 Jun 2009 00:46:54 +0000 (20:46 -0400)
committerAl Viro <viro@zeniv.linux.org.uk>
Wed, 24 Jun 2009 12:17:05 +0000 (08:17 -0400)
reiserfs uses NULL as "unknown" and ERR_PTR(-ENODATA) as "no ACL";
several codepaths store the former instead of the latter.

All those codepaths go through iset_acl() and all cases when it's
called with NULL acl are for the second variety, so the minimal
fix is to teach iset_acl() to deal with that.

Proper fix is to switch to more usual conventions and avoid back
and forth between internally used ERR_PTR(-ENODATA) and NULL
expected by the rest of the kernel.

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
fs/reiserfs/xattr_acl.c

index c303c426fe2ba6ffec8d708323026f36fbd40f90..a1a7e3530e17dc1f891985e1bd3cb9a8a32e34fc 100644 (file)
@@ -194,7 +194,7 @@ static inline void iset_acl(struct inode *inode, struct posix_acl **i_acl,
        spin_lock(&inode->i_lock);
        if (*i_acl != ERR_PTR(-ENODATA))
                posix_acl_release(*i_acl);
-       *i_acl = posix_acl_dup(acl);
+       *i_acl = acl ? posix_acl_dup(acl) : ERR_PTR(-ENODATA);
        spin_unlock(&inode->i_lock);
 }