-
Notifications
You must be signed in to change notification settings - Fork 98
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
x86 Stage1: add verification of components #330
Conversation
bd5f098
to
0a56a70
Compare
6da5215
to
02dfec6
Compare
325248b
to
7712a38
Compare
@rizlik please test on TGL before merging |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
great work!
Tested successfully on TGL
@@ -959,6 +959,10 @@ int wolfBoot_verify_integrity(struct wolfBoot_image *img) | |||
{ | |||
uint8_t *stored_sha; | |||
uint16_t stored_sha_len; | |||
#ifdef STAGE1_AUTH | |||
/* Override global */ | |||
uint8_t digest[WOLFBOOT_SHA_DIGEST_SIZE]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Isn't there already a global with this name? The comment makes it clear you want to use the local here. Can you think of a better way to do this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving and merging. The digest global override is fine for now...
Verify FSP images + configuration at startup.