public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Zhang, Chao B" <chao.b.zhang@intel.com>
To: "Long, Qin" <qin.long@intel.com>,
	"edk2-devel@lists.01.org" <edk2-devel@lists.01.org>
Cc: "Yao, Jiewen" <jiewen.yao@intel.com>
Subject: Re: [PATCH 1/2] SecurityPkg: Tcg2Dxe: Log EV_NO_ACTION event in TCG_PCR_EVENT2 format
Date: Mon, 6 Feb 2017 00:57:34 +0000	[thread overview]
Message-ID: <FF72C7E4248F3C4E9BDF19D4918E90F2494FDD18@shsmsx102.ccr.corp.intel.com> (raw)
In-Reply-To: <BF2CCE9263284D428840004653A28B6E53F5C8D0@SHSMSX103.ccr.corp.intel.com>

Qin: 
  Thanks for your comments. I will follow TCG EFI Protocol spec to use Digests. As for TCG_PCR_EVENT2 structure,
I will keep Digest field unchanged, since it is an incompatible change.

-----Original Message-----
From: Long, Qin 
Sent: Saturday, February 4, 2017 4:53 PM
To: Zhang, Chao B <chao.b.zhang@intel.com>; edk2-devel@lists.01.org
Cc: Yao, Jiewen <jiewen.yao@intel.com>
Subject: RE: [PATCH 1/2] SecurityPkg: Tcg2Dxe: Log EV_NO_ACTION event in TCG_PCR_EVENT2 format

Please check the "Digests" and "Digest" in HDR definition, as my previous comment.

Reviewed-by: Long Qin <qin.long@intel.com>


Best Regards & Thanks,
LONG, Qin

> -----Original Message-----
> From: Zhang, Chao B
> Sent: Saturday, February 04, 2017 9:33 AM
> To: edk2-devel@lists.01.org
> Cc: Long, Qin; Yao, Jiewen; Zhang, Chao B
> Subject: [PATCH 1/2] SecurityPkg: Tcg2Dxe: Log EV_NO_ACTION event in
> TCG_PCR_EVENT2 format
> 
> All the event log entry in TCG2 Event log format should be 
> TCG_PCR_EVENT2 except event log header. Defined by TCG EFI Spec 2.0 
> 00.13
> https://trustedcomputinggroup.org/wp-content/uploads/EFI-Protocol-
> Specification-rev13-160330final.pdf
> 
> Cc: Long Qin <qin.long@intel.com>
> Cc: Yao Jiewen <jiewen.yao@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Chao Zhang <chao.b.zhang@intel.com>
> ---
>  SecurityPkg/Tcg/Tcg2Dxe/Tcg2Dxe.c | 114
> ++++++++++++++++++++++++++++++++------
>  1 file changed, 96 insertions(+), 18 deletions(-)
> 
> diff --git a/SecurityPkg/Tcg/Tcg2Dxe/Tcg2Dxe.c
> b/SecurityPkg/Tcg/Tcg2Dxe/Tcg2Dxe.c
> index 860ee59..08c40db 100644
> --- a/SecurityPkg/Tcg/Tcg2Dxe/Tcg2Dxe.c
> +++ b/SecurityPkg/Tcg/Tcg2Dxe/Tcg2Dxe.c
> @@ -169,6 +169,82 @@ InternalDumpData (
> 
>  /**
> 
> +  This function initialize TCG_PCR_EVENT2 for EV_NO_ACTION Event Type 
> + other than EFI Specification ID event  The behavior is defined by 
> + TCG PC Client PFP Spec. Section 9.3.4 EV_NO_ACTION Event Types
> +
> +  @param[in, out]   NoActionEvent  Event Header of EV_NO_ACTION Event
> +  @param[in]        EventSize      Event Size of the EV_NO_ACTION Event
> +
> +**/
> +VOID
> +InitNoActionEvent (
> +  IN OUT TCG_PCR_EVENT2_HDR  *NoActionEvent,
> +  IN UINT32                  EventSize
> + )
> +{
> +  UINT32          DigestListCount;
> +  TPMI_ALG_HASH   HashAlgId;
> +  UINT8           *DigestBuffer;
> +
> +  DigestBuffer    = (UINT8 *)NoActionEvent->Digest.digests;
> +  DigestListCount = 0;
> +
> +  NoActionEvent->PCRIndex  = 0;
> +  NoActionEvent->EventType = EV_NO_ACTION;
> +
> +  //
> +  // Set Hash count & hashAlg accordingly, while 
> + Digest.digests[n].digest to all 0  //  ZeroMem 
> + (&NoActionEvent->Digest, sizeof(NoActionEvent->Digest));
> +
> +  if ((mTcgDxeData.BsCap.ActivePcrBanks &
> EFI_TCG2_BOOT_HASH_ALG_SHA1) != 0) {
> +     HashAlgId = TPM_ALG_SHA1;
> +     CopyMem (DigestBuffer, &HashAlgId, sizeof(TPMI_ALG_HASH));
> +     DigestBuffer += sizeof(TPMI_ALG_HASH) + GetHashSizeFromAlgo
> (HashAlgId);
> +     DigestListCount++;
> +  }
> +
> +  if ((mTcgDxeData.BsCap.ActivePcrBanks &
> EFI_TCG2_BOOT_HASH_ALG_SHA256) != 0) {
> +     HashAlgId = TPM_ALG_SHA256;
> +     CopyMem (DigestBuffer, &HashAlgId, sizeof(TPMI_ALG_HASH));
> +     DigestBuffer += sizeof(TPMI_ALG_HASH) + GetHashSizeFromAlgo
> (HashAlgId);
> +     DigestListCount++;
> +  }
> +
> +  if ((mTcgDxeData.BsCap.ActivePcrBanks &
> EFI_TCG2_BOOT_HASH_ALG_SHA384) != 0) {
> +    HashAlgId = TPM_ALG_SHA384;
> +    CopyMem (DigestBuffer, &HashAlgId, sizeof(TPMI_ALG_HASH));
> +    DigestBuffer += sizeof(TPMI_ALG_HASH) + GetHashSizeFromAlgo
> (HashAlgId);
> +    DigestListCount++;
> +  }
> +
> +  if ((mTcgDxeData.BsCap.ActivePcrBanks &
> EFI_TCG2_BOOT_HASH_ALG_SHA512) != 0) {
> +    HashAlgId = TPM_ALG_SHA512;
> +    CopyMem (DigestBuffer, &HashAlgId, sizeof(TPMI_ALG_HASH));
> +    DigestBuffer += sizeof(TPMI_ALG_HASH) + GetHashSizeFromAlgo
> (HashAlgId);
> +    DigestListCount++;
> +  }
> +
> +  if ((mTcgDxeData.BsCap.ActivePcrBanks &
> EFI_TCG2_BOOT_HASH_ALG_SM3_256) != 0) {
> +    HashAlgId = TPM_ALG_SM3_256;
> +    CopyMem (DigestBuffer, &HashAlgId, sizeof(TPMI_ALG_HASH));
> +    DigestBuffer += sizeof(TPMI_ALG_HASH) + GetHashSizeFromAlgo
> (HashAlgId);
> +    DigestListCount++;
> +  }
> +
> +  //
> +  // Set Digests Count
> +  //
> +  WriteUnaligned32 ((UINT32 *)&NoActionEvent->Digest.count, 
> + DigestListCount);
> +
> +  //
> +  // Set Event Size
> +  //
> +  WriteUnaligned32((UINT32 *)DigestBuffer, EventSize); }
> +
> +/**
> +
>    This function dump raw data with colume format.
> 
>    @param  Data  raw data
> @@ -1381,7 +1457,8 @@ SetupEventLog (
>    UINT32                          HashAlgorithmMaskCopied;
>    TCG_EfiSpecIDEventStruct        *TcgEfiSpecIdEventStruct;
>    UINT8                           TempBuf[sizeof(TCG_EfiSpecIDEventStruct) +
> sizeof(UINT32) + (HASH_COUNT * 
> sizeof(TCG_EfiSpecIdEventAlgorithmSize))
> + sizeof(UINT8)];
> -  TCG_PCR_EVENT_HDR               NoActionEvent;
> +  TCG_PCR_EVENT_HDR               SpecIdEvent;
> +  TCG_PCR_EVENT2_HDR              NoActionEvent;
>    TCG_EfiSpecIdEventAlgorithmSize *DigestSize;
>    TCG_EfiSpecIdEventAlgorithmSize *TempDigestSize;
>    UINT8                           *VendorInfoSize;
> @@ -1469,25 +1546,26 @@ SetupEventLog (
>          VendorInfoSize = (UINT8 *)TempDigestSize;
>          *VendorInfoSize = 0;
> 
> -        NoActionEvent.PCRIndex = 0;
> -        NoActionEvent.EventType = EV_NO_ACTION;
> -        ZeroMem (&NoActionEvent.Digest, sizeof(NoActionEvent.Digest));
> -        NoActionEvent.EventSize = (UINT32)GetTcgEfiSpecIdEventStructSize
> (TcgEfiSpecIdEventStruct);
> +        SpecIdEvent.PCRIndex = 0;
> +        SpecIdEvent.EventType = EV_NO_ACTION;
> +        ZeroMem (&SpecIdEvent.Digest, sizeof(SpecIdEvent.Digest));
> +        SpecIdEvent.EventSize = 
> + (UINT32)GetTcgEfiSpecIdEventStructSize
> + (TcgEfiSpecIdEventStruct);
> 
>          //
> -        // Log TcgEfiSpecIdEventStruct as the first Event
> +        // Log TcgEfiSpecIdEventStruct as the first Event. Event 
> + format is
> TCG_PCR_EVENT.
> +        //   TCG EFI Protocol Spec. Section 5.3 Event Log Header
>          //   TCG PC Client PFP spec. Section 9.2 Measurement Event Entries and
> Log
>          //
>          Status = TcgDxeLogEvent (
>                     mTcg2EventInfo[Index].LogFormat,
> -                   &NoActionEvent,
> -                   sizeof(NoActionEvent),
> +                   &SpecIdEvent,
> +                   sizeof(SpecIdEvent),
>                     (UINT8 *)TcgEfiSpecIdEventStruct,
> -                   NoActionEvent.EventSize
> +                   SpecIdEvent.EventSize
>                     );
> 
>          //
> -        // EfiStartupLocalityEvent
> +        // EfiStartupLocalityEvent. Event format is TCG_PCR_EVENT2
>          //
>          GuidHob.Guid = GetFirstGuidHob (&gTpm2StartupLocalityHobGuid);
>          if (GuidHob.Guid != NULL) {
> @@ -1496,25 +1574,25 @@ SetupEventLog (
>            //
>            StartupLocalityEvent.StartupLocality = *(UINT8 
> *)(GET_GUID_HOB_DATA (GuidHob.Guid));
>            CopyMem (StartupLocalityEvent.Signature, 
> TCG_EfiStartupLocalityEvent_SIGNATURE,
> sizeof(StartupLocalityEvent.Signature));
> -
> -          NoActionEvent.PCRIndex = 0;
> -          NoActionEvent.EventType = EV_NO_ACTION;
> -          ZeroMem (&NoActionEvent.Digest, sizeof(NoActionEvent.Digest));
> -          NoActionEvent.EventSize = sizeof(StartupLocalityEvent);
> -
>            DEBUG ((DEBUG_INFO, "SetupEventLog: Set Locality from HOB 
> into StartupLocalityEvent 0x%02x\n", 
> StartupLocalityEvent.StartupLocality));
> 
>            //
> +          // Initialize StartupLocalityEvent
> +          //
> +          InitNoActionEvent(&NoActionEvent, 
> + sizeof(StartupLocalityEvent));
> +
> +          //
>            // Log EfiStartupLocalityEvent as the second Event
>            //   TCG PC Client PFP spec. Section 9.3.4.3 Startup Locality Event
>            //
>            Status = TcgDxeLogEvent (
>                       mTcg2EventInfo[Index].LogFormat,
>                       &NoActionEvent,
> -                     sizeof(NoActionEvent),
> +                     sizeof(NoActionEvent.PCRIndex) +
> + sizeof(NoActionEvent.EventType) + GetDigestListBinSize
> + (&NoActionEvent.Digest) + sizeof(NoActionEvent.EventSize),
>                       (UINT8 *)&StartupLocalityEvent,
> -                     NoActionEvent.EventSize
> +                     sizeof(StartupLocalityEvent)
>                       );
> +
>          }
>        }
>      }
> --
> 1.9.5.msysgit.1



      reply	other threads:[~2017-02-06  0:57 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-02-04  1:33 [PATCH 1/2] SecurityPkg: Tcg2Dxe: Log EV_NO_ACTION event in TCG_PCR_EVENT2 format Zhang, Chao B
2017-02-04  1:33 ` [PATCH 2/2] MdePkg: UefiTcgPlatform.h: Add TCG_PCR_EVENT2_HDR definition Zhang, Chao B
2017-02-04  8:49   ` Long, Qin
2017-02-04  8:53 ` [PATCH 1/2] SecurityPkg: Tcg2Dxe: Log EV_NO_ACTION event in TCG_PCR_EVENT2 format Long, Qin
2017-02-06  0:57   ` Zhang, Chao B [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=FF72C7E4248F3C4E9BDF19D4918E90F2494FDD18@shsmsx102.ccr.corp.intel.com \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox