diff options
| author | Ravi Sarawadi <ravishankar.sarawadi@intel.com> | 2017-11-30 11:25:40 -0800 |
|---|---|---|
| committer | Martin Roth <martinroth@google.com> | 2018-01-17 17:23:25 +0000 |
| commit | 6522bf1a81735d8f0fa6f4b8f5285e0b7e60be01 (patch) | |
| tree | 78cacd3f0aefb723b489ecb2deec2c791ec786c2 /src/soc/intel | |
| parent | 7e687b84b212bef4444e7b2703f794bab0d86bb0 (diff) | |
| download | coreboot-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/soc/intel')
| -rw-r--r-- | src/soc/intel/apollolake/meminit_util_glk.c | 4 |
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 |