diff options
| author | Robert Baldyga <r.baldyga@samsung.com> | 2014-09-18 17:13:07 +0200 |
|---|---|---|
| committer | Simon Glass <sjg@chromium.org> | 2014-09-23 12:44:31 -0600 |
| commit | cae025aab3e8ea8ad455cce8b0e4647401cdd091 (patch) | |
| tree | 4c6da6074b1a63ba1042e17147b494110664ccd8 /drivers | |
| parent | 59990bf0eaa12b123759cb0485f38e156103e93c (diff) | |
| download | talos-obmc-uboot-cae025aab3e8ea8ad455cce8b0e4647401cdd091.tar.gz talos-obmc-uboot-cae025aab3e8ea8ad455cce8b0e4647401cdd091.zip | |
dm: avoid dev->req_seq overflow
Since dev->req_seq value is initialized from "reg" property of fdt node,
there is posibility, that address value contained in fdt is greater than
INT_MAX, and then value in dev->req_seq is negative which led to probe()
fail.
This patch fix this problem by ensuring that req_seq is positive, unless
it's one of errno codes.
Signed-off-by: Robert Baldyga <r.baldyga@samsung.com>
Acked-by: Simon Glass <sjg@chromium.org>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/core/device.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/core/device.c b/drivers/core/device.c index ef41a9be3e..32e80e82b5 100644 --- a/drivers/core/device.c +++ b/drivers/core/device.c @@ -109,6 +109,8 @@ int device_bind(struct udevice *parent, struct driver *drv, const char *name, dev->seq = -1; #ifdef CONFIG_OF_CONTROL dev->req_seq = fdtdec_get_int(gd->fdt_blob, of_offset, "reg", -1); + if (!IS_ERR_VALUE(dev->req_seq)) + dev->req_seq &= INT_MAX; if (uc->uc_drv->name && of_offset != -1) { fdtdec_get_alias_seq(gd->fdt_blob, uc->uc_drv->name, of_offset, &dev->req_seq); |
