From: Colin Ian King <colin.king@xxxxxxxxxxxxx> The variable ret is being assigned a value however this is never read and later it is being reassigned to a new value. The assignment is redundant and hence can be removed. Addresses-Coverity: ("Unused Value") Signed-off-by: Colin Ian King <colin.king@xxxxxxxxxxxxx> --- fs/afs/xattr.c | 1 - 1 file changed, 1 deletion(-) diff --git a/fs/afs/xattr.c b/fs/afs/xattr.c index c81f85003fc7..25cb7ad4fca3 100644 --- a/fs/afs/xattr.c +++ b/fs/afs/xattr.c @@ -71,7 +71,6 @@ static int afs_xattr_get_acl(const struct xattr_handler *handler, if (ret == 0) { ret = acl->size; if (size > 0) { - ret = -ERANGE; if (acl->size > size) return -ERANGE; memcpy(buffer, acl->data, acl->size); -- 2.20.1