diff options
| author | Jacob Garber <jgarber1@ualberta.ca> | 2019-03-12 22:27:52 -0600 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2019-03-19 21:43:57 +0000 |
| commit | 14e826f3f8891e29fb8838fba3beb4ab62ee2d27 (patch) | |
| tree | 1ab9a6e0bb97669eef85ca6d649f536b9e72b34d /src/southbridge/intel | |
| parent | 8d0f59935d340287b15e21f19e2f2c28fcb45cce (diff) | |
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>
Diffstat (limited to 'src/southbridge/intel')
| -rw-r--r-- | src/southbridge/intel/i82801gx/pcie.c | 2 |
1 files changed, 1 insertions, 1 deletions
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; |