4 comments:
File src/security/vboot/secdata_tpm.c:
Patch Set #3, Line 96: int attempts = 3;
We got rid of retries in depthcharge because we decided that our communication with Cr50 is reliable. Is that not the case here? (Or are we not making the assumption of using Cr50 here?)
/*
* This seems the first time we've run. Initialize the TPM.
*/
Or perhaps also making this shorter than three lines?
Patch Set #3, Line 425: //RETURN_ON_FAILURE(factory_initialize_tpm(ctx));
If we're fixing any other random stuff in this CL, should we think about removing this line?
File src/security/vboot/vboot_logic.c:
Patch Set #3, Line 268: VB2_CONTEXT_SECDATA_FIRMWARE_CHANGED
Good catch, thanks.
To view, visit change 37471. To unsubscribe, or for help writing mail filters, visit settings.