From 14e826f3f8891e29fb8838fba3beb4ab62ee2d27 Mon Sep 17 00:00:00 2001 From: Jacob Garber Date: Tue, 12 Mar 2019 22:27:52 -0600 Subject: src/southbridge/intel/i82801gx/pcie.c: Correct NULL check Check if `pcie_dev` is NULL instead of `dev`. This was flagged as REVERSE_INULL during a Coverity scan, but is a simple typo. Signed-off-by: Jacob Garber Change-Id: Idc40574b9341d1b10cb2136cbc1a865efa3ab3ee Reviewed-on: https://review.coreboot.org/c/coreboot/+/31866 Reviewed-by: HAOUAS Elyes Tested-by: build bot (Jenkins) --- src/southbridge/intel/i82801gx/pcie.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/southbridge/intel/i82801gx/pcie.c b/src/southbridge/intel/i82801gx/pcie.c index 4679ee58f8..9446527c7b 100644 --- a/src/southbridge/intel/i82801gx/pcie.c +++ b/src/southbridge/intel/i82801gx/pcie.c @@ -195,7 +195,7 @@ static void root_port_commit_config(struct device *dev) pcie_dev = rpc.ports[i]; - if (dev == NULL) { + if (pcie_dev == NULL) { printk(BIOS_ERR, "Root Port %d device is NULL?\n", i + 1); continue; -- cgit v1.2.3