f2fs: remove redundant assignment to variable err
authorColin Ian King <colin.i.king@gmail.com>
Fri, 16 Jun 2023 14:20:09 +0000 (15:20 +0100)
committerJaegeuk Kim <jaegeuk@kernel.org>
Mon, 26 Jun 2023 13:21:41 +0000 (06:21 -0700)
The assignment to variable err is redundant since the code jumps to
label next and err is then re-assigned a new value on the call to
sanity_check_node_chain. Remove the assignment.

Cleans up clang scan build warning:
fs/f2fs/recovery.c:464:6: warning: Value stored to 'err' is never read [deadcode.DeadStores]

Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
fs/f2fs/recovery.c

index f0cf153..4e7d4ce 100644 (file)
@@ -460,10 +460,8 @@ static int find_fsync_dnodes(struct f2fs_sb_info *sbi, struct list_head *head,
                                                                quota_inode);
                        if (IS_ERR(entry)) {
                                err = PTR_ERR(entry);
-                               if (err == -ENOENT) {
-                                       err = 0;
+                               if (err == -ENOENT)
                                        goto next;
-                               }
                                f2fs_put_page(page, 1);
                                break;
                        }