Check whether access is out of bounds of the partition and
return an error. This way there is no danger of esp. write
or erase outside of the confines of partition.

Signed-off-by: Marek Vasut <marek.vasut+rene...@mailbox.org>
---
Cc: AKASHI Takahiro <takahiro.aka...@linaro.org>
Cc: Abdellatif El Khlifi <abdellatif.elkhl...@arm.com>
Cc: Bin Meng <bmeng...@gmail.com>
Cc: Heinrich Schuchardt <xypron.gl...@gmx.de>
Cc: Joshua Watt <jpewhac...@gmail.com>
Cc: Michal Suchanek <msucha...@suse.de>
Cc: Simon Glass <s...@chromium.org>
Cc: Tobias Waldekranz <tob...@waldekranz.com>
---
 disk/disk-uclass.c | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/disk/disk-uclass.c b/disk/disk-uclass.c
index 32722cf9176..f262105375b 100644
--- a/disk/disk-uclass.c
+++ b/disk/disk-uclass.c
@@ -27,9 +27,17 @@
  */
 static int disk_blk_part_validate(struct udevice *dev, lbaint_t start, 
lbaint_t blkcnt)
 {
+       struct disk_part *part = dev_get_uclass_plat(dev);
+
        if (device_get_uclass_id(dev) != UCLASS_PARTITION)
                return -ENOSYS;
 
+       if (start >= part->gpt_part_info.size)
+               return -E2BIG;
+
+       if ((start + blkcnt) > part->gpt_part_info.size)
+               return -ERANGE;
+
        return 0;
 }
 
-- 
2.40.1

Reply via email to