From 347234c1ef0cb185df5da7c8fd5c2896cd269c2b Mon Sep 17 00:00:00 2001 From: Timothy Pearson Date: Tue, 1 Dec 2015 13:49:59 -0600 Subject: util/nvramtool: Use correct virtual address when mapping tables The existing code used a stale pointer from a previously unmapped region of memory when parsing the coreboot tables. Use the correct pointer from the currently mapped memory region when parsing. Change-Id: Id9a1c70655fe25bc079e5bee55f15adf674694f8 Signed-off-by: Timothy Pearson Reviewed-on: https://review.coreboot.org/12619 Reviewed-by: Aaron Durbin Tested-by: build bot (Jenkins) Reviewed-by: Martin Roth Reviewed-by: Stefan Reinauer --- util/nvramtool/lbtable.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/util/nvramtool/lbtable.c b/util/nvramtool/lbtable.c index 1c7bc0ab3d..5db61c507d 100644 --- a/util/nvramtool/lbtable.c +++ b/util/nvramtool/lbtable.c @@ -489,6 +489,9 @@ static const struct lb_header *lbtable_scan(unsigned long start, } map_pages(p, table->table_bytes + sizeof(*table)); + + table = (const struct lb_header *)phystov(p); + /* validate table checksum */ if (table->table_checksum != compute_ip_checksum(((char *)table) + sizeof(*table), -- cgit v1.2.3