I made open_ctree fail if the chunk tree couldn't be open, which means that fsck now segfaults if it can't open the chunk tree. So fix fsck to check the fs_info we get back from open_ctree_fsinfo to make sure it's valid and exit if it's not instead of segfaulting. Thanks,
Signed-off-by: Josef Bacik <[email protected]> --- cmds-check.c | 5 +++++ 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/cmds-check.c b/cmds-check.c index 35e9177..eef959e 100644 --- a/cmds-check.c +++ b/cmds-check.c @@ -3642,6 +3642,11 @@ int cmd_check(int argc, char **argv) } info = open_ctree_fs_info(argv[optind], bytenr, rw, 1); + if (!info) { + fprintf(stderr, "Couldn't open file system\n"); + return -EIO; + } + uuid_unparse(info->super_copy.fsid, uuidbuf); printf("Checking filesystem on %s\nUUID: %s\n", argv[optind], uuidbuf); -- 1.7.7.6 -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
