Skip to content

Comments

Extract EFIBootServicesDriverEvents and populate them into FirmwareLogState#18

Merged
yawangwang merged 1 commit intogoogle:mainfrom
yawangwang:extract_EFIBootServicesDriver
Apr 9, 2025
Merged

Extract EFIBootServicesDriverEvents and populate them into FirmwareLogState#18
yawangwang merged 1 commit intogoogle:mainfrom
yawangwang:extract_EFIBootServicesDriver

Conversation

@yawangwang
Copy link
Collaborator

@yawangwang yawangwang commented Apr 5, 2025

The Host Attestation doc listed out all PCRs that need to be validated. One requirement is to validate each efi_boot_services_driver entry against a list of golden measurements.

This PR adds support to extract efi_boot_services_driver and efi_runtime_services_driver and populate them into FirmwareLogState.

@yawangwang yawangwang requested a review from alexmwu April 7, 2025 16:24
@yawangwang yawangwang changed the title Extract EFIBootSErvicesDriverEvents when parsing secureBoot state Extract EFIBootServicesDriverEvents when parsing secureBoot state Apr 7, 2025
@yawangwang yawangwang force-pushed the extract_EFIBootServicesDriver branch from 5c9ce46 to a31e3b5 Compare April 8, 2025 00:39
@yawangwang yawangwang requested a review from alexmwu April 8, 2025 00:45
@yawangwang yawangwang changed the title Extract EFIBootServicesDriverEvents when parsing secureBoot state Extract EFIBootServicesDriverEvents and populate them into FirmwareLogState Apr 8, 2025
Copy link
Collaborator

@alexmwu alexmwu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what about EV_EFI_RUNTIME_SERVICES_DRIVERs?

@yawangwang yawangwang force-pushed the extract_EFIBootServicesDriver branch 3 times, most recently from 25a0e0b to bc445cd Compare April 9, 2025 18:59
@yawangwang
Copy link
Collaborator Author

what about EV_EFI_RUNTIME_SERVICES_DRIVERs?

According to the section 3.3.4.3 PCR[2]-UEFI Drivers and UEFI Applications https://trustedcomputinggroup.org/wp-content/uploads/TCG-PC-Client-Platform-Firmware-Profile-Version-1.06-Revision-52_pub-3.pdf, this event MUST be measured into PCR2 as well, so adding the extraction logics to EfiDriverState(similar to EV_EFI_BOOT_SERVICES_DRIVER)

@yawangwang yawangwang requested a review from alexmwu April 9, 2025 19:03
@yawangwang yawangwang force-pushed the extract_EFIBootServicesDriver branch from f1696de to 880ebab Compare April 9, 2025 21:47
@yawangwang yawangwang requested a review from alexmwu April 9, 2025 21:51
@yawangwang yawangwang merged commit 8eb5f34 into google:main Apr 9, 2025
7 checks passed
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

Successfully merging this pull request may close these issues.

2 participants