squashfs: xattr_lookup sparse fix
authorStephen Hemminger <shemminger@vyatta.com>
Thu, 13 May 2010 23:32:21 +0000 (16:32 -0700)
committerPhillip Lougher <phillip@lougher.demon.co.uk>
Mon, 17 May 2010 19:02:03 +0000 (20:02 +0100)
Sparse detected that unsigned pointer was being passed as int pointer.

Signed-off-by: Stephen Hemminger <shemminger@vyatta.com>
[fixed up to deal with code refactoring]
Signed-off-by: Phillip Lougher <phillip@lougher.demon.co.uk>
fs/squashfs/xattr.h
fs/squashfs/xattr_id.c

index 4a96366bfcc97453f487dab7ba7cd59ee89969f4..9da071ae181c0e7067a66da17abf37964c8c4ba5 100644 (file)
@@ -25,7 +25,7 @@
 extern __le64 *squashfs_read_xattr_id_table(struct super_block *, u64,
                u64 *, int *);
 extern int squashfs_xattr_lookup(struct super_block *, unsigned int, int *,
-               int *, long long *);
+               int *, unsigned long long *);
 #else
 static inline __le64 *squashfs_read_xattr_id_table(struct super_block *sb,
                u64 start, u64 *xattr_table_start, int *xattr_ids)
@@ -35,7 +35,8 @@ static inline __le64 *squashfs_read_xattr_id_table(struct super_block *sb,
 }
 
 static inline int squashfs_xattr_lookup(struct super_block *sb,
-               unsigned int index, int *count, int *size, long long *xattr)
+               unsigned int index, int *count, int *size,
+               unsigned long long *xattr)
 {
        return 0;
 }
index a74101244a980429eb5a3c12d1ee74711d195bf5..cfb41106098fbbe2faef42bc2392c02479539479 100644 (file)
@@ -39,7 +39,7 @@
  * Map xattr id using the xattr id look up table
  */
 int squashfs_xattr_lookup(struct super_block *sb, unsigned int index,
-               int *count, int *size, long long *xattr)
+               int *count, unsigned int *size, unsigned long long *xattr)
 {
        struct squashfs_sb_info *msblk = sb->s_fs_info;
        int block = SQUASHFS_XATTR_BLOCK(index);