ocfs2: add ocfs2_check_acl
authorTiger Yang <tiger.yang@oracle.com>
Fri, 14 Nov 2008 03:17:18 +0000 (11:17 +0800)
committerMark Fasheh <mfasheh@suse.com>
Mon, 5 Jan 2009 16:34:20 +0000 (08:34 -0800)
This function is used to enhance permission checking with POSIX ACLs.

Signed-off-by: Tiger Yang <tiger.yang@oracle.com>
Signed-off-by: Mark Fasheh <mfasheh@suse.com>
fs/ocfs2/acl.c
fs/ocfs2/acl.h
fs/ocfs2/file.c

index 62d0faad600b32d04bdb7931ebc2c8ba262ebb88..a6a2bf6d684504ccd8d2aa1c5abcb1bb69594392 100644 (file)
@@ -230,6 +230,21 @@ static int ocfs2_set_acl(handle_t *handle,
        return ret;
 }
 
+int ocfs2_check_acl(struct inode *inode, int mask)
+{
+       struct posix_acl *acl = ocfs2_get_acl(inode, ACL_TYPE_ACCESS);
+
+       if (IS_ERR(acl))
+               return PTR_ERR(acl);
+       if (acl) {
+               int ret = posix_acl_permission(inode, acl, mask);
+               posix_acl_release(acl);
+               return ret;
+       }
+
+       return -EAGAIN;
+}
+
 static size_t ocfs2_xattr_list_acl_access(struct inode *inode,
                                          char *list,
                                          size_t list_len,
index 1b39f3e14c1bd600d6d6c1b8d5f426a2f98b6d21..fef10f1b782b2213f496edd9798f58b91fa45217 100644 (file)
@@ -26,4 +26,14 @@ struct ocfs2_acl_entry {
        __le32 e_id;
 };
 
+#ifdef CONFIG_OCFS2_FS_POSIX_ACL
+
+extern int ocfs2_check_acl(struct inode *, int);
+
+#else /* CONFIG_OCFS2_FS_POSIX_ACL*/
+
+#define ocfs2_check_acl NULL
+
+#endif /* CONFIG_OCFS2_FS_POSIX_ACL*/
+
 #endif /* OCFS2_ACL_H */
index 360549161e2018308edc4152a457614d10ec924c..7bad7d9b9a2c23f5b697495703e722a9572f8523 100644 (file)
@@ -56,6 +56,7 @@
 #include "suballoc.h"
 #include "super.h"
 #include "xattr.h"
+#include "acl.h"
 
 #include "buffer_head_io.h"
 
@@ -1035,7 +1036,7 @@ int ocfs2_permission(struct inode *inode, int mask)
                goto out;
        }
 
-       ret = generic_permission(inode, mask, NULL);
+       ret = generic_permission(inode, mask, ocfs2_check_acl);
 
        ocfs2_inode_unlock(inode, 0);
 out: