Nico Huber has uploaded this change for review.
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),
To view, visit change 21799. To unsubscribe, visit settings.