ceph: remove redundant variable ino
authorColin Ian King <colin.i.king@gmail.com>
Wed, 18 May 2022 08:55:08 +0000 (09:55 +0100)
committerIlya Dryomov <idryomov@gmail.com>
Wed, 25 May 2022 18:45:14 +0000 (20:45 +0200)
Variable ino is being assigned a value that is never read. The variable
and assignment are redundant, remove it.

Cleans up clang scan build warning:
warning: Although the value stored to 'ino' is used in the enclosing
expression, the value is never actually read from 'ino'
[deadcode.DeadStores]

Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
Reviewed-by: Jeff Layton <jlayton@kernel.org>
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
fs/ceph/mds_client.c

index 23a6c11..e9eb996 100644 (file)
@@ -437,7 +437,7 @@ static int ceph_parse_deleg_inos(void **p, void *end,
        ceph_decode_32_safe(p, end, sets, bad);
        dout("got %u sets of delegated inodes\n", sets);
        while (sets--) {
-               u64 start, len, ino;
+               u64 start, len;
 
                ceph_decode_64_safe(p, end, start, bad);
                ceph_decode_64_safe(p, end, len, bad);
@@ -449,7 +449,7 @@ static int ceph_parse_deleg_inos(void **p, void *end,
                        continue;
                }
                while (len--) {
-                       int err = xa_insert(&s->s_delegated_inos, ino = start++,
+                       int err = xa_insert(&s->s_delegated_inos, start++,
                                            DELEGATED_INO_AVAILABLE,
                                            GFP_KERNEL);
                        if (!err) {