Skip to content
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

Add WithPlatformConfigProfile() (for PCR1) #322

Open
chrisccoulson opened this issue Aug 9, 2024 · 0 comments
Open

Add WithPlatformConfigProfile() (for PCR1) #322

chrisccoulson opened this issue Aug 9, 2024 · 0 comments

Comments

@chrisccoulson
Copy link
Collaborator

chrisccoulson commented Aug 9, 2024

If PCR7 isn't supported for some reason, eg, because secure boot is disabled or the device is not in deployed mode, then we should be able to fall back to a safe profile that includes PCRs 1, 2, 3, 4 and 5. PCR 2 and 4 are for binding to all code that's loaded outside of the platform firmware.

PCR1 is to bind to all of the platform firmware's configuration - currently, changes to security-sensitive settings such as enabling a debugger or disabling DMA remapping change the value of PCR7 by adding extra EVI_EFI_ACTION events.

PCR3 contains any configuration related to UEFI drivers that are not part of the platform firmware.

PCR5 contains the GPT of the IBL and can be used for any bootloader configuration, and support for this will be tracked in a separate issue.

To support this, the efi package needs a new WithPlatformConfigProfile API that will support PCR1.

Most events for PCR1 will be copied from the log to the profile, with the exception of:

  • BootOrder and BootXXXX global variable EV_EFI_VARIABLE_BOOT and EV_EFI_VARIABLE_BOOT2 events, as these can be computed in order to support updating these variables.
  • EV_ACTION events with the data "Entering ROM Based Setup" - profile generation should fail in this case and the pre-install checks should catch this and advise a reboot if PCR1 is required.
  • EV_ACTION events with the data "Chassis Intrusion" - profile generation should fail in this case and the pre-install checks should catch this and advise this condition is cleared before continuing if PCR1 is required.
@chrisccoulson chrisccoulson changed the title Add WithPlatformFirmwareConfigProfile() (for PCR1) Add WithPlatformConfigProfile() (for PCR1) Oct 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant