summaryrefslogtreecommitdiff
path: root/fs
diff options
context:
space:
mode:
authorDan Carpenter <dan.carpenter@linaro.org>2023-09-14 12:47:44 +0300
committerKent Overstreet <kent.overstreet@linux.dev>2023-10-22 17:10:14 -0400
commit867c1fe0187f6df4dca84a34332e00f21ef80f69 (patch)
treeddc4595495f076ad108b98941e59ec0a86ebb2aa /fs
parent4ba985b84de627ba4f257c9843d0dd7146df2180 (diff)
bcachefs: fix error checking in bch2_fs_alloc()
There is a typo here where it uses ";" instead of "?:". The result is that bch2_fs_fs_io_direct_init() is called unconditionally and the errors from it are not checked. Fixes: 0060c68159fc ("bcachefs: Split up fs-io.[ch]") Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org> Signed-off-by: Kent Overstreet <kent.overstreet@linux.dev> Reviewed-by: Brian Foster <bfoster@redhat.com>
Diffstat (limited to 'fs')
-rw-r--r--fs/bcachefs/super.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/bcachefs/super.c b/fs/bcachefs/super.c
index 2990eed85adf..e94a63a22704 100644
--- a/fs/bcachefs/super.c
+++ b/fs/bcachefs/super.c
@@ -852,7 +852,7 @@ static struct bch_fs *bch2_fs_alloc(struct bch_sb *sb, struct bch_opts opts)
bch2_fs_compress_init(c) ?:
bch2_fs_ec_init(c) ?:
bch2_fs_fsio_init(c) ?:
- bch2_fs_fs_io_buffered_init(c);
+ bch2_fs_fs_io_buffered_init(c) ?:
bch2_fs_fs_io_direct_init(c);
if (ret)
goto err;