]> git.proxmox.com Git - mirror_edk2.git/blobdiff - SecurityPkg/Library/DxeImageVerificationLib/DxeImageVerificationLib.c
SecurityPkg: remove PE/COFF header workaround for ELILO on IPF
[mirror_edk2.git] / SecurityPkg / Library / DxeImageVerificationLib / DxeImageVerificationLib.c
index 4b4d3bf77de53eefdbe0e9221e95434fd1d003aa..66d96a9396b9137cd4a7bb3367c82affc9f59389 100644 (file)
@@ -12,7 +12,8 @@
   DxeImageVerificationHandler(), HashPeImageByType(), HashPeImage() function will accept\r
   untrusted PE/COFF image and validate its data structure within this image buffer before use.\r
 \r
-Copyright (c) 2009 - 2015, Intel Corporation. All rights reserved.<BR>\r
+Copyright (c) 2009 - 2018, Intel Corporation. All rights reserved.<BR>\r
+(C) Copyright 2016 Hewlett Packard Enterprise Development LP<BR>\r
 This program and the accompanying materials\r
 are licensed and made available under the terms and conditions of the BSD License\r
 which accompanies this distribution.  The full text of the license may be found at\r
@@ -71,6 +72,8 @@ HASH_TABLE mHash[] = {
   { L"SHA512", 64, &mHashOidValue[32], 9, Sha512GetContextSize, Sha512Init, Sha512Update, Sha512Final}\r
 };\r
 \r
+EFI_STRING mHashTypeStr;\r
+\r
 /**\r
   SecureBoot Hook for processing image verification.\r
 \r
@@ -277,6 +280,9 @@ GetImageType (
   PE/COFF image is external input, so this function will validate its data structure\r
   within this image buffer before use.\r
 \r
+  Notes: PE/COFF image has been checked by BasePeCoffLib PeCoffLoaderGetImageInfo() in\r
+  its caller function DxeImageVerificationHandler().\r
+\r
   @param[in]    HashAlg   Hash algorithm type.\r
 \r
   @retval TRUE            Successfully hash image.\r
@@ -289,7 +295,6 @@ HashPeImage (
   )\r
 {\r
   BOOLEAN                   Status;\r
-  UINT16                    Magic;\r
   EFI_IMAGE_SECTION_HEADER  *Section;\r
   VOID                      *HashCtx;\r
   UINTN                     CtxSize;\r
@@ -340,6 +345,7 @@ HashPeImage (
     return FALSE;\r
   }\r
 \r
+  mHashTypeStr = mHash[HashAlg].Name;\r
   CtxSize   = mHash[HashAlg].GetContextSize();\r
 \r
   HashCtx = AllocatePool (CtxSize);\r
@@ -360,37 +366,23 @@ HashPeImage (
   // Measuring PE/COFF Image Header;\r
   // But CheckSum field and SECURITY data directory (certificate) are excluded\r
   //\r
-  if (mNtHeader.Pe32->FileHeader.Machine == IMAGE_FILE_MACHINE_IA64 && mNtHeader.Pe32->OptionalHeader.Magic == EFI_IMAGE_NT_OPTIONAL_HDR32_MAGIC) {\r
-    //\r
-    // NOTE: Some versions of Linux ELILO for Itanium have an incorrect magic value\r
-    //       in the PE/COFF Header. If the MachineType is Itanium(IA64) and the\r
-    //       Magic value in the OptionalHeader is EFI_IMAGE_NT_OPTIONAL_HDR32_MAGIC\r
-    //       then override the magic value to EFI_IMAGE_NT_OPTIONAL_HDR64_MAGIC\r
-    //\r
-    Magic = EFI_IMAGE_NT_OPTIONAL_HDR64_MAGIC;\r
-  } else {\r
-    //\r
-    // Get the magic value from the PE/COFF Optional Header\r
-    //\r
-    Magic =  mNtHeader.Pe32->OptionalHeader.Magic;\r
-  }\r
 \r
   //\r
   // 3.  Calculate the distance from the base of the image header to the image checksum address.\r
   // 4.  Hash the image header from its base to beginning of the image checksum.\r
   //\r
   HashBase = mImageBase;\r
-  if (Magic == EFI_IMAGE_NT_OPTIONAL_HDR32_MAGIC) {\r
+  if (mNtHeader.Pe32->OptionalHeader.Magic == EFI_IMAGE_NT_OPTIONAL_HDR32_MAGIC) {\r
     //\r
     // Use PE32 offset.\r
     //\r
-    HashSize = (UINTN) ((UINT8 *) (&mNtHeader.Pe32->OptionalHeader.CheckSum) - HashBase);\r
+    HashSize = (UINTN) (&mNtHeader.Pe32->OptionalHeader.CheckSum) - (UINTN) HashBase;\r
     NumberOfRvaAndSizes = mNtHeader.Pe32->OptionalHeader.NumberOfRvaAndSizes;\r
-  } else if (Magic == EFI_IMAGE_NT_OPTIONAL_HDR64_MAGIC) {\r
+  } else if (mNtHeader.Pe32->OptionalHeader.Magic == EFI_IMAGE_NT_OPTIONAL_HDR64_MAGIC) {\r
     //\r
     // Use PE32+ offset.\r
     //\r
-    HashSize = (UINTN) ((UINT8 *) (&mNtHeader.Pe32Plus->OptionalHeader.CheckSum) - HashBase);\r
+    HashSize = (UINTN) (&mNtHeader.Pe32Plus->OptionalHeader.CheckSum) - (UINTN) HashBase;\r
     NumberOfRvaAndSizes = mNtHeader.Pe32Plus->OptionalHeader.NumberOfRvaAndSizes;\r
   } else {\r
     //\r
@@ -413,18 +405,18 @@ HashPeImage (
     // 6.  Since there is no Cert Directory in optional header, hash everything\r
     //     from the end of the checksum to the end of image header.\r
     //\r
-    if (Magic == EFI_IMAGE_NT_OPTIONAL_HDR32_MAGIC) {\r
+    if (mNtHeader.Pe32->OptionalHeader.Magic == EFI_IMAGE_NT_OPTIONAL_HDR32_MAGIC) {\r
       //\r
       // Use PE32 offset.\r
       //\r
       HashBase = (UINT8 *) &mNtHeader.Pe32->OptionalHeader.CheckSum + sizeof (UINT32);\r
-      HashSize = mNtHeader.Pe32->OptionalHeader.SizeOfHeaders - (UINTN) (HashBase - mImageBase);\r
+      HashSize = mNtHeader.Pe32->OptionalHeader.SizeOfHeaders - ((UINTN) HashBase - (UINTN) mImageBase);\r
     } else {\r
       //\r
       // Use PE32+ offset.\r
       //\r
       HashBase = (UINT8 *) &mNtHeader.Pe32Plus->OptionalHeader.CheckSum + sizeof (UINT32);\r
-      HashSize = mNtHeader.Pe32Plus->OptionalHeader.SizeOfHeaders - (UINTN) (HashBase - mImageBase);\r
+      HashSize = mNtHeader.Pe32Plus->OptionalHeader.SizeOfHeaders - ((UINTN) HashBase - (UINTN) mImageBase);\r
     }\r
 \r
     if (HashSize != 0) {\r
@@ -437,18 +429,18 @@ HashPeImage (
     //\r
     // 7.  Hash everything from the end of the checksum to the start of the Cert Directory.\r
     //\r
-    if (Magic == EFI_IMAGE_NT_OPTIONAL_HDR32_MAGIC) {\r
+    if (mNtHeader.Pe32->OptionalHeader.Magic == EFI_IMAGE_NT_OPTIONAL_HDR32_MAGIC) {\r
       //\r
       // Use PE32 offset.\r
       //\r
       HashBase = (UINT8 *) &mNtHeader.Pe32->OptionalHeader.CheckSum + sizeof (UINT32);\r
-      HashSize = (UINTN) ((UINT8 *) (&mNtHeader.Pe32->OptionalHeader.DataDirectory[EFI_IMAGE_DIRECTORY_ENTRY_SECURITY]) - HashBase);\r
+      HashSize = (UINTN) (&mNtHeader.Pe32->OptionalHeader.DataDirectory[EFI_IMAGE_DIRECTORY_ENTRY_SECURITY]) - (UINTN) HashBase;\r
     } else {\r
       //\r
       // Use PE32+ offset.\r
       //\r
       HashBase = (UINT8 *) &mNtHeader.Pe32Plus->OptionalHeader.CheckSum + sizeof (UINT32);\r
-      HashSize = (UINTN) ((UINT8 *) (&mNtHeader.Pe32Plus->OptionalHeader.DataDirectory[EFI_IMAGE_DIRECTORY_ENTRY_SECURITY]) - HashBase);\r
+      HashSize = (UINTN) (&mNtHeader.Pe32Plus->OptionalHeader.DataDirectory[EFI_IMAGE_DIRECTORY_ENTRY_SECURITY]) - (UINTN) HashBase;\r
     }\r
 \r
     if (HashSize != 0) {\r
@@ -462,18 +454,18 @@ HashPeImage (
     // 8.  Skip over the Cert Directory. (It is sizeof(IMAGE_DATA_DIRECTORY) bytes.)\r
     // 9.  Hash everything from the end of the Cert Directory to the end of image header.\r
     //\r
-    if (Magic == EFI_IMAGE_NT_OPTIONAL_HDR32_MAGIC) {\r
+    if (mNtHeader.Pe32->OptionalHeader.Magic == EFI_IMAGE_NT_OPTIONAL_HDR32_MAGIC) {\r
       //\r
       // Use PE32 offset\r
       //\r
       HashBase = (UINT8 *) &mNtHeader.Pe32->OptionalHeader.DataDirectory[EFI_IMAGE_DIRECTORY_ENTRY_SECURITY + 1];\r
-      HashSize = mNtHeader.Pe32->OptionalHeader.SizeOfHeaders - (UINTN) (HashBase - mImageBase);\r
+      HashSize = mNtHeader.Pe32->OptionalHeader.SizeOfHeaders - ((UINTN) HashBase - (UINTN) mImageBase);\r
     } else {\r
       //\r
       // Use PE32+ offset.\r
       //\r
       HashBase = (UINT8 *) &mNtHeader.Pe32Plus->OptionalHeader.DataDirectory[EFI_IMAGE_DIRECTORY_ENTRY_SECURITY + 1];\r
-      HashSize = mNtHeader.Pe32Plus->OptionalHeader.SizeOfHeaders - (UINTN) (HashBase - mImageBase);\r
+      HashSize = mNtHeader.Pe32Plus->OptionalHeader.SizeOfHeaders - ((UINTN) HashBase - (UINTN) mImageBase);\r
     }\r
 \r
     if (HashSize != 0) {\r
@@ -487,7 +479,7 @@ HashPeImage (
   //\r
   // 10. Set the SUM_OF_BYTES_HASHED to the size of the header.\r
   //\r
-  if (Magic == EFI_IMAGE_NT_OPTIONAL_HDR32_MAGIC) {\r
+  if (mNtHeader.Pe32->OptionalHeader.Magic == EFI_IMAGE_NT_OPTIONAL_HDR32_MAGIC) {\r
     //\r
     // Use PE32 offset.\r
     //\r
@@ -570,7 +562,7 @@ HashPeImage (
     if (NumberOfRvaAndSizes <= EFI_IMAGE_DIRECTORY_ENTRY_SECURITY) {\r
       CertSize = 0;\r
     } else {\r
-      if (Magic == EFI_IMAGE_NT_OPTIONAL_HDR32_MAGIC) {\r
+      if (mNtHeader.Pe32->OptionalHeader.Magic == EFI_IMAGE_NT_OPTIONAL_HDR32_MAGIC) {\r
         //\r
         // Use PE32 offset.\r
         //\r
@@ -711,58 +703,6 @@ GetImageExeInfoTableSize (
   return TotalSize;\r
 }\r
 \r
-/**\r
-  Create signature list based on input signature data and certificate type GUID. Caller is reposible \r
-  to free new created SignatureList.\r
-\r
-  @param[in]   SignatureData           Signature data in SignatureList.\r
-  @param[in]   SignatureDataSize       Signature data size.\r
-  @param[in]   CertType                Certificate Type.\r
-  @param[out]  SignatureList           Created SignatureList.\r
-  @param[out]  SignatureListSize       Created SignatureListSize.\r
-\r
-  @return EFI_OUT_OF_RESOURCES         The operation is failed due to lack of resources.\r
-  @retval EFI_SUCCESS          Successfully create signature list.\r
-\r
-**/\r
-EFI_STATUS\r
-CreateSignatureList(\r
-  IN UINT8                *SignatureData,\r
-  IN UINTN                SignatureDataSize,\r
-  IN EFI_GUID             *CertType,\r
-  OUT EFI_SIGNATURE_LIST  **SignatureList,\r
-  OUT UINTN               *SignatureListSize\r
-  )\r
-{\r
-  EFI_SIGNATURE_LIST   *SignList;\r
-  UINTN                SignListSize;\r
-  EFI_SIGNATURE_DATA   *Signature;\r
-\r
-  SignList       = NULL;\r
-  *SignatureList = NULL;\r
-\r
-  SignListSize = sizeof (EFI_SIGNATURE_LIST) + sizeof (EFI_SIGNATURE_DATA) - 1 + SignatureDataSize;\r
-  SignList     = (EFI_SIGNATURE_LIST *) AllocateZeroPool (SignListSize);\r
-  if (SignList == NULL) {\r
-    return EFI_OUT_OF_RESOURCES;\r
-  }\r
-\r
-  SignList->SignatureHeaderSize = 0;\r
-  SignList->SignatureListSize   = (UINT32) SignListSize;\r
-  SignList->SignatureSize       = (UINT32) SignatureDataSize + sizeof (EFI_SIGNATURE_DATA) - 1;\r
-  CopyMem (&SignList->SignatureType, CertType, sizeof (EFI_GUID));\r
-\r
-  DEBUG((EFI_D_INFO, "SignatureDataSize %x\n", SignatureDataSize));\r
-  Signature = (EFI_SIGNATURE_DATA *) ((UINT8 *) SignList + sizeof (EFI_SIGNATURE_LIST));\r
-  CopyMem (Signature->SignatureData, SignatureData, SignatureDataSize);\r
-\r
-  *SignatureList     = SignList;\r
-  *SignatureListSize = SignListSize;\r
-\r
-  return EFI_SUCCESS;\r
-\r
-}\r
-\r
 /**\r
   Create an Image Execution Information Table entry and add it to system configuration table.\r
 \r
@@ -1071,7 +1011,12 @@ IsSignatureFoundInDatabase (
           // Find the signature in database.\r
           //\r
           IsFound = TRUE;\r
-          SecureBootHook (VariableName, &gEfiImageSecurityDatabaseGuid, CertList->SignatureSize, Cert);\r
+          //\r
+          // Entries in UEFI_IMAGE_SECURITY_DATABASE that are used to validate image should be measured\r
+          //\r
+          if (StrCmp(VariableName, EFI_IMAGE_SECURITY_DATABASE) == 0) {\r
+            SecureBootHook (VariableName, &gEfiImageSecurityDatabaseGuid, CertList->SignatureSize, Cert);\r
+          }\r
           break;\r
         }\r
 \r
@@ -1149,61 +1094,14 @@ IsTimeZero (
 }\r
 \r
 /**\r
-  Record multiple certificate list & verification state of a verified image to \r
-  IMAGE_EXECUTION_TABLE.\r
-\r
-  @param[in]  CertBuf              Certificate list buffer.\r
-  @param[in]  CertBufLength        Certificate list buffer.\r
-  @param[in]  Action               Certificate list action to be record.\r
-  @param[in]  ImageName            Image name.\r
-  @param[in]  ImageDevicePath      Image device path.\r
-\r
-**/\r
-VOID \r
-RecordCertListToImageExeuctionTable(\r
-  IN UINT8                          *CertBuf,\r
-  IN UINTN                           CertBufLength,\r
-  IN EFI_IMAGE_EXECUTION_ACTION      Action,\r
-  IN CHAR16                         *ImageName OPTIONAL,\r
-  IN CONST EFI_DEVICE_PATH_PROTOCOL *ImageDevicePath OPTIONAL\r
-  )\r
-{\r
-  UINT8               CertNumber;\r
-  UINT8               *CertPtr;\r
-  UINTN               Index;\r
-  UINT8               *Cert;\r
-  UINTN               CertSize;\r
-  EFI_STATUS          Status;\r
-  EFI_SIGNATURE_LIST  *SignatureList;\r
-  UINTN               SignatureListSize;\r
-\r
-  CertNumber = (UINT8) (*CertBuf);\r
-  CertPtr    = CertBuf + 1;\r
-  for (Index = 0; Index < CertNumber; Index++) {\r
-    CertSize = (UINTN) ReadUnaligned32 ((UINT32 *)CertPtr);\r
-    Cert     = (UINT8 *)CertPtr + sizeof (UINT32);\r
-\r
-    //\r
-    // Record all cert in cert chain to be passed\r
-    //\r
-    Status = CreateSignatureList(Cert, CertSize, &gEfiCertX509Guid, &SignatureList, &SignatureListSize);\r
-    if (!EFI_ERROR(Status)) {\r
-      AddImageExeInfo (Action, ImageName, ImageDevicePath, SignatureList, SignatureListSize);\r
-      FreePool (SignatureList);\r
-    }\r
-  }\r
-}\r
-\r
-\r
-/**\r
-  Check whether the timestamp signature is valid and the signing time is also earlier than \r
+  Check whether the timestamp signature is valid and the signing time is also earlier than\r
   the revocation time.\r
 \r
   @param[in]  AuthData        Pointer to the Authenticode signature retrieved from signed image.\r
   @param[in]  AuthDataSize    Size of the Authenticode signature in bytes.\r
   @param[in]  RevocationTime  The time that the certificate was revoked.\r
 \r
-  @retval TRUE      Timestamp signature is valid and signing time is no later than the \r
+  @retval TRUE      Timestamp signature is valid and signing time is no later than the\r
                     revocation time.\r
   @retval FALSE     Timestamp signature is not valid or the signing time is later than the\r
                     revocation time.\r
@@ -1305,11 +1203,8 @@ Done:
   Check whether the image signature is forbidden by the forbidden database (dbx).\r
   The image is forbidden to load if any certificates for signing are revoked before signing time.\r
 \r
-  @param[in]  AuthData             Pointer to the Authenticode signature retrieved from the signed image.\r
-  @param[in]  AuthDataSize         Size of the Authenticode signature in bytes.\r
-  @param[in]  IsAuditMode          Whether system Secure Boot Mode is in AuditMode.\r
-  @param[in]  ImageName            Name of the image to verify.\r
-  @param[in]  ImageDevicePath      DevicePath of the image to verify.\r
+  @param[in]  AuthData      Pointer to the Authenticode signature retrieved from the signed image.\r
+  @param[in]  AuthDataSize  Size of the Authenticode signature in bytes.\r
 \r
   @retval TRUE              Image is forbidden by dbx.\r
   @retval FALSE             Image is not forbidden by dbx.\r
@@ -1317,11 +1212,8 @@ Done:
 **/\r
 BOOLEAN\r
 IsForbiddenByDbx (\r
-  IN UINT8                          *AuthData,\r
-  IN UINTN                          AuthDataSize,\r
-  IN BOOLEAN                        IsAuditMode,\r
-  IN CHAR16                         *ImageName OPTIONAL,\r
-  IN CONST EFI_DEVICE_PATH_PROTOCOL *ImageDevicePath OPTIONAL\r
+  IN UINT8                  *AuthData,\r
+  IN UINTN                  AuthDataSize\r
   )\r
 {\r
   EFI_STATUS                Status;\r
@@ -1344,10 +1236,6 @@ IsForbiddenByDbx (
   UINT8                     *Cert;\r
   UINTN                     CertSize;\r
   EFI_TIME                  RevocationTime;\r
-  UINT8                     *SignerCert;\r
-  UINTN                     SignerCertLength;\r
-  UINT8                     *UnchainCert;\r
-  UINTN                     UnchainCertLength;\r
   //\r
   // Variable Initialization\r
   //\r
@@ -1362,10 +1250,6 @@ IsForbiddenByDbx (
   BufferLength      = 0;\r
   TrustedCert       = NULL;\r
   TrustedCertLength = 0;\r
-  SignerCert        = NULL;\r
-  SignerCertLength  = 0;\r
-  UnchainCert       = NULL;\r
-  UnchainCertLength = 0;\r
 \r
   //\r
   // The image will not be forbidden if dbx can't be got.\r
@@ -1415,7 +1299,7 @@ IsForbiddenByDbx (
                         mImageDigestSize\r
                         );\r
         if (IsForbidden) {\r
-          SecureBootHook (EFI_IMAGE_SECURITY_DATABASE1, &gEfiImageSecurityDatabaseGuid, CertList->SignatureSize, CertData);\r
+          DEBUG ((DEBUG_INFO, "DxeImageVerificationLib: Image is signed but signature is forbidden by DBX.\n"));\r
           goto Done;\r
         }\r
 \r
@@ -1457,6 +1341,10 @@ IsForbiddenByDbx (
   for (Index = 0; Index < CertNumber; Index++) {\r
     CertSize = (UINTN) ReadUnaligned32 ((UINT32 *)CertPtr);\r
     Cert     = (UINT8 *)CertPtr + sizeof (UINT32);\r
+    //\r
+    // Advance CertPtr to the next cert in image signer's cert list\r
+    //\r
+    CertPtr = CertPtr + sizeof (UINT32) + CertSize;\r
 \r
     if (IsCertHashFoundInDatabase (Cert, CertSize, (EFI_SIGNATURE_LIST *)Data, DataSize, &RevocationTime)) {\r
       //\r
@@ -1465,49 +1353,24 @@ IsForbiddenByDbx (
       IsForbidden = TRUE;\r
       if (PassTimestampCheck (AuthData, AuthDataSize, &RevocationTime)) {\r
         IsForbidden = FALSE;\r
+        //\r
+        // Pass DBT check. Continue to check other certs in image signer's cert list against DBX, DBT\r
+        //\r
+        continue;\r
       }\r
+      DEBUG ((DEBUG_INFO, "DxeImageVerificationLib: Image is signed but signature failed the timestamp check.\n"));\r
       goto Done;\r
     }\r
 \r
-    CertPtr = CertPtr + sizeof (UINT32) + CertSize;\r
   }\r
 \r
 Done:\r
-  if (IsForbidden && IsAuditMode) {\r
-    Pkcs7GetCertificatesList(AuthData, AuthDataSize, &SignerCert, &SignerCertLength, &UnchainCert, &UnchainCertLength);\r
-\r
-    //\r
-    // Record all certs in image to be failed\r
-    //\r
-    if ((SignerCertLength != 0) && (SignerCert != NULL)) {\r
-      RecordCertListToImageExeuctionTable(\r
-        SignerCert,\r
-        SignerCertLength,\r
-        EFI_IMAGE_EXECUTION_AUTH_SIG_FAILED | EFI_IMAGE_EXECUTION_INITIALIZED,\r
-        ImageName,\r
-        ImageDevicePath\r
-        );\r
-    }\r
-\r
-    if ((UnchainCertLength != 0) && (UnchainCert != NULL)) {\r
-      RecordCertListToImageExeuctionTable(\r
-        UnchainCert,\r
-        UnchainCertLength,\r
-        EFI_IMAGE_EXECUTION_AUTH_SIG_FAILED | EFI_IMAGE_EXECUTION_INITIALIZED,\r
-        ImageName,\r
-        ImageDevicePath\r
-        );\r
-    }\r
-  }\r
-\r
   if (Data != NULL) {\r
     FreePool (Data);\r
   }\r
 \r
   Pkcs7FreeSigners (CertBuffer);\r
   Pkcs7FreeSigners (TrustedCert);\r
-  Pkcs7FreeSigners (SignerCert);\r
-  Pkcs7FreeSigners (UnchainCert);\r
 \r
   return IsForbidden;\r
 }\r
@@ -1516,11 +1379,8 @@ Done:
 /**\r
   Check whether the image signature can be verified by the trusted certificates in DB database.\r
 \r
-  @param[in]  AuthData              Pointer to the Authenticode signature retrieved from signed image.\r
-  @param[in]  AuthDataSize          Size of the Authenticode signature in bytes.\r
-  @param[in]  IsAuditMode           Whether system Secure Boot Mode is in AuditMode.\r
-  @param[in]  ImageName             Name of the image to verify.\r
-  @param[in]  ImageDevicePath       DevicePath of the image to verify.\r
+  @param[in]  AuthData      Pointer to the Authenticode signature retrieved from signed image.\r
+  @param[in]  AuthDataSize  Size of the Authenticode signature in bytes.\r
 \r
   @retval TRUE         Image passed verification using certificate in db.\r
   @retval FALSE        Image didn't pass verification using certificate in db.\r
@@ -1528,11 +1388,8 @@ Done:
 **/\r
 BOOLEAN\r
 IsAllowedByDb (\r
-  IN UINT8                          *AuthData,\r
-  IN UINTN                          AuthDataSize,\r
-  IN BOOLEAN                        IsAuditMode,\r
-  IN CHAR16                         *ImageName OPTIONAL,\r
-  IN CONST EFI_DEVICE_PATH_PROTOCOL *ImageDevicePath OPTIONAL\r
+  IN UINT8              *AuthData,\r
+  IN UINTN              AuthDataSize\r
   )\r
 {\r
   EFI_STATUS                Status;\r
@@ -1548,10 +1405,6 @@ IsAllowedByDb (
   UINTN                     DbxDataSize;\r
   UINT8                     *DbxData;\r
   EFI_TIME                  RevocationTime;\r
-  UINT8                     *SignerCert;\r
-  UINTN                     SignerCertLength;\r
-  UINT8                     *UnchainCert;\r
-  UINTN                     UnchainCertLength;\r
 \r
   Data              = NULL;\r
   CertList          = NULL;\r
@@ -1560,10 +1413,6 @@ IsAllowedByDb (
   DbxData           = NULL;\r
   RootCertSize      = 0;\r
   VerifyStatus      = FALSE;\r
-  SignerCert        = NULL;\r
-  SignerCertLength  = 0;\r
-  UnchainCert       = NULL;\r
-  UnchainCertLength = 0;\r
 \r
   DataSize = 0;\r
   Status   = gRT->GetVariable (EFI_IMAGE_SECURITY_DATABASE, &gEfiImageSecurityDatabaseGuid, NULL, &DataSize, NULL);\r
@@ -1625,9 +1474,12 @@ IsAllowedByDb (
 \r
             if (IsCertHashFoundInDatabase (RootCert, RootCertSize, (EFI_SIGNATURE_LIST *)DbxData, DbxDataSize, &RevocationTime)) {\r
               //\r
-              // Check the timestamp signature and signing time to determine if the image can be trusted.\r
+              // Check the timestamp signature and signing time to determine if the RootCert can be trusted.\r
               //\r
               VerifyStatus = PassTimestampCheck (AuthData, AuthDataSize, &RevocationTime);\r
+              if (!VerifyStatus) {\r
+                DEBUG ((DEBUG_INFO, "DxeImageVerificationLib: Image is signed and signature is accepted by DB, but its root cert failed the timestamp check.\n"));\r
+              }\r
             }\r
 \r
             goto Done;\r
@@ -1648,62 +1500,6 @@ Done:
     SecureBootHook (EFI_IMAGE_SECURITY_DATABASE, &gEfiImageSecurityDatabaseGuid, CertList->SignatureSize, CertData);\r
   }\r
 \r
-  if (IsAuditMode) {\r
-\r
-    Pkcs7GetCertificatesList(AuthData, AuthDataSize, &SignerCert, &SignerCertLength, &UnchainCert, &UnchainCertLength);\r
-    if (VerifyStatus) {\r
-      if ((SignerCertLength != 0) && (SignerCert != NULL)) {\r
-        //\r
-        // Record all cert in signer's cert chain to be passed\r
-        //\r
-        RecordCertListToImageExeuctionTable(\r
-          SignerCert,\r
-          SignerCertLength,\r
-          EFI_IMAGE_EXECUTION_AUTH_SIG_PASSED | EFI_IMAGE_EXECUTION_INITIALIZED,\r
-          ImageName,\r
-          ImageDevicePath\r
-          );\r
-      }\r
-\r
-      if ((UnchainCertLength != 0) && (UnchainCert != NULL)) {\r
-        //\r
-        // Record all certs in unchained certificates lists to be failed\r
-        //\r
-        RecordCertListToImageExeuctionTable(\r
-          UnchainCert,\r
-          UnchainCertLength,\r
-          EFI_IMAGE_EXECUTION_AUTH_SIG_FAILED | EFI_IMAGE_EXECUTION_INITIALIZED,\r
-          ImageName,\r
-          ImageDevicePath\r
-          );\r
-      }\r
-    } else {\r
-      //\r
-      // Record all certs in image to be failed\r
-      //\r
-      if ((SignerCertLength != 0) && (SignerCert != NULL)) {\r
-        RecordCertListToImageExeuctionTable(\r
-          SignerCert,\r
-          SignerCertLength,\r
-          EFI_IMAGE_EXECUTION_AUTH_SIG_FAILED | EFI_IMAGE_EXECUTION_INITIALIZED,\r
-          ImageName,\r
-          ImageDevicePath\r
-          );\r
-      }\r
-\r
-      if ((UnchainCertLength != 0) && (UnchainCert != NULL)) {\r
-        RecordCertListToImageExeuctionTable(\r
-          UnchainCert,\r
-          UnchainCertLength,\r
-          EFI_IMAGE_EXECUTION_AUTH_SIG_FAILED | EFI_IMAGE_EXECUTION_INITIALIZED,\r
-          ImageName,\r
-          ImageDevicePath\r
-          );\r
-      }\r
-    }\r
-  }\r
-\r
-\r
   if (Data != NULL) {\r
     FreePool (Data);\r
   }\r
@@ -1711,369 +1507,9 @@ Done:
     FreePool (DbxData);\r
   }\r
 \r
-  Pkcs7FreeSigners (SignerCert);\r
-  Pkcs7FreeSigners (UnchainCert);\r
-\r
   return VerifyStatus;\r
 }\r
 \r
-/**\r
-  Provide verification service for signed images in AuditMode, which include both signature validation\r
-  and platform policy control. For signature types, both UEFI WIN_CERTIFICATE_UEFI_GUID and\r
-  MSFT Authenticode type signatures are supported. \r
-\r
-  In this implementation, only verify external executables when in AuditMode.\r
-  Executables from FV is bypass, so pass in AuthenticationStatus is ignored. Other authentication status\r
-  are record into IMAGE_EXECUTION_TABLE.\r
-\r
-  The image verification policy is:\r
-    If the image is signed,\r
-      At least one valid signature or at least one hash value of the image must match a record\r
-      in the security database "db", and no valid signature nor any hash value of the image may\r
-      be reflected in the security database "dbx".\r
-    Otherwise, the image is not signed,\r
-      The SHA256 hash value of the image must match a record in the security database "db", and\r
-      not be reflected in the security data base "dbx".\r
-\r
-  Caution: This function may receive untrusted input.\r
-  PE/COFF image is external input, so this function will validate its data structure\r
-  within this image buffer before use.\r
-\r
-  @param[in]    AuthenticationStatus\r
-                           This is the authentication status returned from the security\r
-                           measurement services for the input file.\r
-  @param[in]    File       This is a pointer to the device path of the file that is\r
-                           being dispatched. This will optionally be used for logging.\r
-  @param[in]    FileBuffer File buffer matches the input file device path.\r
-  @param[in]    FileSize   Size of File buffer matches the input file device path.\r
-  @param[in]    BootPolicy A boot policy that was used to call LoadImage() UEFI service.\r
-\r
-  @retval EFI_SUCCESS            The authenticate info is sucessfully stored for the file \r
-                                 specified by DevicePath and non-NULL FileBuffer \r
-  @retval EFI_ACCESS_DENIED      The file specified by File and FileBuffer did not\r
-                                 authenticate, and the platform policy dictates that the DXE\r
-                                 Foundation many not use File.\r
-\r
-**/\r
-EFI_STATUS\r
-EFIAPI\r
-ImageVerificationInAuditMode (\r
-  IN  UINT32                           AuthenticationStatus,\r
-  IN  CONST EFI_DEVICE_PATH_PROTOCOL   *File,\r
-  IN  VOID                             *FileBuffer,\r
-  IN  UINTN                            FileSize,\r
-  IN  BOOLEAN                          BootPolicy\r
-  )\r
-{\r
-  EFI_STATUS                           Status;\r
-  UINT16                               Magic;\r
-  EFI_IMAGE_DOS_HEADER                 *DosHdr;\r
-  EFI_SIGNATURE_LIST                   *SignatureList;\r
-  EFI_IMAGE_EXECUTION_ACTION           Action;\r
-  WIN_CERTIFICATE                      *WinCertificate;\r
-  UINT32                               Policy;\r
-  PE_COFF_LOADER_IMAGE_CONTEXT         ImageContext;\r
-  UINT32                               NumberOfRvaAndSizes;\r
-  WIN_CERTIFICATE_EFI_PKCS             *PkcsCertData;\r
-  WIN_CERTIFICATE_UEFI_GUID            *WinCertUefiGuid;\r
-  UINT8                                *AuthData;\r
-  UINTN                                AuthDataSize;\r
-  EFI_IMAGE_DATA_DIRECTORY             *SecDataDir;\r
-  UINT32                               OffSet;\r
-  CHAR16                               *FilePathStr;\r
-  UINTN                                SignatureListSize;\r
-\r
-  SignatureList     = NULL;\r
-  WinCertificate    = NULL;\r
-  SecDataDir        = NULL;\r
-  PkcsCertData      = NULL;\r
-  FilePathStr       = NULL;\r
-  Action            = EFI_IMAGE_EXECUTION_AUTH_SIG_FAILED | EFI_IMAGE_EXECUTION_INITIALIZED;\r
-  Status            = EFI_ACCESS_DENIED;\r
-\r
-\r
-  //\r
-  // Check the image type and get policy setting.\r
-  //\r
-  switch (GetImageType (File)) {\r
-\r
-  case IMAGE_FROM_FV:\r
-    Policy = ALWAYS_EXECUTE;\r
-    break;\r
-\r
-  case IMAGE_FROM_OPTION_ROM:\r
-    Policy = PcdGet32 (PcdOptionRomImageVerificationPolicy);\r
-    break;\r
-\r
-  case IMAGE_FROM_REMOVABLE_MEDIA:\r
-    Policy = PcdGet32 (PcdRemovableMediaImageVerificationPolicy);\r
-    break;\r
-\r
-  case IMAGE_FROM_FIXED_MEDIA:\r
-    Policy = PcdGet32 (PcdFixedMediaImageVerificationPolicy);\r
-    break;\r
-\r
-  default:\r
-    Policy = DENY_EXECUTE_ON_SECURITY_VIOLATION;\r
-    break;\r
-  }\r
-\r
-  //\r
-  // If policy is always/never execute, return directly.\r
-  //\r
-  if (Policy == ALWAYS_EXECUTE) {\r
-    return EFI_SUCCESS;\r
-  }\r
-\r
-  //\r
-  // Get Image Device Path Str\r
-  //\r
-  FilePathStr = ConvertDevicePathToText (File, FALSE, TRUE);\r
-\r
-  //\r
-  // Authentication failed because of (unspecified) firmware security policy\r
-  //\r
-  if (Policy == NEVER_EXECUTE) {\r
-    //\r
-    // No signature, record FilePath/FilePathStr only\r
-    //\r
-    AddImageExeInfo (EFI_IMAGE_EXECUTION_POLICY_FAILED | EFI_IMAGE_EXECUTION_INITIALIZED, FilePathStr, File, NULL, 0);\r
-    goto END;\r
-  }\r
-\r
-  //\r
-  // The policy QUERY_USER_ON_SECURITY_VIOLATION and ALLOW_EXECUTE_ON_SECURITY_VIOLATION\r
-  // violates the UEFI spec and has been removed.\r
-  //\r
-  ASSERT (Policy != QUERY_USER_ON_SECURITY_VIOLATION && Policy != ALLOW_EXECUTE_ON_SECURITY_VIOLATION);\r
-  if (Policy == QUERY_USER_ON_SECURITY_VIOLATION || Policy == ALLOW_EXECUTE_ON_SECURITY_VIOLATION) {\r
-    CpuDeadLoop ();\r
-  }\r
-\r
-  //\r
-  // Read the Dos header.\r
-  //\r
-  if (FileBuffer == NULL) {\r
-    Status = EFI_INVALID_PARAMETER;\r
-    goto END;\r
-  }\r
-\r
-  mImageBase  = (UINT8 *) FileBuffer;\r
-  mImageSize  = FileSize;\r
-\r
-  ZeroMem (&ImageContext, sizeof (ImageContext));\r
-  ImageContext.Handle    = (VOID *) FileBuffer;\r
-  ImageContext.ImageRead = (PE_COFF_LOADER_READ_FILE) DxeImageVerificationLibImageRead;\r
-\r
-  //\r
-  // Get information about the image being loaded\r
-  //\r
-  Status = PeCoffLoaderGetImageInfo (&ImageContext);\r
-  if (EFI_ERROR (Status)) {\r
-    //\r
-    // The information can't be got from the invalid PeImage\r
-    //\r
-    goto END;\r
-  }\r
-\r
-\r
-  DosHdr = (EFI_IMAGE_DOS_HEADER *) mImageBase;\r
-  if (DosHdr->e_magic == EFI_IMAGE_DOS_SIGNATURE) {\r
-    //\r
-    // DOS image header is present,\r
-    // so read the PE header after the DOS image header.\r
-    //\r
-    mPeCoffHeaderOffset = DosHdr->e_lfanew;\r
-  } else {\r
-    mPeCoffHeaderOffset = 0;\r
-  }\r
-\r
-  //\r
-  // Check PE/COFF image.\r
-  //\r
-  mNtHeader.Pe32 = (EFI_IMAGE_NT_HEADERS32 *) (mImageBase + mPeCoffHeaderOffset);\r
-  if (mNtHeader.Pe32->Signature != EFI_IMAGE_NT_SIGNATURE) {\r
-    //\r
-    // It is not a valid Pe/Coff file.\r
-    //\r
-    Status = EFI_ACCESS_DENIED;\r
-    goto END;\r
-  }\r
-\r
-  if (mNtHeader.Pe32->FileHeader.Machine == IMAGE_FILE_MACHINE_IA64 && mNtHeader.Pe32->OptionalHeader.Magic == EFI_IMAGE_NT_OPTIONAL_HDR32_MAGIC) {\r
-    //\r
-    // NOTE: Some versions of Linux ELILO for Itanium have an incorrect magic value\r
-    //       in the PE/COFF Header. If the MachineType is Itanium(IA64) and the\r
-    //       Magic value in the OptionalHeader is EFI_IMAGE_NT_OPTIONAL_HDR32_MAGIC\r
-    //       then override the magic value to EFI_IMAGE_NT_OPTIONAL_HDR64_MAGIC\r
-    //\r
-    Magic = EFI_IMAGE_NT_OPTIONAL_HDR64_MAGIC;\r
-  } else {\r
-    //\r
-    // Get the magic value from the PE/COFF Optional Header\r
-    //\r
-    Magic = mNtHeader.Pe32->OptionalHeader.Magic;\r
-  }\r
-\r
-  if (Magic == EFI_IMAGE_NT_OPTIONAL_HDR32_MAGIC) {\r
-    //\r
-    // Use PE32 offset.\r
-    //\r
-    NumberOfRvaAndSizes = mNtHeader.Pe32->OptionalHeader.NumberOfRvaAndSizes;\r
-    if (NumberOfRvaAndSizes > EFI_IMAGE_DIRECTORY_ENTRY_SECURITY) {\r
-      SecDataDir = (EFI_IMAGE_DATA_DIRECTORY *) &mNtHeader.Pe32->OptionalHeader.DataDirectory[EFI_IMAGE_DIRECTORY_ENTRY_SECURITY];\r
-    }\r
-  } else {\r
-    //\r
-    // Use PE32+ offset.\r
-    //\r
-    NumberOfRvaAndSizes = mNtHeader.Pe32Plus->OptionalHeader.NumberOfRvaAndSizes;\r
-    if (NumberOfRvaAndSizes > EFI_IMAGE_DIRECTORY_ENTRY_SECURITY) {\r
-      SecDataDir = (EFI_IMAGE_DATA_DIRECTORY *) &mNtHeader.Pe32Plus->OptionalHeader.DataDirectory[EFI_IMAGE_DIRECTORY_ENTRY_SECURITY];\r
-    }\r
-  }\r
-\r
-  //\r
-  // Start Image Validation.\r
-  //\r
-  if (SecDataDir == NULL || SecDataDir->Size == 0) {\r
-    //\r
-    // This image is not signed. The SHA256 hash value of the image must match a record in the security database "db",\r
-    // and not be reflected in the security data base "dbx".\r
-    //\r
-    if (!HashPeImage (HASHALG_SHA256)) {\r
-      Status = EFI_ACCESS_DENIED;\r
-      goto END;\r
-    }\r
-\r
-    //\r
-    // Image Hash is in forbidden database (DBX).\r
-    //\r
-    if (!IsSignatureFoundInDatabase (EFI_IMAGE_SECURITY_DATABASE1, mImageDigest, &mCertType, mImageDigestSize)) {\r
-      //\r
-      // Image Hash is in allowed database (DB).\r
-      //\r
-      if (IsSignatureFoundInDatabase (EFI_IMAGE_SECURITY_DATABASE, mImageDigest, &mCertType, mImageDigestSize)) {\r
-        Action = EFI_IMAGE_EXECUTION_AUTH_SIG_PASSED | EFI_IMAGE_EXECUTION_INITIALIZED;\r
-      }\r
-    }\r
-\r
-    //\r
-    // Add HASH digest for image without signature\r
-    //\r
-    Status = CreateSignatureList(mImageDigest, mImageDigestSize, &mCertType, &SignatureList, &SignatureListSize);\r
-    if (!EFI_ERROR(Status)) {\r
-      AddImageExeInfo (Action, FilePathStr, File, SignatureList, SignatureListSize);\r
-      FreePool (SignatureList);\r
-    }\r
-    goto END;\r
-  }\r
-\r
-  //\r
-  // Verify the signature of the image, multiple signatures are allowed as per PE/COFF Section 4.7\r
-  // "Attribute Certificate Table".\r
-  // The first certificate starts at offset (SecDataDir->VirtualAddress) from the start of the file.\r
-  //\r
-  for (OffSet = SecDataDir->VirtualAddress;\r
-       OffSet < (SecDataDir->VirtualAddress + SecDataDir->Size);\r
-       OffSet += (WinCertificate->dwLength + ALIGN_SIZE (WinCertificate->dwLength))) {\r
-    WinCertificate = (WIN_CERTIFICATE *) (mImageBase + OffSet);\r
-    if ((SecDataDir->VirtualAddress + SecDataDir->Size - OffSet) <= sizeof (WIN_CERTIFICATE) ||\r
-        (SecDataDir->VirtualAddress + SecDataDir->Size - OffSet) < WinCertificate->dwLength) {\r
-      break;\r
-    }\r
-\r
-    //\r
-    // Verify the image's Authenticode signature, only DER-encoded PKCS#7 signed data is supported.\r
-    //\r
-    if (WinCertificate->wCertificateType == WIN_CERT_TYPE_PKCS_SIGNED_DATA) {\r
-      //\r
-      // The certificate is formatted as WIN_CERTIFICATE_EFI_PKCS which is described in the\r
-      // Authenticode specification.\r
-      //\r
-      PkcsCertData = (WIN_CERTIFICATE_EFI_PKCS *) WinCertificate;\r
-      if (PkcsCertData->Hdr.dwLength <= sizeof (PkcsCertData->Hdr)) {\r
-        break;\r
-      }\r
-      AuthData   = PkcsCertData->CertData;\r
-      AuthDataSize = PkcsCertData->Hdr.dwLength - sizeof(PkcsCertData->Hdr);\r
-    } else if (WinCertificate->wCertificateType == WIN_CERT_TYPE_EFI_GUID) {\r
-      //\r
-      // The certificate is formatted as WIN_CERTIFICATE_UEFI_GUID which is described in UEFI Spec.\r
-      //\r
-      WinCertUefiGuid = (WIN_CERTIFICATE_UEFI_GUID *) WinCertificate;\r
-      if (WinCertUefiGuid->Hdr.dwLength <= OFFSET_OF(WIN_CERTIFICATE_UEFI_GUID, CertData)) {\r
-        break;\r
-      }\r
-      if (!CompareGuid (&WinCertUefiGuid->CertType, &gEfiCertPkcs7Guid)) {\r
-        continue;\r
-      }\r
-      AuthData = WinCertUefiGuid->CertData;\r
-      AuthDataSize = WinCertUefiGuid->Hdr.dwLength - OFFSET_OF(WIN_CERTIFICATE_UEFI_GUID, CertData);\r
-    } else {\r
-      if (WinCertificate->dwLength < sizeof (WIN_CERTIFICATE)) {\r
-        break;\r
-      }\r
-      continue;\r
-    }\r
-\r
-    Status = HashPeImageByType (AuthData, AuthDataSize);\r
-    if (EFI_ERROR (Status)) {\r
-      continue;\r
-    }\r
-\r
-    Action = EFI_IMAGE_EXECUTION_AUTH_SIG_FAILED | EFI_IMAGE_EXECUTION_INITIALIZED;\r
-\r
-    //\r
-    // Check the digital signature against the revoked certificate in forbidden database (dbx).\r
-    // Check the digital signature against the valid certificate in allowed database (db).\r
-    //\r
-    if (!IsForbiddenByDbx (AuthData, AuthDataSize, TRUE, FilePathStr, File)) {\r
-      IsAllowedByDb (AuthData, AuthDataSize, TRUE, FilePathStr, File);\r
-    }\r
-\r
-    //\r
-    // Check the image's hash value.\r
-    //\r
-    if (!IsSignatureFoundInDatabase (EFI_IMAGE_SECURITY_DATABASE1, mImageDigest, &mCertType, mImageDigestSize)) {\r
-      if (IsSignatureFoundInDatabase (EFI_IMAGE_SECURITY_DATABASE, mImageDigest, &mCertType, mImageDigestSize)) {\r
-        Action = EFI_IMAGE_EXECUTION_AUTH_SIG_PASSED | EFI_IMAGE_EXECUTION_INITIALIZED; \r
-      }\r
-    }\r
-\r
-    //\r
-    // Add HASH digest for image with signature\r
-    //\r
-    Status = CreateSignatureList(mImageDigest, mImageDigestSize, &mCertType, &SignatureList, &SignatureListSize);\r
-\r
-    if (!EFI_ERROR(Status)) {\r
-      AddImageExeInfo (Action, FilePathStr, File, SignatureList, SignatureListSize);\r
-      FreePool (SignatureList);\r
-    } else {\r
-      goto END;\r
-    }\r
-  }\r
-\r
-\r
-  if (OffSet != (SecDataDir->VirtualAddress + SecDataDir->Size)) {\r
-    //\r
-    // The Size in Certificate Table or the attribute certicate table is corrupted.\r
-    //\r
-    Status = EFI_ACCESS_DENIED;\r
-  } else {\r
-    Status = EFI_SUCCESS;\r
-  }\r
-\r
-END:\r
-\r
-  if (FilePathStr != NULL) {\r
-    FreePool(FilePathStr);\r
-    FilePathStr = NULL;\r
-  }\r
-\r
-  return Status;\r
-}\r
-\r
 /**\r
   Provide verification service for signed images, which include both signature validation\r
   and platform policy control. For signature types, both UEFI WIN_CERTIFICATE_UEFI_GUID and\r
@@ -2132,7 +1568,6 @@ DxeImageVerificationHandler (
   )\r
 {\r
   EFI_STATUS                           Status;\r
-  UINT16                               Magic;\r
   EFI_IMAGE_DOS_HEADER                 *DosHdr;\r
   EFI_STATUS                           VerifyStatus;\r
   EFI_SIGNATURE_LIST                   *SignatureList;\r
@@ -2141,9 +1576,7 @@ DxeImageVerificationHandler (
   EFI_IMAGE_EXECUTION_ACTION           Action;\r
   WIN_CERTIFICATE                      *WinCertificate;\r
   UINT32                               Policy;\r
-  UINT8                                *VarData;\r
-  UINT8                                SecureBoot;\r
-  UINT8                                AuditMode;\r
+  UINT8                                *SecureBoot;\r
   PE_COFF_LOADER_IMAGE_CONTEXT         ImageContext;\r
   UINT32                               NumberOfRvaAndSizes;\r
   WIN_CERTIFICATE_EFI_PKCS             *PkcsCertData;\r
@@ -2163,19 +1596,6 @@ DxeImageVerificationHandler (
   Status            = EFI_ACCESS_DENIED;\r
   VerifyStatus      = EFI_ACCESS_DENIED;\r
 \r
-  GetEfiGlobalVariable2 (EFI_AUDIT_MODE_NAME, (VOID**)&VarData, NULL);\r
-  //\r
-  // Skip verification if AuditMode variable doesn't exist. AuditMode should always exist\r
-  //\r
-  if (VarData == NULL) {\r
-    return EFI_SUCCESS;\r
-  }\r
-  AuditMode = *VarData;\r
-  FreePool(VarData);\r
-\r
-  if (AuditMode == AUDIT_MODE_ENABLE) {\r
-    return ImageVerificationInAuditMode(AuthenticationStatus, File, FileBuffer, FileSize, BootPolicy);\r
-  }\r
 \r
   //\r
   // Check the image type and get policy setting.\r
@@ -2220,22 +1640,22 @@ DxeImageVerificationHandler (
     CpuDeadLoop ();\r
   }\r
 \r
-  GetEfiGlobalVariable2 (EFI_SECURE_BOOT_MODE_NAME, (VOID**)&VarData, NULL);\r
+  GetEfiGlobalVariable2 (EFI_SECURE_BOOT_MODE_NAME, (VOID**)&SecureBoot, NULL);\r
   //\r
   // Skip verification if SecureBoot variable doesn't exist.\r
   //\r
-  if (VarData == NULL) {\r
+  if (SecureBoot == NULL) {\r
     return EFI_SUCCESS;\r
   }\r
-  SecureBoot = *VarData;\r
-  FreePool(VarData);\r
 \r
   //\r
   // Skip verification if SecureBoot is disabled but not AuditMode\r
   //\r
-  if (SecureBoot == SECURE_BOOT_MODE_DISABLE) {\r
+  if (*SecureBoot == SECURE_BOOT_MODE_DISABLE) {\r
+    FreePool (SecureBoot);\r
     return EFI_SUCCESS;\r
   }\r
+  FreePool (SecureBoot);\r
 \r
   //\r
   // Read the Dos header.\r
@@ -2259,6 +1679,7 @@ DxeImageVerificationHandler (
     //\r
     // The information can't be got from the invalid PeImage\r
     //\r
+    DEBUG ((DEBUG_INFO, "DxeImageVerificationLib: PeImage invalid. Cannot retrieve image information.\n"));\r
     goto Done;\r
   }\r
 \r
@@ -2282,25 +1703,11 @@ DxeImageVerificationHandler (
     //\r
     // It is not a valid Pe/Coff file.\r
     //\r
+    DEBUG ((DEBUG_INFO, "DxeImageVerificationLib: Not a valid PE/COFF image.\n"));\r
     goto Done;\r
   }\r
 \r
-  if (mNtHeader.Pe32->FileHeader.Machine == IMAGE_FILE_MACHINE_IA64 && mNtHeader.Pe32->OptionalHeader.Magic == EFI_IMAGE_NT_OPTIONAL_HDR32_MAGIC) {\r
-    //\r
-    // NOTE: Some versions of Linux ELILO for Itanium have an incorrect magic value\r
-    //       in the PE/COFF Header. If the MachineType is Itanium(IA64) and the\r
-    //       Magic value in the OptionalHeader is EFI_IMAGE_NT_OPTIONAL_HDR32_MAGIC\r
-    //       then override the magic value to EFI_IMAGE_NT_OPTIONAL_HDR64_MAGIC\r
-    //\r
-    Magic = EFI_IMAGE_NT_OPTIONAL_HDR64_MAGIC;\r
-  } else {\r
-    //\r
-    // Get the magic value from the PE/COFF Optional Header\r
-    //\r
-    Magic = mNtHeader.Pe32->OptionalHeader.Magic;\r
-  }\r
-\r
-  if (Magic == EFI_IMAGE_NT_OPTIONAL_HDR32_MAGIC) {\r
+  if (mNtHeader.Pe32->OptionalHeader.Magic == EFI_IMAGE_NT_OPTIONAL_HDR32_MAGIC) {\r
     //\r
     // Use PE32 offset.\r
     //\r
@@ -2327,6 +1734,7 @@ DxeImageVerificationHandler (
     // and not be reflected in the security data base "dbx".\r
     //\r
     if (!HashPeImage (HASHALG_SHA256)) {\r
+      DEBUG ((DEBUG_INFO, "DxeImageVerificationLib: Failed to hash this image using %s.\n", mHashTypeStr));\r
       goto Done;\r
     }\r
 \r
@@ -2334,6 +1742,7 @@ DxeImageVerificationHandler (
       //\r
       // Image Hash is in forbidden database (DBX).\r
       //\r
+      DEBUG ((DEBUG_INFO, "DxeImageVerificationLib: Image is not signed and %s hash of image is forbidden by DBX.\n", mHashTypeStr));\r
       goto Done;\r
     }\r
 \r
@@ -2347,6 +1756,7 @@ DxeImageVerificationHandler (
     //\r
     // Image Hash is not found in both forbidden and allowed database.\r
     //\r
+    DEBUG ((DEBUG_INFO, "DxeImageVerificationLib: Image is not signed and %s hash of image is not found in DB/DBX.\n", mHashTypeStr));\r
     goto Done;\r
   }\r
 \r
@@ -2406,7 +1816,7 @@ DxeImageVerificationHandler (
     //\r
     // Check the digital signature against the revoked certificate in forbidden database (dbx).\r
     //\r
-    if (IsForbiddenByDbx (AuthData, AuthDataSize, FALSE, NULL, NULL)) {\r
+    if (IsForbiddenByDbx (AuthData, AuthDataSize)) {\r
       Action = EFI_IMAGE_EXECUTION_AUTH_SIG_FAILED;\r
       VerifyStatus = EFI_ACCESS_DENIED;\r
       break;\r
@@ -2416,7 +1826,7 @@ DxeImageVerificationHandler (
     // Check the digital signature against the valid certificate in allowed database (db).\r
     //\r
     if (EFI_ERROR (VerifyStatus)) {\r
-      if (IsAllowedByDb (AuthData, AuthDataSize, FALSE, NULL, NULL)) {\r
+      if (IsAllowedByDb (AuthData, AuthDataSize)) {\r
         VerifyStatus = EFI_SUCCESS;\r
       }\r
     }\r
@@ -2426,11 +1836,14 @@ DxeImageVerificationHandler (
     //\r
     if (IsSignatureFoundInDatabase (EFI_IMAGE_SECURITY_DATABASE1, mImageDigest, &mCertType, mImageDigestSize)) {\r
       Action = EFI_IMAGE_EXECUTION_AUTH_SIG_FOUND;\r
+      DEBUG ((DEBUG_INFO, "DxeImageVerificationLib: Image is signed but %s hash of image is found in DBX.\n", mHashTypeStr));\r
       VerifyStatus = EFI_ACCESS_DENIED;\r
       break;\r
     } else if (EFI_ERROR (VerifyStatus)) {\r
       if (IsSignatureFoundInDatabase (EFI_IMAGE_SECURITY_DATABASE, mImageDigest, &mCertType, mImageDigestSize)) {\r
         VerifyStatus = EFI_SUCCESS;\r
+      } else {\r
+        DEBUG ((DEBUG_INFO, "DxeImageVerificationLib: Image is signed but signature is not allowed by DB and %s hash of image is not found in DB/DBX.\n", mHashTypeStr));\r
       }\r
     }\r
   }\r