Hello Simon,

Am 18.02.2015 22:10, schrieb Simon Glass:
This strdup() is missing a check. Add it.

Signed-off-by: Simon Glass <s...@chromium.org>
---

  drivers/i2c/i2c-uclass.c | 2 ++
  1 file changed, 2 insertions(+)

Good catch, Thanks!

Acked-by: Heiko Schocher <h...@denx.de>

bye,
Heiko

diff --git a/drivers/i2c/i2c-uclass.c b/drivers/i2c/i2c-uclass.c
index a6991bf..b890806 100644
--- a/drivers/i2c/i2c-uclass.c
+++ b/drivers/i2c/i2c-uclass.c
@@ -230,6 +230,8 @@ static int i2c_bind_driver(struct udevice *bus, uint 
chip_addr, uint offset_len,

        snprintf(name, sizeof(name), "generic_%x", chip_addr);
        str = strdup(name);
+       if (!str)
+               return -ENOMEM;
        ret = device_bind_driver(bus, "i2c_generic_chip_drv", str, &dev);
        debug("%s:  device_bind_driver: ret=%d\n", __func__, ret);
        if (ret)


--
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to