From: Colin Ian King <colin.i.king@gmail.com>
Date: Fri, 16 Jun 2023 14:20:09 +0000 (+0100)
Subject: f2fs: remove redundant assignment to variable err
X-Git-Url: http://git.maquefel.me/?a=commitdiff_plain;h=3f8ac7da8c6efd72908e0a16d4a149e79f356a00;p=linux.git

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>
---

diff --git a/fs/f2fs/recovery.c b/fs/f2fs/recovery.c
index f0cf1538389cb..4e7d4ceeb084c 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;
 			}