fix jffs2 ACLs on big-endian with 16bit mode_t
authorAl Viro <viro@zeniv.linux.org.uk>
Sat, 23 Jul 2011 22:18:58 +0000 (18:18 -0400)
committerAl Viro <viro@zeniv.linux.org.uk>
Sun, 24 Jul 2011 14:12:01 +0000 (10:12 -0400)
casting int * to mode_t * is not a good thing - on a *lot* of big-endian
architectures mode_t happens to be smaller than int and there it breaks
quite spectaculary...

Fucked-up-by: commit cfc8dc6f6f69ede939e09c2af06a01adee577285
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
fs/jffs2/acl.c
fs/jffs2/acl.h
fs/jffs2/fs.c
fs/jffs2/os-linux.h

index 3675b3cdee8921e9d958338489ab2e00b6e3a58d..f9c302430aa12e68b75835a431d6d3a26b6d383e 100644 (file)
@@ -278,7 +278,7 @@ int jffs2_check_acl(struct inode *inode, int mask)
        return -EAGAIN;
 }
 
-int jffs2_init_acl_pre(struct inode *dir_i, struct inode *inode, int *i_mode)
+int jffs2_init_acl_pre(struct inode *dir_i, struct inode *inode, mode_t *i_mode)
 {
        struct posix_acl *acl, *clone;
        int rc;
@@ -301,7 +301,7 @@ int jffs2_init_acl_pre(struct inode *dir_i, struct inode *inode, int *i_mode)
                clone = posix_acl_clone(acl, GFP_KERNEL);
                if (!clone)
                        return -ENOMEM;
-               rc = posix_acl_create_masq(clone, (mode_t *)i_mode);
+               rc = posix_acl_create_masq(clone, i_mode);
                if (rc < 0) {
                        posix_acl_release(clone);
                        return rc;
index 5e42de8d954126a46664125678cdd037a76470a7..9973073b9c4772bee9d3c3e36174c40fe2b8fba5 100644 (file)
@@ -28,7 +28,7 @@ struct jffs2_acl_header {
 
 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_pre(struct inode *, struct inode *, mode_t *);
 extern int jffs2_init_acl_post(struct inode *);
 
 extern const struct xattr_handler jffs2_acl_access_xattr_handler;
index 46ad619b6124fdd92ab0ac83e439311a58c4ca9b..eeead33d8ef0e617901fc7762891e8d24cde4c8f 100644 (file)
@@ -406,7 +406,7 @@ int jffs2_remount_fs (struct super_block *sb, int *flags, char *data)
 
 /* jffs2_new_inode: allocate a new inode and inocache, add it to the hash,
    fill in the raw_inode while you're at it. */
-struct inode *jffs2_new_inode (struct inode *dir_i, int mode, struct jffs2_raw_inode *ri)
+struct inode *jffs2_new_inode (struct inode *dir_i, mode_t mode, struct jffs2_raw_inode *ri)
 {
        struct inode *inode;
        struct super_block *sb = dir_i->i_sb;
index 9c252835e8e59507c8d9b56ddd985c5ed3e9ed84..526979c607b6b91e5354b16e35f825aa0ef21d11 100644 (file)
@@ -173,7 +173,7 @@ int jffs2_do_setattr (struct inode *, struct iattr *);
 struct inode *jffs2_iget(struct super_block *, unsigned long);
 void jffs2_evict_inode (struct inode *);
 void jffs2_dirty_inode(struct inode *inode, int flags);
-struct inode *jffs2_new_inode (struct inode *dir_i, int mode,
+struct inode *jffs2_new_inode (struct inode *dir_i, mode_t mode,
                               struct jffs2_raw_inode *ri);
 int jffs2_statfs (struct dentry *, struct kstatfs *);
 int jffs2_remount_fs (struct super_block *, int *, char *);