proxmox-mirrors/efi-boot-shim
1
0
Fork 0
mirror of https://git.proxmox.com/git/efi-boot-shim synced 2025-08-15 10:21:21 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

shim/tpm: Take out GetCapability() call from tpm2_present() logic

Browse Source 
The EFI_TCG2_PROTOCOL.GetCapability() function is used to learn if a TPM2
chip is present. But the protocol capability information is also needed
for other reasons, for example to determine what event log formats are
supported by the firmware.

Take out the GetCapability() call from the tpm2_present() logic and reduce
that function to just checking if a TPM2 chip is available or not, so the
capabilities can later be used to determine the supported TPM log formats.

Signed-off-by: Javier Martinez Canillas <javierm@redhat.com>
This commit is contained in:
Javier Martinez Canillas 2017-06-15 15:16:05 +02:00 committed by Peter Jones
parent 9c40fb7c05
commit 0baa915056
1 changed files with 31 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

44
tpm.c
View File

@ -35,29 +35,41 @@ static BOOLEAN tpm_present(efi_tpm_protocol_t *tpm)
return TRUE; return TRUE;
} }
static BOOLEAN tpm2_present(efi_tpm2_protocol_t *tpm) static EFI_STATUS tpm2_get_caps(efi_tpm2_protocol_t *tpm,
EFI_TCG2_BOOT_SERVICE_CAPABILITY *caps,
BOOLEAN *old_caps)
{ {
EFI_STATUS status; EFI_STATUS status;
EFI_TCG2_BOOT_SERVICE_CAPABILITY caps;
TREE_BOOT_SERVICE_CAPABILITY *caps_1_0;
caps.Size = (UINT8)sizeof(caps); caps->Size = (UINT8)sizeof(*caps);
status = uefi_call_wrapper(tpm->get_capability, 2, tpm, &caps); status = uefi_call_wrapper(tpm->get_capability, 2, tpm, caps);
if (status != EFI_SUCCESS) if (status != EFI_SUCCESS)
return FALSE; return status;
if (caps.StructureVersion.Major == 1 && if (caps->StructureVersion.Major == 1 &&
caps.StructureVersion.Minor == 0) { caps->StructureVersion.Minor == 0)
caps_1_0 = (TREE_BOOT_SERVICE_CAPABILITY *)&caps; *old_caps = TRUE;
return EFI_SUCCESS;
}
static BOOLEAN tpm2_present(efi_tpm2_protocol_t *tpm,
EFI_TCG2_BOOT_SERVICE_CAPABILITY *caps,
BOOLEAN old_caps)
{
TREE_BOOT_SERVICE_CAPABILITY *caps_1_0;
if (old_caps) {
caps_1_0 = (TREE_BOOT_SERVICE_CAPABILITY *)caps;
if (caps_1_0->TrEEPresentFlag) if (caps_1_0->TrEEPresentFlag)
return TRUE; return TRUE;
} else {
if (caps.TPMPresentFlag)
return TRUE;
} }
if (caps->TPMPresentFlag)
return TRUE;
return FALSE; return FALSE;
} }
@ -90,9 +102,15 @@ EFI_STATUS tpm_log_event(EFI_PHYSICAL_ADDRESS buf, UINTN size, UINT8 pcr,
status = LibLocateProtocol(&tpm2_guid, (VOID **)&tpm2); status = LibLocateProtocol(&tpm2_guid, (VOID **)&tpm2);
/* TPM 2.0 */ /* TPM 2.0 */
if (status == EFI_SUCCESS) { if (status == EFI_SUCCESS) {
BOOLEAN old_caps;
EFI_TCG2_EVENT *event; EFI_TCG2_EVENT *event;
EFI_TCG2_BOOT_SERVICE_CAPABILITY caps;
if (!tpm2_present(tpm2)) status = tpm2_get_caps(tpm2, &caps, &old_caps);
if (status != EFI_SUCCESS)
return EFI_SUCCESS;
if (!tpm2_present(tpm2, &caps, old_caps))
return EFI_SUCCESS; return EFI_SUCCESS;
status = trigger_tcg2_final_events_table(tpm2); status = trigger_tcg2_final_events_table(tpm2);