From: Adrian Bunk The Coverity checker wasn't happy seeing a size_t compared with -ENODATA and -ENOSYS. Since the only place where size is set is through the result of reiserfs_xattr_get() which is an int, we could simply make size an int. Signed-off-by: Adrian Bunk Cc: Jeff Mahoney Cc: Chris Mason Cc: Hans Reiser Signed-off-by: Andrew Morton --- fs/reiserfs/xattr_acl.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff -puN fs/reiserfs/xattr_acl.c~reiserfs-xattr_aclcreiserfs_get_acl-make-size-an-int fs/reiserfs/xattr_acl.c --- devel/fs/reiserfs/xattr_acl.c~reiserfs-xattr_aclcreiserfs_get_acl-make-size-an-int 2006-03-11 00:02:41.000000000 -0800 +++ devel-akpm/fs/reiserfs/xattr_acl.c 2006-03-11 00:02:41.000000000 -0800 @@ -182,7 +182,7 @@ struct posix_acl *reiserfs_get_acl(struc { char *name, *value; struct posix_acl *acl, **p_acl; - size_t size; + int size; int retval; struct reiserfs_inode_info *reiserfs_i = REISERFS_I(inode); @@ -206,7 +206,7 @@ struct posix_acl *reiserfs_get_acl(struc return posix_acl_dup(*p_acl); size = reiserfs_xattr_get(inode, name, NULL, 0); - if ((int)size < 0) { + if (size < 0) { if (size == -ENODATA || size == -ENOSYS) { *p_acl = ERR_PTR(-ENODATA); return NULL; _