fs/ntfs3: remove unnecessary NULL check

This code triggers a Smatch warning:

    fs/ntfs3/fsntfs.c:1606 ntfs_bio_fill_1()
    warn: variable dereferenced before check 'bio' (see line 1591)

The "bio" pointer cannot be NULL so there is no need to check.
Originally there was more extensive NULL checking but it was removed
because bio_alloc() will never fail if it is allowed to sleep.

Remove this check as well.

Fixes: 39146b6f66ba ("ntfs3: remove ntfs_alloc_bio")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Link: https://lore.kernel.org/r/20220128140922.GA29766@kili
Signed-off-by: Jens Axboe <axboe@kernel.dk>

diff --git a/fs/ntfs3/fsntfs.c b/fs/ntfs3/fsntfs.c
index 0660a07c5a96e02f7c6321af6cc62fdaa3cd9b6a..3de5700a9b833ed4003ffca31bb6be199588452d 100644 (file)
--- a/fs/ntfs3/fsntfs.c
+++ b/fs/ntfs3/fsntfs.c
@@ -1603,11 +1603,10 @@ new_bio:
                }
        } while (run_get_entry(run, ++run_idx, NULL, &lcn, &clen));
 
-       if (bio) {
-               if (!err)
-                       err = submit_bio_wait(bio);
-               bio_put(bio);
-       }
+       if (!err)
+               err = submit_bio_wait(bio);
+       bio_put(bio);
+
        blk_finish_plug(&plug);
 out:
        unlock_page(fill);