From 85d67ac1450ff245af79ac907092fa55eb52fb32 Mon Sep 17 00:00:00 2001 From: Zach Brown Date: Wed, 15 Oct 2014 16:14:19 -0700 Subject: btrfs-progs: check read extent errors when mapping coverity barked out a warning that btrfs-map-logical was storing but ignoring errors from read_extent_from_disk(). So don't ignore 'em. I made extent reading errors fatal to match the fatal errors from mapping mirrors above. And while we're at it have read_extent_from_disk() return -errno pread errors instead of -EIO or -1 (-EPERM). The only other caller who tests errors clobbers them with -EIO. Signed-off-by: Zach Brown Signed-off-by: David Sterba --- btrfs-map-logical.c | 12 +++++++++++- 1 file changed, 11 insertions(+), 1 deletion(-) (limited to 'btrfs-map-logical.c') diff --git a/btrfs-map-logical.c b/btrfs-map-logical.c index 6b475fc6..47d1104d 100644 --- a/btrfs-map-logical.c +++ b/btrfs-map-logical.c @@ -76,8 +76,18 @@ static struct extent_buffer * debug_read_block(struct btrfs_root *root, (unsigned long long)eb->dev_bytenr, device->name); kfree(multi); - if (!copy || mirror_num == copy) + if (!copy || mirror_num == copy) { ret = read_extent_from_disk(eb, 0, eb->len); + if (ret) { + fprintf(info_file, + "Error: failed to read extent: mirror %d logical %llu: %s\n", + mirror_num, (unsigned long long)eb->start, + strerror(-ret)); + free_extent_buffer(eb); + eb = NULL; + break; + } + } num_copies = btrfs_num_copies(&root->fs_info->mapping_tree, eb->start, eb->len); -- cgit v1.2.3