]> git.dujemihanovic.xyz Git - u-boot.git/commitdiff
stm32mp: bsec: add check on null size in misc ops
authorPatrick Delaunay <patrick.delaunay@foss.st.com>
Thu, 27 Apr 2023 13:36:34 +0000 (15:36 +0200)
committerPatrice Chotard <patrice.chotard@foss.st.com>
Fri, 16 Jun 2023 09:16:31 +0000 (11:16 +0200)
Add a protection in misc bsec ops for request with null size.

For example OP-TEE error occurs when get_eth_nb() return 0 in
setup_mac_address() for unknown part number because U-Boot read 0 OTPs.

Signed-off-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
Reviewed-by: Patrice Chotard <patrice.chotard@foss.st.com>
arch/arm/mach-stm32mp/bsec.c

index f5f4b20d4776fd57659a34814f172051102d7211..0dc1e5c3fdc4de72a42b3b07186a70ee5e12b22c 100644 (file)
@@ -622,7 +622,7 @@ static int stm32mp_bsec_read(struct udevice *dev, int offset,
                shadow = false;
        }
 
-       if ((offs % 4) || (size % 4))
+       if ((offs % 4) || (size % 4) || !size)
                return -EINVAL;
 
        if (IS_ENABLED(CONFIG_OPTEE) && priv->tee) {
@@ -678,7 +678,7 @@ static int stm32mp_bsec_write(struct udevice *dev, int offset,
                shadow = false;
        }
 
-       if ((offs % 4) || (size % 4))
+       if ((offs % 4) || (size % 4) || !size)
                return -EINVAL;
 
        if (IS_ENABLED(CONFIG_OPTEE) && priv->tee) {