openwrt/flashrom
1
0
Fork 0
mirror of https://review.coreboot.org/flashrom.git synced 2025-04-27 07:02:34 +02:00
Code Issues Releases Wiki Activity

dmi: Correctly check for ERROR_PTR

Browse Source 
For the physmap*() functions, NULL is considered valid return value.
Fixes a segmentation fault when DMI tables can't be mapped.

Tested on intel/eblake board with broken coreboot.

Change-Id: Ic403c2940c2b91acbd113f0acfa3ce9ef6c6bb6c
Signed-off-by: Nico Huber <nico.h@gmx.de>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/62616
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
This commit is contained in:
Nico Huber 2022-03-06 00:21:52 +01:00
parent f9b8fece4f
commit 2386e94850
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
dmi.c
View File

@ -169,7 +169,7 @@ static void dmi_table(uint32_t base, uint16_t len, uint16_t num)
int i = 0, j = 0; int i = 0, j = 0;
uint8_t *dmi_table_mem = physmap_ro("DMI Table", base, len); uint8_t *dmi_table_mem = physmap_ro("DMI Table", base, len);
if (dmi_table_mem == NULL) { if (dmi_table_mem == ERROR_PTR) {
msg_perr("Unable to access DMI Table\n"); msg_perr("Unable to access DMI Table\n");
return; return;
} }