Patch set updated for coreboot: commonlib: update fsp_relocate to make it compatible with UEFI 2.6

[Brandon Breitenstein]

Brandon Breitenstein ([e-mail]) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/16309

-gerrit

commit e9dcdc56cfa6cc6e5136e10d1e5a4018c9b56a8e
Author: Brandon Breitenstein 
Date:   Tue Aug 23 14:55:13 2016 -0700

commonlib: update fsp_relocate to make it compatible with UEFI 2.6

UEFI 2.6 spec casts the return of FFS_FILE2_SIZE to a UINT32
    which cannot be read using read_le32(&returnval). Add in a
    cast in order to safeguard for any non x86 architecture that may
    use this relocate. The proper change will be to get the UEFI
    header files changed to not cast this return value.

Change-Id: Ie1b50d99576ac42a0413204bbd599bab9f01828e
    Signed-off-by: Brandon Breitenstein 
---
 src/commonlib/fsp_relocate.c | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

diff --git a/src/commonlib/fsp_relocate.c b/src/commonlib/fsp_relocate.c
index 7f1e49a..57c0ac9 100644
--- a/src/commonlib/fsp_relocate.c
+++ b/src/commonlib/fsp_relocate.c
@@ -258,8 +258,15 @@ static size_t ffs_file_size(const EFI_FFS_FILE_HEADER *ffsfh)
 {
    size_t size;

-	if (IS_FFS_FILE2(ffsfh))
-		size = read_le32(&FFS_FILE2_SIZE(ffsfh));
+	if (IS_FFS_FILE2(ffsfh)) {
+		/*
+		 * this cast is needed with UEFI 2.6 headers in order
+		 * to read the UINT32 value that FFS_FILE2_SIZE converts
+		 * the return into
+		 */
+		uint32_t file2_size = FFS_FILE2_SIZE(ffsfh);
+		size = read_le32(&file2_size);
+	}
    else {
    	size = read_le8(&ffsfh->Size[0]) << 0;
    	size |= read_le8(&ffsfh->Size[1]) << 8;