jfs: remove redundant assignments to ipaimap and ipaimap2
authorColin Ian King <colin.i.king@gmail.com>
Tue, 18 Oct 2022 15:23:10 +0000 (16:23 +0100)
committerDave Kleikamp <dave.kleikamp@oracle.com>
Tue, 18 Oct 2022 15:41:59 +0000 (10:41 -0500)
The pointers ipaimap and ipaimap2 are re-assigned with values a second
time with the same values when they were initialized. The re-assignments
are redundant and can be removed.

Cleans up two clang scan build warnings:
fs/jfs/jfs_umount.c:42:16: warning: Value stored to 'ipaimap' during
its initialization is never read [deadcode.DeadStores]

fs/jfs/jfs_umount.c:43:16: warning: Value stored to 'ipaimap2' during
its initialization is never read [deadcode.DeadStores]

Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
Signed-off-by: Dave Kleikamp <dave.kleikamp@oracle.com>
fs/jfs/jfs_umount.c

index 3e8b13e..95ebcd1 100644 (file)
@@ -68,7 +68,6 @@ int jfs_umount(struct super_block *sb)
        /*
         * close secondary aggregate inode allocation map
         */
-       ipaimap2 = sbi->ipaimap2;
        if (ipaimap2) {
                diUnmount(ipaimap2, 0);
                diFreeSpecial(ipaimap2);
@@ -78,7 +77,6 @@ int jfs_umount(struct super_block *sb)
        /*
         * close aggregate inode allocation map
         */
-       ipaimap = sbi->ipaimap;
        diUnmount(ipaimap, 0);
        diFreeSpecial(ipaimap);
        sbi->ipaimap = NULL;