summaryrefslogtreecommitdiff
path: root/src/soc/intel/skylake
diff options
context:
space:
mode:
authorShaunak Saha <shaunak.saha@intel.com>2017-10-06 12:50:38 -0700
committerMartin Roth <martinroth@google.com>2017-10-16 00:19:44 +0000
commitec1a24ca02221daeef521ccc8d934a7e0abb7ae5 (patch)
tree46fd86104c1abfb08ea9d9c654a7ad5da16ae6f0 /src/soc/intel/skylake
parentf329f0c3af07793facfbe4b4eb4892f919eaf908 (diff)
downloadcoreboot-ec1a24ca02221daeef521ccc8d934a7e0abb7ae5.tar.xz
src/soc/skylake: Fix Null pointer dereferences
Fix bug detected by coverity to handle the NULL pointer dereference Coverity Issues: * 1379849 * 1379848 TEST=Build and run on skylake platform Change-Id: Iec7a88a03531bbfeb72cedab5ad93d3a4c23eef5 Signed-off-by: Shaunak Saha <shaunak.saha@intel.com> Reviewed-on: https://review.coreboot.org/21909 Reviewed-by: Aaron Durbin <adurbin@chromium.org> Reviewed-by: Furquan Shaikh <furquan@google.com> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/soc/intel/skylake')
-rw-r--r--src/soc/intel/skylake/pmutil.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/src/soc/intel/skylake/pmutil.c b/src/soc/intel/skylake/pmutil.c
index 4cb4a20cca..19370e1b4d 100644
--- a/src/soc/intel/skylake/pmutil.c
+++ b/src/soc/intel/skylake/pmutil.c
@@ -216,6 +216,7 @@ void soc_get_gpi_gpe_configs(uint8_t *dw0, uint8_t *dw1, uint8_t *dw2)
DEVTREE_CONST struct device *dev = dev_find_slot(0, PCH_DEVFN_PMC);
if (!dev || !dev->chip_info) {
printk(BIOS_ERR, "BUG! Could not find SOC devicetree config\n");
+ return;
}
config = dev->chip_info;