summaryrefslogtreecommitdiffstats
path: root/fs
diff options
context:
space:
mode:
authorColin Ian King <colin.i.king@gmail.com>2023-06-16 16:20:09 +0200
committerJaegeuk Kim <jaegeuk@kernel.org>2023-06-26 15:21:41 +0200
commit3f8ac7da8c6efd72908e0a16d4a149e79f356a00 (patch)
treee8ad91613d37981c4a3677b4eba7e2d4a8ad8ba3 /fs
parentf2fs: do not issue small discard commands during checkpoint (diff)
downloadlinux-3f8ac7da8c6efd72908e0a16d4a149e79f356a00.tar.xz
linux-3f8ac7da8c6efd72908e0a16d4a149e79f356a00.zip
f2fs: remove redundant assignment to variable err
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>
Diffstat (limited to 'fs')
-rw-r--r--fs/f2fs/recovery.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/fs/f2fs/recovery.c b/fs/f2fs/recovery.c
index f0cf1538389c..4e7d4ceeb084 100644
--- a/fs/f2fs/recovery.c
+++ b/fs/f2fs/recovery.c
@@ -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;
}