->permission() sanitizing: don't pass flags to ->check_acl()
authorAl Viro <viro@zeniv.linux.org.uk>
Mon, 20 Jun 2011 23:12:17 +0000 (19:12 -0400)
committerAl Viro <viro@zeniv.linux.org.uk>
Wed, 20 Jul 2011 05:43:21 +0000 (01:43 -0400)
not used in the instances anymore.

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
28 files changed:
Documentation/filesystems/Locking
Documentation/filesystems/vfs.txt
fs/9p/acl.c
fs/9p/acl.h
fs/btrfs/acl.c
fs/btrfs/ctree.h
fs/ext2/acl.c
fs/ext2/acl.h
fs/ext3/acl.c
fs/ext3/acl.h
fs/ext4/acl.c
fs/ext4/acl.h
fs/generic_acl.c
fs/gfs2/acl.c
fs/gfs2/acl.h
fs/jffs2/acl.c
fs/jffs2/acl.h
fs/jfs/acl.c
fs/jfs/jfs_acl.h
fs/namei.c
fs/ocfs2/acl.c
fs/ocfs2/acl.h
fs/reiserfs/xattr.c
fs/xfs/linux-2.6/xfs_acl.c
fs/xfs/xfs_acl.h
include/linux/fs.h
include/linux/generic_acl.h
include/linux/reiserfs_xattr.h

index 57d827d6071d6b77e18c2c984ae444ac43adbabb..9b6ed7c9f34f4b5cc4424e4413e2d57c7f3f6c03 100644 (file)
@@ -52,7 +52,7 @@ ata *);
        void (*put_link) (struct dentry *, struct nameidata *, void *);
        void (*truncate) (struct inode *);
        int (*permission) (struct inode *, int, unsigned int);
-       int (*check_acl)(struct inode *, int, unsigned int);
+       int (*check_acl)(struct inode *, int);
        int (*setattr) (struct dentry *, struct iattr *);
        int (*getattr) (struct vfsmount *, struct dentry *, struct kstat *);
        int (*setxattr) (struct dentry *, const char *,const void *,size_t,int);
index 88b9f5519af976e2cb2f472588cf43bd572b920e..8b4c8e04d87979181d21952f62a784b17e527f00 100644 (file)
@@ -334,7 +334,7 @@ struct inode_operations {
         void (*put_link) (struct dentry *, struct nameidata *, void *);
        void (*truncate) (struct inode *);
        int (*permission) (struct inode *, int, unsigned int);
-       int (*check_acl)(struct inode *, int, unsigned int);
+       int (*check_acl)(struct inode *, int);
        int (*setattr) (struct dentry *, struct iattr *);
        int (*getattr) (struct vfsmount *mnt, struct dentry *, struct kstat *);
        int (*setxattr) (struct dentry *, const char *,const void *,size_t,int);
index 94af68b092aff3237c395e204fa5edace2d5b93f..e98f56d3787dff9f68bbd80772f8690f903ee114 100644 (file)
@@ -96,7 +96,7 @@ static struct posix_acl *v9fs_get_cached_acl(struct inode *inode, int type)
        return acl;
 }
 
-int v9fs_check_acl(struct inode *inode, int mask, unsigned int flags)
+int v9fs_check_acl(struct inode *inode, int mask)
 {
        struct posix_acl *acl;
        struct v9fs_session_info *v9ses;
index 7ef3ac9f6d95ba8b2111929c5340ae63512151a9..59e18c2e8c7e6ba3b7e1fd4386155bad519adc9e 100644 (file)
@@ -16,7 +16,7 @@
 
 #ifdef CONFIG_9P_FS_POSIX_ACL
 extern int v9fs_get_acl(struct inode *, struct p9_fid *);
-extern int v9fs_check_acl(struct inode *inode, int mask, unsigned int flags);
+extern int v9fs_check_acl(struct inode *inode, int mask);
 extern int v9fs_acl_chmod(struct dentry *);
 extern int v9fs_set_create_acl(struct dentry *,
                               struct posix_acl *, struct posix_acl *);
index a25a4a2e0df21206b75cb355465c5cbd78851f2c..9f62ab2a7282ed49aeacc560ef8586b09ce37b24 100644 (file)
@@ -195,7 +195,7 @@ out:
        return ret;
 }
 
-int btrfs_check_acl(struct inode *inode, int mask, unsigned int flags)
+int btrfs_check_acl(struct inode *inode, int mask)
 {
        int error = -EAGAIN;
 
index 3b859a3e6a0e9354a653e324f8c08ba85d45f959..9864cec801ed4e5dcc84cda0b380489558e1d362 100644 (file)
@@ -2642,7 +2642,7 @@ do {                                                              \
 
 /* acl.c */
 #ifdef CONFIG_BTRFS_FS_POSIX_ACL
-int btrfs_check_acl(struct inode *inode, int mask, unsigned int flags);
+int btrfs_check_acl(struct inode *inode, int mask);
 #else
 #define btrfs_check_acl NULL
 #endif
index 6b9442d1be5295c593f8f36023879ef33882cd00..bfe651f9ae1631ab0abd40281d92f18974530565 100644 (file)
@@ -232,7 +232,7 @@ ext2_set_acl(struct inode *inode, int type, struct posix_acl *acl)
 }
 
 int
-ext2_check_acl(struct inode *inode, int mask, unsigned int flags)
+ext2_check_acl(struct inode *inode, int mask)
 {
        struct posix_acl *acl;
 
index c939b7b1209905cba3aa6622e04ad48600ede984..3ff6cbb9ac4486e2c6bc4e7b0ca83485ab01fd1f 100644 (file)
@@ -54,7 +54,7 @@ static inline int ext2_acl_count(size_t size)
 #ifdef CONFIG_EXT2_FS_POSIX_ACL
 
 /* acl.c */
-extern int ext2_check_acl (struct inode *, int, unsigned int);
+extern int ext2_check_acl (struct inode *, int);
 extern int ext2_acl_chmod (struct inode *);
 extern int ext2_init_acl (struct inode *, struct inode *);
 
index 0a6940d6c30cfaf4603ddceab880de965e43cd84..edfeb293d4cb601457339ca97d5ceeae64416623 100644 (file)
@@ -240,7 +240,7 @@ ext3_set_acl(handle_t *handle, struct inode *inode, int type,
 }
 
 int
-ext3_check_acl(struct inode *inode, int mask, unsigned int flags)
+ext3_check_acl(struct inode *inode, int mask)
 {
        struct posix_acl *acl;
 
index 5faf8048e906403bb9065c006d3513e438f4d2e7..597334626de93aee68631305d98a823e815058c4 100644 (file)
@@ -54,7 +54,7 @@ static inline int ext3_acl_count(size_t size)
 #ifdef CONFIG_EXT3_FS_POSIX_ACL
 
 /* acl.c */
-extern int ext3_check_acl (struct inode *, int, unsigned int);
+extern int ext3_check_acl (struct inode *, int);
 extern int ext3_acl_chmod (struct inode *);
 extern int ext3_init_acl (handle_t *, struct inode *, struct inode *);
 
index 4f54252e439ed409635048cf3dee77801b3098f7..60d900fcc3dbbee54bba6d8ef4c77c8a3aa3bc39 100644 (file)
@@ -238,7 +238,7 @@ ext4_set_acl(handle_t *handle, struct inode *inode, int type,
 }
 
 int
-ext4_check_acl(struct inode *inode, int mask, unsigned int flags)
+ext4_check_acl(struct inode *inode, int mask)
 {
        struct posix_acl *acl;
 
index dec821168fd49a86ecf7097e508b5df8dac12a46..9d843d5deac402593b8b557e27c6ed25aafe401d 100644 (file)
@@ -54,7 +54,7 @@ static inline int ext4_acl_count(size_t size)
 #ifdef CONFIG_EXT4_FS_POSIX_ACL
 
 /* acl.c */
-extern int ext4_check_acl(struct inode *, int, unsigned int);
+extern int ext4_check_acl(struct inode *, int);
 extern int ext4_acl_chmod(struct inode *);
 extern int ext4_init_acl(handle_t *, struct inode *, struct inode *);
 
index 5976bb1fa4cac93d86ad2a866733ab6c435ddfe4..70e90b4974cedd355467552653eae47ab3b07cd0 100644 (file)
@@ -190,7 +190,7 @@ generic_acl_chmod(struct inode *inode)
 }
 
 int
-generic_check_acl(struct inode *inode, int mask, unsigned int flags)
+generic_check_acl(struct inode *inode, int mask)
 {
        if (mask & MAY_NOT_BLOCK) {
                if (!negative_cached_acl(inode, ACL_TYPE_ACCESS))
index 4d97352d39a105d524455cdd1c69ce17405c27d6..8ef1079f166503e9c4c1a16dd9ea0eb57f2c5875 100644 (file)
@@ -75,7 +75,7 @@ static struct posix_acl *gfs2_acl_get(struct gfs2_inode *ip, int type)
  * Returns: errno
  */
 
-int gfs2_check_acl(struct inode *inode, int mask, unsigned int flags)
+int gfs2_check_acl(struct inode *inode, int mask)
 {
        struct posix_acl *acl;
        int error;
index a93907c8159b78d8eaafa09232574c367e02c124..b522b0cb39eaa12798649c2a8bd2b1e952a746f9 100644 (file)
@@ -16,7 +16,7 @@
 #define GFS2_POSIX_ACL_DEFAULT         "posix_acl_default"
 #define GFS2_ACL_MAX_ENTRIES           25
 
-extern int gfs2_check_acl(struct inode *inode, int mask, unsigned int);
+extern int gfs2_check_acl(struct inode *inode, int mask);
 extern int gfs2_acl_create(struct gfs2_inode *dip, struct inode *inode);
 extern int gfs2_acl_chmod(struct gfs2_inode *ip, struct iattr *attr);
 extern const struct xattr_handler gfs2_xattr_system_handler;
index 952afb59e6f16065849d4486d08cc62a98a7e1db..3675b3cdee8921e9d958338489ab2e00b6e3a58d 100644 (file)
@@ -259,7 +259,7 @@ static int jffs2_set_acl(struct inode *inode, int type, struct posix_acl *acl)
        return rc;
 }
 
-int jffs2_check_acl(struct inode *inode, int mask, unsigned int flags)
+int jffs2_check_acl(struct inode *inode, int mask)
 {
        struct posix_acl *acl;
        int rc;
index 3119f59253d3c178b85ee7b21bb90f7ae7e9836b..5e42de8d954126a46664125678cdd037a76470a7 100644 (file)
@@ -26,7 +26,7 @@ struct jffs2_acl_header {
 
 #ifdef CONFIG_JFFS2_FS_POSIX_ACL
 
-extern int jffs2_check_acl(struct inode *, int, unsigned int);
+extern int jffs2_check_acl(struct inode *, int);
 extern int jffs2_acl_chmod(struct inode *);
 extern int jffs2_init_acl_pre(struct inode *, struct inode *, int *);
 extern int jffs2_init_acl_post(struct inode *);
index 859ae5a92166a24cb35af408bd97a0742b0ebbda..8a0a0666d5a6f141e4f1f5f387d3b69db9a02938 100644 (file)
@@ -114,7 +114,7 @@ out:
        return rc;
 }
 
-int jfs_check_acl(struct inode *inode, int mask, unsigned int flags)
+int jfs_check_acl(struct inode *inode, int mask)
 {
        struct posix_acl *acl;
 
index f9285c4900fa5410f82af317360e1028f4a05e6f..54e07559878d12e2fe77c7b39e4f2114c7a3abdb 100644 (file)
@@ -20,7 +20,7 @@
 
 #ifdef CONFIG_JFS_POSIX_ACL
 
-int jfs_check_acl(struct inode *, int, unsigned int flags);
+int jfs_check_acl(struct inode *, int);
 int jfs_init_acl(tid_t, struct inode *, struct inode *);
 int jfs_acl_chmod(struct inode *inode);
 
index e0624e2f0bba6f9efe150c43776b636519d845f9..560fd1dff1d073594a227cd9be75333070c9c2e7 100644 (file)
@@ -176,9 +176,9 @@ EXPORT_SYMBOL(putname);
 /*
  * This does basic POSIX ACL permission checking
  */
-static int acl_permission_check(struct inode *inode, int mask, unsigned int flags)
+static int acl_permission_check(struct inode *inode, int mask)
 {
-       int (*check_acl)(struct inode *inode, int mask, unsigned int flags);
+       int (*check_acl)(struct inode *inode, int mask);
        unsigned int mode = inode->i_mode;
 
        mask &= MAY_READ | MAY_WRITE | MAY_EXEC | MAY_NOT_BLOCK;
@@ -191,7 +191,7 @@ static int acl_permission_check(struct inode *inode, int mask, unsigned int flag
        else {
                check_acl = inode->i_op->check_acl;
                if (IS_POSIXACL(inode) && (mode & S_IRWXG) && check_acl) {
-                       int error = check_acl(inode, mask, flags);
+                       int error = check_acl(inode, mask);
                        if (error != -EAGAIN)
                                return error;
                }
@@ -231,7 +231,7 @@ int generic_permission(struct inode *inode, int mask, unsigned int flags)
        /*
         * Do the basic POSIX ACL permission checks.
         */
-       ret = acl_permission_check(inode, mask, flags);
+       ret = acl_permission_check(inode, mask);
        if (ret != -EACCES)
                return ret;
 
@@ -327,7 +327,7 @@ static inline int exec_permission(struct inode *inode, unsigned int flags)
                if (likely(!ret))
                        goto ok;
        } else {
-               ret = acl_permission_check(inode, mask, flags);
+               ret = acl_permission_check(inode, mask);
                if (likely(!ret))
                        goto ok;
                if (ret != -EACCES)
index 4b683ccc45063a4cfe595fe496fa080d40622302..1cee970eb55a520e258434125c21466cc6160acb 100644 (file)
@@ -290,7 +290,7 @@ static int ocfs2_set_acl(handle_t *handle,
        return ret;
 }
 
-int ocfs2_check_acl(struct inode *inode, int mask, unsigned int flags)
+int ocfs2_check_acl(struct inode *inode, int mask)
 {
        struct ocfs2_super *osb;
        struct buffer_head *di_bh = NULL;
index 4fe7c9cf4bfb82fcd4b1d829068cd2d9ef2c88bc..5c5d31f05853175476cc0c9ed4c2460c67f40da9 100644 (file)
@@ -26,7 +26,7 @@ struct ocfs2_acl_entry {
        __le32 e_id;
 };
 
-extern int ocfs2_check_acl(struct inode *, int, unsigned int);
+extern int ocfs2_check_acl(struct inode *, int);
 extern int ocfs2_acl_chmod(struct inode *);
 extern int ocfs2_init_acl(handle_t *, struct inode *, struct inode *,
                          struct buffer_head *, struct buffer_head *,
index 6747470ec10396f38e9a5188ba92be426ace5158..6ee3c11aa8d9c91ea8aec4b8de9f53399f675ff6 100644 (file)
@@ -868,7 +868,7 @@ out:
        return err;
 }
 
-int reiserfs_check_acl(struct inode *inode, int mask, unsigned int flags)
+int reiserfs_check_acl(struct inode *inode, int mask)
 {
        struct posix_acl *acl;
        int error = -EAGAIN; /* do regular unix permission checks by default */
index 278e6736135acafddd5305cfd05b77c755fb2de8..a5dcd6a0f1b5fc7470cf74c3554767fe3a382200 100644 (file)
@@ -219,7 +219,7 @@ xfs_set_acl(struct inode *inode, int type, struct posix_acl *acl)
 }
 
 int
-xfs_check_acl(struct inode *inode, int mask, unsigned int flags)
+xfs_check_acl(struct inode *inode, int mask)
 {
        struct xfs_inode *ip;
        struct posix_acl *acl;
index 11dd72070cbb12ab0457bea528f70db969a8b1d5..0135e2a669d78924c2e4261450ad5d17676371b8 100644 (file)
@@ -42,7 +42,7 @@ struct xfs_acl {
 #define SGI_ACL_DEFAULT_SIZE   (sizeof(SGI_ACL_DEFAULT)-1)
 
 #ifdef CONFIG_XFS_POSIX_ACL
-extern int xfs_check_acl(struct inode *inode, int mask, unsigned int flags);
+extern int xfs_check_acl(struct inode *inode, int mask);
 extern struct posix_acl *xfs_get_acl(struct inode *inode, int type);
 extern int xfs_inherit_acl(struct inode *inode, struct posix_acl *default_acl);
 extern int xfs_acl_chmod(struct inode *inode);
index 60c1fe65bb2da000d401ccae736508f5ce5f7ad6..f218b42718aae8056928055257a7dd8690b52f1b 100644 (file)
@@ -1579,7 +1579,7 @@ struct inode_operations {
        struct dentry * (*lookup) (struct inode *,struct dentry *, struct nameidata *);
        void * (*follow_link) (struct dentry *, struct nameidata *);
        int (*permission) (struct inode *, int, unsigned int);
-       int (*check_acl)(struct inode *, int, unsigned int);
+       int (*check_acl)(struct inode *, int);
 
        int (*readlink) (struct dentry *, char __user *,int);
        void (*put_link) (struct dentry *, struct nameidata *, void *);
index 0437e377b555ca0c4f26d4d3c244f189e289929b..574bea4013b618d0816d2280ced3670fa4e75bd4 100644 (file)
@@ -10,6 +10,6 @@ extern const struct xattr_handler generic_acl_default_handler;
 
 int generic_acl_init(struct inode *, struct inode *);
 int generic_acl_chmod(struct inode *);
-int generic_check_acl(struct inode *inode, int mask, unsigned int flags);
+int generic_check_acl(struct inode *inode, int mask);
 
 #endif /* LINUX_GENERIC_ACL_H */
index 1a3ca8f802000d245ac1e258cbbe1a69753045db..424ba6416e7cf2c0af901465d65a2fc9d12c46ef 100644 (file)
@@ -45,7 +45,7 @@ int reiserfs_permission(struct inode *inode, int mask, unsigned int flags);
 
 #ifdef CONFIG_REISERFS_FS_XATTR
 #define has_xattr_dir(inode) (REISERFS_I(inode)->i_flags & i_has_xattr_dir)
-int reiserfs_check_acl(struct inode *inode, int mask, unsigned int flags);
+int reiserfs_check_acl(struct inode *inode, int mask);
 ssize_t reiserfs_getxattr(struct dentry *dentry, const char *name,
                          void *buffer, size_t size);
 int reiserfs_setxattr(struct dentry *dentry, const char *name,