SecurityPkg/TPM2: Move CopyDigestListToBuffer() to Tpm2CommandLib

This patch just moves function CopyDigestListToBuffer() from
drivers to library with HashAlgorithmMask parameter added to
make the interface more applicable.

The related function IsHashAlgSupportedInHashAlgorithmMask()
is also moved from drivers to library as internal function.

Cc: Chao B Zhang <chao.b.zhang@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Jiewen Yao <jiewen.yao@intel.com>
Signed-off-by: Star Zeng <star.zeng@intel.com>
Reviewed-by: Chao Zhang <chao.b.zhang@intel.com>

1 files changed, 1 insertions, 82 deletions

diff --git a/SecurityPkg/Tcg/Tcg2Pei/Tcg2Pei.c b/SecurityPkg/Tcg/Tcg2Pei/Tcg2Pei.c
index c67cdffe48..a72b8d9bda 100644
--- a/SecurityPkg/Tcg/Tcg2Pei/Tcg2Pei.c
+++ b/SecurityPkg/Tcg/Tcg2Pei/Tcg2Pei.c
@@ -190,87 +190,6 @@ EndofPeiSignalNotifyCallBack (
 }

 

 /**

-  Return if hash alg is supported in TPM PCR bank.

-

-  @param HashAlg  Hash algorithm to be checked.

-

-  @retval TRUE  Hash algorithm is supported.

-  @retval FALSE Hash algorithm is not supported.

-**/

-BOOLEAN

-IsHashAlgSupportedInPcrBank (

-  IN TPMI_ALG_HASH  HashAlg

-  )

-{

-  UINT32  ActivePcrBanks;

-

-  ActivePcrBanks = PcdGet32 (PcdTpm2HashMask);

-  switch (HashAlg) {

-  case TPM_ALG_SHA1:

-    if ((ActivePcrBanks & EFI_TCG2_BOOT_HASH_ALG_SHA1) != 0) {

-      return TRUE;

-    }

-    break;

-  case TPM_ALG_SHA256:

-    if ((ActivePcrBanks & EFI_TCG2_BOOT_HASH_ALG_SHA256) != 0) {

-      return TRUE;

-    }

-    break;

-  case TPM_ALG_SHA384:

-    if ((ActivePcrBanks & EFI_TCG2_BOOT_HASH_ALG_SHA384) != 0) {

-      return TRUE;

-    }

-    break;

-  case TPM_ALG_SHA512:

-    if ((ActivePcrBanks & EFI_TCG2_BOOT_HASH_ALG_SHA512) != 0) {

-      return TRUE;

-    }

-    break;

-  case TPM_ALG_SM3_256:

-    if ((ActivePcrBanks & EFI_TCG2_BOOT_HASH_ALG_SM3_256) != 0) {

-      return TRUE;

-    }

-    break;

-  }

-

-  return FALSE;

-}

-

-/**

-  Copy TPML_DIGEST_VALUES into a buffer

-

-  @param[in,out] Buffer        Buffer to hold TPML_DIGEST_VALUES.

-  @param[in]     DigestList    TPML_DIGEST_VALUES to be copied.

-

-  @return The end of buffer to hold TPML_DIGEST_VALUES.

-**/

-VOID *

-CopyDigestListToBuffer (

-  IN OUT VOID                       *Buffer,

-  IN TPML_DIGEST_VALUES             *DigestList

-  )

-{

-  UINTN  Index;

-  UINT16 DigestSize;

-

-  CopyMem (Buffer, &DigestList->count, sizeof(DigestList->count));

-  Buffer = (UINT8 *)Buffer + sizeof(DigestList->count);

-  for (Index = 0; Index < DigestList->count; Index++) {

-    if (!IsHashAlgSupportedInPcrBank (DigestList->digests[Index].hashAlg)) {

-      DEBUG ((EFI_D_ERROR, "WARNING: TPM2 Event log has HashAlg unsupported by PCR bank (0x%x)\n", DigestList->digests[Index].hashAlg));

-      continue;

-    }

-    CopyMem (Buffer, &DigestList->digests[Index].hashAlg, sizeof(DigestList->digests[Index].hashAlg));

-    Buffer = (UINT8 *)Buffer + sizeof(DigestList->digests[Index].hashAlg);

-    DigestSize = GetHashSizeFromAlgo (DigestList->digests[Index].hashAlg);

-    CopyMem (Buffer, &DigestList->digests[Index].digest, DigestSize);

-    Buffer = (UINT8 *)Buffer + DigestSize;

-  }

-

-  return Buffer;

-}

-

-/**

   Set Tpm2HashMask PCD value according to TPM2 PCR bank.

 **/

 VOID

@@ -390,7 +309,7 @@ LogHashEvent (
         TcgPcrEvent2->PCRIndex = NewEventHdr->PCRIndex;

         TcgPcrEvent2->EventType = NewEventHdr->EventType;

         DigestBuffer = (UINT8 *)&TcgPcrEvent2->Digest;

-        DigestBuffer = CopyDigestListToBuffer (DigestBuffer, DigestList);

+        DigestBuffer = CopyDigestListToBuffer (DigestBuffer, DigestList, PcdGet32 (PcdTpm2HashMask));

         CopyMem (DigestBuffer, &NewEventHdr->EventSize, sizeof(TcgPcrEvent2->EventSize));

         DigestBuffer = DigestBuffer + sizeof(TcgPcrEvent2->EventSize);

         CopyMem (DigestBuffer, NewEventData, NewEventHdr->EventSize);