Jacob Garber has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/31866
Change subject: src/southbridge/intel/i82801gx/pcie.c: Correct NULL check ......................................................................
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 jgarber1@ualberta.ca Change-Id: Idc40574b9341d1b10cb2136cbc1a865efa3ab3ee --- M src/southbridge/intel/i82801gx/pcie.c 1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/66/31866/1
diff --git a/src/southbridge/intel/i82801gx/pcie.c b/src/southbridge/intel/i82801gx/pcie.c index 81679a9..f8bce6a 100644 --- a/src/southbridge/intel/i82801gx/pcie.c +++ b/src/southbridge/intel/i82801gx/pcie.c @@ -191,7 +191,7 @@
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; }