Change in flashrom[master]: Fix segfault when iomem=strict

[Evgeny Zinoviev (Code Review)]

Evgeny Zinoviev has uploaded this change for review. ( https://review.coreboot.org/c/flashrom/+/48361 )

Change subject: Fix segfault when iomem=strict
......................................................................

Fix segfault when iomem=strict

As I can see, physmap_common() only returns ERROR_PTR or a real pointer
but never NULL.

Tested on Linux 5.9.12.

Change-Id: I5b11957fc10fd5c7a4fbb7f54fb94d791590f373
Signed-off-by: Evgeny Zinoviev 
---
M dmi.c
1 file changed, 1 insertion(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/flashrom refs/changes/61/48361/1

diff --git a/dmi.c b/dmi.c
index c44221c..7488d61 100644
--- a/dmi.c
+++ b/dmi.c
@@ -164,7 +164,7 @@
    unsigned int i = 0, j = 0;

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");
    	return;
    }

[Angel Pons (Code Review)]

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/48361 )

Change subject: Fix segfault when iomem=strict
......................................................................

Patch Set 1: Code-Review+1

[Nico Huber (Code Review)]

Attention is currently required from: Angel Pons, Evgeny Zinoviev.
Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/48361 )

Change subject: Fix segfault when iomem=strict
......................................................................

Patch Set 2:

(1 comment)

Patchset:

PS2: 
Soooory, looks like I ran this down ._.
Had the very same fix lying around for quite some time. It's
merged now.

[Angel Pons (Code Review)]

Angel Pons has abandoned this change. ( https://review.coreboot.org/c/flashrom/+/48361 )

Change subject: Fix segfault when iomem=strict
......................................................................

Abandoned

Superseded by CB:62611