[U-Boot] [PATCH] fdt: Check for NULL return from fdt_getprop in 'fdt set'
Tom Rini
trini at konsulko.com
Tue Jun 13 11:10:35 UTC 2017
While the previous pass through fixed one place where we knew that
fdt_getprop would be given a positive len, in the case of 'fdt set' we
do not, so check that we did no get NULL from fdt_getprop().
Cc: Simon Glass <sjg at chromium.org>
Reported-by: Coverity (CID: 163249)
Fixes 72c98ed1ab48 ("fdt: Add a check to do_fdt() for coverity")
Signed-off-by: Tom Rini <trini at konsulko.com>
---
cmd/fdt.c | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/cmd/fdt.c b/cmd/fdt.c
index 31a536198c8e..05e19f8a72bf 100644
--- a/cmd/fdt.c
+++ b/cmd/fdt.c
@@ -284,6 +284,10 @@ static int do_fdt(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
len = 0;
} else {
ptmp = fdt_getprop(working_fdt, nodeoffset, prop, &len);
+ if (!ptmp) {
+ printf("prop (%s) not found!\n", prop);
+ return 1;
+ }
if (len > SCRATCHPAD) {
printf("prop (%d) doesn't fit in scratchpad!\n",
len);
--
1.9.1
More information about the U-Boot
mailing list