If checksum root is corrupted, fsck will get segmentation. This
is because if we fail to load checksum root, root's node is NULL which
cause NULL pointer deferences later.
To fix this problem, we just did something like extent tree rebuilding.
Allocate a new one and clear uptodate flag. We will do sanity check
before fsck going on.
Signed-off-by: Wang Shilong <wangsl.f...@cn.fujitsu.com>
---
v1->v2: fix typo for output message.
---
cmds-check.c | 5 +++++
disk-io.c | 7 +++++++
2 files changed, 12 insertions(+)
diff --git a/cmds-check.c b/cmds-check.c
index 0e4e042..ad5514e 100644
--- a/cmds-check.c
+++ b/cmds-check.c
@@ -6963,6 +6963,11 @@ int cmd_check(int argc, char **argv)
ret = -EIO;
goto close_out;
}
+ if (!extent_buffer_uptodate(info->csum_root->node)) {
+ fprintf(stderr, "Checksum root corrupted, rerun with
--init-csum-tree option\n");
+ ret = -EIO;
+ goto close_out;
+ }
fprintf(stderr, "checking extents\n");
ret = check_chunks_and_extents(root);
diff --git a/disk-io.c b/disk-io.c
index 63e153d..bbfd8e7 100644
--- a/disk-io.c
+++ b/disk-io.c
@@ -914,6 +914,13 @@ int btrfs_setup_all_roots(struct btrfs_fs_info *fs_info,
u64 root_tree_bytenr,
printk("Couldn't setup csum tree\n");
if (!(flags & OPEN_CTREE_PARTIAL))
return -EIO;
+ /* do the same thing as extent tree rebuilding */
+ fs_info->csum_root->node =
+ btrfs_find_create_tree_block(fs_info->extent_root, 0,
+ leafsize);
+ if (!fs_info->csum_root->node)
+ return -ENOMEM;
+ clear_extent_buffer_uptodate(NULL, fs_info->csum_root->node);
}
fs_info->csum_root->track_dirty = 1;
--
1.9.0
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
