Nico Huber has uploaded this change for review. ( https://review.coreboot.org/21799
Change subject: ich_descriptors_tool: Fix an off-by-one ......................................................................
ich_descriptors_tool: Fix an off-by-one
Original-Change-Id: I008abd78c7c42bf3f17e68c192cd79dd427c5cb5 Original-Reviewed-on: https://review.coreboot.org/19045 Original-Tested-by: build bot (Jenkins) Original-Reviewed-by: Stefan Reinauer stefan.reinauer@coreboot.org
Change-Id: If494234e9529d83564c21799bf8ea424ae633667 Signed-off-by: Nico Huber nico.huber@secunet.com --- M util/ich_descriptors_tool/ich_descriptors_tool.c 1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/flashrom refs/changes/99/21799/1
diff --git a/util/ich_descriptors_tool/ich_descriptors_tool.c b/util/ich_descriptors_tool/ich_descriptors_tool.c index 0da27d2..250abf0 100644 --- a/util/ich_descriptors_tool/ich_descriptors_tool.c +++ b/util/ich_descriptors_tool/ich_descriptors_tool.c @@ -230,7 +230,7 @@ prettyprint_ich_descriptors(cs, &desc);
pMAC = (uint8_t *) &buf[ICH_FREG_BASE(desc.region.reg3_base) >> 2]; - if (len >= ICH_FREG_BASE(desc.region.reg3_base) + 5 && pMAC[0] != 0xff) + if (len >= ICH_FREG_BASE(desc.region.reg3_base) + 6 && pMAC[0] != 0xff) printf("The MAC address might be at offset 0x%x: " "%02x:%02x:%02x:%02x:%02x:%02x\n", ICH_FREG_BASE(desc.region.reg3_base),