Skip to content

Commit

Permalink
f2fs: remove redundant assignment to variable err
Browse files Browse the repository at this point in the history
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>
  • Loading branch information
Colin Ian King authored and Jaegeuk Kim committed Jun 26, 2023
1 parent 9ac00e7 commit 3f8ac7d
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions fs/f2fs/recovery.c
Original file line number Diff line number Diff line change
Expand Up @@ -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;
}
Expand Down

0 comments on commit 3f8ac7d

Please sign in to comment.