diff options
| author | Qu Wenruo | 2018-10-05 11:45:55 +0200 |
|---|---|---|
| committer | David Sterba | 2018-12-17 14:51:27 +0100 |
| commit | 05a37c48604c19b50873fd9663f9140c150469d1 (patch) | |
| tree | 520b587fdf413bd3859b08278137f82c64e0f323 /fs/btrfs/volumes.c | |
| parent | btrfs: volumes: Make sure there is no overlap of dev extents at mount time (diff) | |
| download | kernel-qcow2-linux-05a37c48604c19b50873fd9663f9140c150469d1.tar.gz kernel-qcow2-linux-05a37c48604c19b50873fd9663f9140c150469d1.tar.xz kernel-qcow2-linux-05a37c48604c19b50873fd9663f9140c150469d1.zip | |
btrfs: volumes: Make sure no dev extent is beyond device boundary
Add extra dev extent end check against device boundary.
Signed-off-by: Qu Wenruo <wqu@suse.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/volumes.c')
| -rw-r--r-- | fs/btrfs/volumes.c | 17 |
1 files changed, 17 insertions, 0 deletions
diff --git a/fs/btrfs/volumes.c b/fs/btrfs/volumes.c index a567ee0bf060..303ae9de85b1 100644 --- a/fs/btrfs/volumes.c +++ b/fs/btrfs/volumes.c @@ -7387,6 +7387,7 @@ static int verify_one_dev_extent(struct btrfs_fs_info *fs_info, struct extent_map_tree *em_tree = &fs_info->mapping_tree.map_tree; struct extent_map *em; struct map_lookup *map; + struct btrfs_device *dev; u64 stripe_len; bool found = false; int ret = 0; @@ -7436,6 +7437,22 @@ static int verify_one_dev_extent(struct btrfs_fs_info *fs_info, physical_offset, devid); ret = -EUCLEAN; } + + /* Make sure no dev extent is beyond device bondary */ + dev = btrfs_find_device(fs_info, devid, NULL, NULL); + if (!dev) { + btrfs_err(fs_info, "failed to find devid %llu", devid); + ret = -EUCLEAN; + goto out; + } + if (physical_offset + physical_len > dev->disk_total_bytes) { + btrfs_err(fs_info, +"dev extent devid %llu physical offset %llu len %llu is beyond device boundary %llu", + devid, physical_offset, physical_len, + dev->disk_total_bytes); + ret = -EUCLEAN; + goto out; + } out: free_extent_map(em); return ret; |