summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorRavi Sarawadi <ravishankar.sarawadi@intel.com>2017-11-30 11:25:40 -0800
committerMartin Roth <martinroth@google.com>2018-01-17 17:23:25 +0000
commit6522bf1a81735d8f0fa6f4b8f5285e0b7e60be01 (patch)
tree78cacd3f0aefb723b489ecb2deec2c791ec786c2 /src
parent7e687b84b212bef4444e7b2703f794bab0d86bb0 (diff)
downloadcoreboot-6522bf1a81735d8f0fa6f4b8f5285e0b7e60be01.tar.xz
soc/intel/apollolake/meminit_util_glk.c: Check for NULL
We check for NULL here for memory_info_hob and return if it's NULL so that the future dereferencing is proper. Change-Id: Ie34931504ad92739fdaa68ec7989e76e8eee2595 Found-by: Klockworks Signed-off-by: Ravi Sarawadi <ravishankar.sarawadi@intel.com> Reviewed-on: https://review.coreboot.org/23223 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Aaron Durbin <adurbin@chromium.org> Reviewed-by: Furquan Shaikh <furquan@google.com>
Diffstat (limited to 'src')
-rw-r--r--src/soc/intel/apollolake/meminit_util_glk.c4
1 files changed, 4 insertions, 0 deletions
diff --git a/src/soc/intel/apollolake/meminit_util_glk.c b/src/soc/intel/apollolake/meminit_util_glk.c
index 2ef2e49bfe..20e7cd29ed 100644
--- a/src/soc/intel/apollolake/meminit_util_glk.c
+++ b/src/soc/intel/apollolake/meminit_util_glk.c
@@ -39,6 +39,10 @@ void save_lpddr4_dimm_info(const struct lpddr4_cfg *lp4cfg, size_t mem_sku)
}
memory_info_hob = fsp_find_smbios_memory_info(&hob_size);
+ if (memory_info_hob == NULL) {
+ printk(BIOS_ERR, "SMBIOS memory info HOB is missing\n");
+ return;
+ }
/*
* Allocate CBMEM area for DIMM information used to populate SMBIOS