[coreboot-gerrit] New patch to review for coreboot: vboot: Init vb2 work memory area before calling verstage

Patrick Georgi (pgeorgi@google.com) gerrit at coreboot.org
Thu Jun 25 16:59:42 CEST 2015


Patrick Georgi (pgeorgi at google.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/10660

-gerrit

commit fa0606a3d1faa1b33b4d7b925bac221f16865274
Author: Patrick Georgi <pgeorgi at chromium.org>
Date:   Thu Jun 25 16:57:57 2015 +0200

    vboot: Init vb2 work memory area before calling verstage
    
    Otherwise it'll determine some offsets from uninitialized data and hilarity
    ensues.
    
    Change-Id: I6a671987857cfd3f3cd6078aebd13dd09fc79020
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
---
 src/vendorcode/google/chromeos/vboot2/vboot_loader.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/vendorcode/google/chromeos/vboot2/vboot_loader.c b/src/vendorcode/google/chromeos/vboot2/vboot_loader.c
index cbb5f89..d6e760a 100644
--- a/src/vendorcode/google/chromeos/vboot2/vboot_loader.c
+++ b/src/vendorcode/google/chromeos/vboot2/vboot_loader.c
@@ -90,6 +90,7 @@ static int vboot_active(struct asset *asset)
 		init_vb2_working_data();
 		verstage_main();
 	} else if (verstage_should_load()) {
+		init_vb2_working_data();
 		struct prog verstage =
 			PROG_INIT(ASSET_VERSTAGE,
 				CONFIG_CBFS_PREFIX "/verstage");



More information about the coreboot-gerrit mailing list