[coreboot-gerrit] [S] Change in coreboot[main]: drivers/efi/capsules.c: fix recording capsule size

[Sergii Dmytruk (Code Review)]

Sergii Dmytruk has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/84542?usp=email )

Change subject: drivers/efi/capsules.c: fix recording capsule size
......................................................................

drivers/efi/capsules.c: fix recording capsule size

Change-Id: I97a528e2611fcd711c555d0f01e9aadcd2031217
Signed-off-by: Sergii Dmytruk 
---
M src/drivers/efi/capsules.c
1 file changed, 8 insertions(+), 4 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/42/84542/1

diff --git a/src/drivers/efi/capsules.c b/src/drivers/efi/capsules.c
index cf15c3c..3d515a0 100644
--- a/src/drivers/efi/capsules.c
+++ b/src/drivers/efi/capsules.c
@@ -601,6 +601,7 @@
 {
    struct block_descr block = block_chain;
    uint8_t *capsule_start = NULL;
+	uint32_t capsule_size = 0;
    uint32_t size_left = 0;

/* No safety checks in this function, as all of them were done earlier. */
@@ -615,8 +616,10 @@
    	if (size_left == 0) {
    		const EFI_CAPSULE_HEADER *capsule_hdr =
    			map_range(block.addr, sizeof(*capsule_hdr));
-			size_left = capsule_hdr->CapsuleImageSize;
+			capsule_size = capsule_hdr->CapsuleImageSize;
    		capsule_start = target;
+
+			size_left = capsule_size;
    	}

uint64_t addr = block.addr;
@@ -646,13 +649,14 @@
    		}

uefi_capsules[uefi_capsule_count].base = (uintptr_t)capsule_start;
-			uefi_capsules[uefi_capsule_count].len = block.len;
+			uefi_capsules[uefi_capsule_count].len = capsule_size;
    		uefi_capsule_count++;

/* This is to align start of the next capsule (assumes that
    		   initial value of target was suitably aligned). */
-			if (!IS_ALIGNED(block.len, CAPSULE_ALIGNMENT))
-				target += ALIGN_UP(block.len, CAPSULE_ALIGNMENT) - block.len;
+			if (!IS_ALIGNED(capsule_size, CAPSULE_ALIGNMENT))
+				target += ALIGN_UP(capsule_size, CAPSULE_ALIGNMENT) -
+					  capsule_size;
    	}
    }