f2fs: remove redundant assignment to variable err
authorColin Ian King <colin.i.king@gmail.com>
Wed, 19 Mar 2025 11:30:10 +0000 (11:30 +0000)
committerJaegeuk Kim <jaegeuk@kernel.org>
Thu, 10 Apr 2025 03:59:57 +0000 (03:59 +0000)
The variable err is being assigned a value zero and then the following
goto page_hit reassigns err a new value. The zero assignment is redundant
and can be removed.

Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
[Jaegeuk Kim: clean up braces and if condition, suggested by Dan Carpenter]
Reviewed-by: Chao Yu <chao@kernel.org>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
fs/f2fs/node.c

index 5f15c224bf782ef869689de3d0163afae673e46f..3f6f5e54759ced18516c32f24c521dcfe456ed45 100644 (file)
@@ -1494,12 +1494,10 @@ repeat:
                return folio;
 
        err = read_node_page(&folio->page, 0);
-       if (err < 0) {
+       if (err < 0)
                goto out_put_err;
-       } else if (err == LOCKED_PAGE) {
-               err = 0;
+       if (err == LOCKED_PAGE)
                goto page_hit;
-       }
 
        if (parent)
                f2fs_ra_node_pages(parent, start + 1, MAX_RA_NODE);