Huan Truong (htruong@tnhh.net) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/18573
-gerrit
commit 210b75c8281ccfe9620f390acf2a7c28345e2355 Author: Huan Truong htruong@tnhh.net Date: Sun Mar 5 02:24:51 2017 -0600
util/intelmetool: Fix segfault on Dell Chromebook 13/Wildcat Point LP
The intel ME checker tool would segfault if it couldn't find a single eligible device when it finish scanning for pci devices. In that case the dev pointer will be NULL, this patch makes sure that case doesn't happen.
Change-Id: Ia13191799d7e00185947f9df5188cb2666c43e2a Signed-off-by: Huan Truong htruong@tnhh.net --- util/intelmetool/intelmetool.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/util/intelmetool/intelmetool.c b/util/intelmetool/intelmetool.c index 70b23fb..45e8c8f 100644 --- a/util/intelmetool/intelmetool.c +++ b/util/intelmetool/intelmetool.c @@ -137,7 +137,8 @@ static int pci_platform_scan() { } }
- if (!PCI_DEV_HAS_ME_DISABLE(dev->device_id) && + if (dev != NULL && + !PCI_DEV_HAS_ME_DISABLE(dev->device_id) && !PCI_DEV_HAS_ME_DIFFICULT(dev->device_id) && !PCI_DEV_CAN_DISABLE_ME_IF_PRESENT(dev->device_id) && !PCI_DEV_ME_NOT_SURE(dev->device_id)) {