| commit | 14e826f3f8891e29fb8838fba3beb4ab62ee2d27 | [log] [tgz] |
|---|---|---|
| author | Jacob Garber <jgarber1@ualberta.ca> | Tue Mar 12 22:27:52 2019 -0600 |
| committer | Patrick Georgi <pgeorgi@google.com> | Tue Mar 19 21:43:57 2019 +0000 |
| tree | 1ab9a6e0bb97669eef85ca6d649f536b9e72b34d | |
| parent | 8d0f59935d340287b15e21f19e2f2c28fcb45cce [diff] [blame] |
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 Reviewed-on: https://review.coreboot.org/c/coreboot/+/31866 Reviewed-by: HAOUAS Elyes <ehaouas@noos.fr> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
diff --git a/src/southbridge/intel/i82801gx/pcie.c b/src/southbridge/intel/i82801gx/pcie.c index 4679ee5..9446527 100644 --- a/src/southbridge/intel/i82801gx/pcie.c +++ b/src/southbridge/intel/i82801gx/pcie.c
@@ -195,7 +195,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;