[coreboot-gerrit] Change in coreboot[master]: cbfs: mcache: Fix end-of-cache check

[Julius Werner (Code Review)]

Julius Werner has submitted this change. ( https://review.coreboot.org/c/coreboot/+/48277 )

Change subject: cbfs: mcache: Fix end-of-cache check
......................................................................

cbfs: mcache: Fix end-of-cache check

After the mcache is copied into CBMEM, it has *just* the right size to
fit the final tag with no room to spare. That means the test to check if
we walked over the end must be `current + sizeof(tag) <= end`, not
`current + sizeof(tag) < end`.

Signed-off-by: Julius Werner 
Change-Id: I25a0d774fb3294bb4d15f31f432940bfccc84af0
Reviewed-on: https://review.coreboot.org/c/coreboot/+/48277
Tested-by: build bot (Jenkins) 
Reviewed-by: Arthur Heymans 
---
M src/commonlib/bsd/cbfs_mcache.c
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  build bot (Jenkins): Verified
  Arthur Heymans: Looks good to me, approved

diff --git a/src/commonlib/bsd/cbfs_mcache.c b/src/commonlib/bsd/cbfs_mcache.c
index d2f969d..0d5d8e0 100644
--- a/src/commonlib/bsd/cbfs_mcache.c
+++ b/src/commonlib/bsd/cbfs_mcache.c
@@ -95,7 +95,7 @@
    const void *end = mcache + mcache_size;
    const void *current = mcache;

-	while (current + sizeof(uint32_t) < end) {
+	while (current + sizeof(uint32_t) <= end) {
    	const union mcache_entry *entry = current;

if (entry->magic == MCACHE_MAGIC_END)