diff options
| author | Shaunak Saha <shaunak.saha@intel.com> | 2017-10-06 12:50:38 -0700 |
|---|---|---|
| committer | Martin Roth <martinroth@google.com> | 2017-10-16 00:19:44 +0000 |
| commit | ec1a24ca02221daeef521ccc8d934a7e0abb7ae5 (patch) | |
| tree | 46fd86104c1abfb08ea9d9c654a7ad5da16ae6f0 /src/soc/intel/skylake | |
| parent | f329f0c3af07793facfbe4b4eb4892f919eaf908 (diff) | |
| download | coreboot-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.c | 1 |
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; |