Commit 3e5aeec0 authored by Maxime Bizon's avatar Maxime Bizon Committed by Al Viro

cramfs: fix usage on non-MTD device

When both CONFIG_CRAMFS_MTD and CONFIG_CRAMFS_BLOCKDEV are enabled, if
we fail to mount on MTD, we don't try on block device.

Note: this relies upon cramfs_mtd_fill_super() leaving no side
effects on fc state in case of failure; in general, failing
get_tree_...() does *not* mean "fine to try again"; e.g. parsed
options might've been consumed by fill_super callback and freed
on failure.

Fixes: 74f78fc5 ("vfs: Convert cramfs to use the new mount API")
Signed-off-by: default avatarMaxime Bizon <mbizon@freebox.fr>
Signed-off-by: default avatarNicolas Pitre <nico@fluxnic.net>
Signed-off-by: default avatarAl Viro <viro@zeniv.linux.org.uk>
parent 762c6968
...@@ -958,8 +958,8 @@ static int cramfs_get_tree(struct fs_context *fc) ...@@ -958,8 +958,8 @@ static int cramfs_get_tree(struct fs_context *fc)
if (IS_ENABLED(CONFIG_CRAMFS_MTD)) { if (IS_ENABLED(CONFIG_CRAMFS_MTD)) {
ret = get_tree_mtd(fc, cramfs_mtd_fill_super); ret = get_tree_mtd(fc, cramfs_mtd_fill_super);
if (ret < 0) if (!ret)
return ret; return 0;
} }
if (IS_ENABLED(CONFIG_CRAMFS_BLOCKDEV)) if (IS_ENABLED(CONFIG_CRAMFS_BLOCKDEV))
ret = get_tree_bdev(fc, cramfs_blkdev_fill_super); ret = get_tree_bdev(fc, cramfs_blkdev_fill_super);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment