diff options
author | Colin Ian King <colin.i.king@gmail.com> | 2023-06-16 16:20:09 +0200 |
---|---|---|
committer | Jaegeuk Kim <jaegeuk@kernel.org> | 2023-06-26 15:21:41 +0200 |
commit | 3f8ac7da8c6efd72908e0a16d4a149e79f356a00 (patch) | |
tree | e8ad91613d37981c4a3677b4eba7e2d4a8ad8ba3 /fs | |
parent | f2fs: do not issue small discard commands during checkpoint (diff) | |
download | linux-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.c | 4 |
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; } |