John Zhao has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/43553 )
Change subject: superio/common: Avoid NULL pointer dereference ......................................................................
superio/common: Avoid NULL pointer dereference
Coverity detects dereferencing a pointer that might be "NULL" when calling report_resource_stored. Add sanity check for dev to prevent NULL pointer dereference.
Found-by: Coverity CID 1419488
Signed-off-by: John Zhao john.zhao@intel.com Change-Id: I03efad87ba761e914b47e3294c646335cfbaed24 --- M src/superio/common/generic.c 1 file changed, 3 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/53/43553/1
diff --git a/src/superio/common/generic.c b/src/superio/common/generic.c index 096b96d..83f2fea 100644 --- a/src/superio/common/generic.c +++ b/src/superio/common/generic.c @@ -9,6 +9,9 @@ { struct resource *res;
+ if (!dev) + return; + if (dev->link_list) assign_resources(dev->link_list);