From: Genjian Zhang <[email protected]>

<offset> is a backing-device sector offset; ti->len is the virtual
target length. Comparing them rejects valid tables, e.g.:

  dmsetup create ebs0 --table "0 1048576 ebs /dev/sda 2097152 1 8"
  -> ebs: Invalid device offset sector (-EINVAL)

Drop the check. Bounds against the backing device are already
enforced later by device_area_is_invalid() via ebs_iterate_devices().

Fixes: d3c7b35c20d6 ("dm: add emulated block size target")
Signed-off-by: Genjian Zhang <[email protected]>
---
 drivers/md/dm-ebs-target.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/md/dm-ebs-target.c b/drivers/md/dm-ebs-target.c
index 1e52bde48b91..5d67c6c19d4b 100644
--- a/drivers/md/dm-ebs-target.c
+++ b/drivers/md/dm-ebs-target.c
@@ -265,8 +265,7 @@ static int ebs_ctr(struct dm_target *ti, unsigned int argc, 
char **argv)
 
        r = -EINVAL;
        if (sscanf(argv[1], "%llu%c", &tmp, &dummy) != 1 ||
-           tmp != (sector_t)tmp ||
-           (sector_t)tmp >= ti->len) {
+           tmp != (sector_t)tmp) {
                ti->error = "Invalid device offset sector";
                goto bad;
        }
-- 
2.43.0


Reply via email to