Martin L Roth has submitted this change. ( https://review.coreboot.org/c/coreboot/+/78845?usp=email )
Change subject: Revert "vboot: Add catchall recovery reason for unspecified phase 4 errors" ......................................................................
Revert "vboot: Add catchall recovery reason for unspecified phase 4 errors"
This reverts commit 7499d9610016c4a89b722498885f196ed6d217aa.
Reason for revert: coreboot build fails
Change-Id: I8ef853d81ee9b1f18d36dfd82cdf687381ece2c6 Signed-off-by: Martin Roth gaumless@gmail.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/78845 Reviewed-by: Julius Werner jwerner@chromium.org Reviewed-by: Felix Singer service+coreboot-gerrit@felixsinger.de Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Matt DeVillier matt.devillier@gmail.com --- M src/security/vboot/vboot_logic.c 1 file changed, 1 insertion(+), 1 deletion(-)
Approvals: Martin L Roth: Verified build bot (Jenkins): Verified Matt DeVillier: Looks good to me, approved Julius Werner: Looks good to me, approved Felix Singer: Looks good to me, approved
diff --git a/src/security/vboot/vboot_logic.c b/src/security/vboot/vboot_logic.c index 93a188c..11983b9 100644 --- a/src/security/vboot/vboot_logic.c +++ b/src/security/vboot/vboot_logic.c @@ -374,7 +374,7 @@ }
if (rv) - vboot_fail_and_reboot(ctx, VB2_RECOVERY_FW_GET_FW_BODY, rv); + vboot_save_and_reboot(ctx, rv); vboot_save_data(ctx);
/* Only extend PCRs once on boot. */