From 33985e3b52ee2e4b5064331134d983e55a7da904 Mon Sep 17 00:00:00 2001 From: Chao Zhang Date: Mon, 10 Nov 2014 05:01:15 +0000 Subject: [PATCH] =?utf8?q?=20=20=20Fix=20execution=20status=20&=20DEBUG=20?= =?utf8?q?message=20level=20mismatch.=20EFI=5FD=5FERROR=20is=20used=20only?= =?utf8?q?=20when=20failure/case=20can=E2=80=99t=20be=20resolved=20by=20co?= =?utf8?q?de.?= MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Chao Zhang Reviewed-by: Dong Guo git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@16320 6f19259b-4bc3-4df7-8a09-765794883524 --- .../DxeDeferImageLoadLib.c | 4 +- .../DxeImageVerificationLib/Measurement.c | 4 +- .../DxeTpm2MeasureBootLib.c | 6 +- .../DxeTrEEPhysicalPresenceLib.c | 8 +-- .../HashLibBaseCryptoRouterPei.c | 4 +- .../Tpm2DeviceLibRouterPei.c | 2 +- SecurityPkg/Tcg/TrEEConfig/TpmDetection.c | 6 +- SecurityPkg/Tcg/TrEEConfig/TrEEConfigPeim.c | 8 +-- SecurityPkg/Tcg/TrEEDxe/TrEEDxe.c | 56 +++++++++---------- .../EsalVariableDxeSal/Variable.c | 4 +- .../RuntimeDxe/Measurement.c | 8 +-- .../SecureBootConfigImpl.c | 4 +- 12 files changed, 57 insertions(+), 57 deletions(-) diff --git a/SecurityPkg/Library/DxeDeferImageLoadLib/DxeDeferImageLoadLib.c b/SecurityPkg/Library/DxeDeferImageLoadLib/DxeDeferImageLoadLib.c index 1729051694..142b5469af 100644 --- a/SecurityPkg/Library/DxeDeferImageLoadLib/DxeDeferImageLoadLib.c +++ b/SecurityPkg/Library/DxeDeferImageLoadLib/DxeDeferImageLoadLib.c @@ -1,7 +1,7 @@ /** @file Implement defer image load services for user identification in UEFI2.2. -Copyright (c) 2009 - 2013, Intel Corporation. All rights reserved.
+Copyright (c) 2009 - 2014, Intel Corporation. All rights reserved.
This program and the accompanying materials are licensed and made available under the terms and conditions of the BSD License which accompanies this distribution. The full text of the license may be found at @@ -853,7 +853,7 @@ DxeDeferImageLoadHandler ( return EFI_SUCCESS; } - DEBUG ((EFI_D_ERROR, "[Security] No user identified, the image is deferred to load!\n")); + DEBUG ((EFI_D_INFO, "[Security] No user identified, the image is deferred to load!\n")); PutDefferedImageInfo (File, FileBuffer, FileSize); // diff --git a/SecurityPkg/Library/DxeImageVerificationLib/Measurement.c b/SecurityPkg/Library/DxeImageVerificationLib/Measurement.c index 2213423c33..1dc29895f3 100644 --- a/SecurityPkg/Library/DxeImageVerificationLib/Measurement.c +++ b/SecurityPkg/Library/DxeImageVerificationLib/Measurement.c @@ -1,7 +1,7 @@ /** @file Measure TrEE required variable. -Copyright (c) 2013, Intel Corporation. All rights reserved.
+Copyright (c) 2013 - 2014, Intel Corporation. All rights reserved.
This program and the accompanying materials are licensed and made available under the terms and conditions of the BSD License which accompanies this distribution. The full text of the license may be found at @@ -312,7 +312,7 @@ SecureBootHook ( Data, DataSize ); - DEBUG ((EFI_D_ERROR, "MeasureBootPolicyVariable - %r\n", Status)); + DEBUG ((EFI_D_INFO, "MeasureBootPolicyVariable - %r\n", Status)); if (!EFI_ERROR (Status)) { AddDataMeasured (VariableName, VendorGuid, Data, DataSize); diff --git a/SecurityPkg/Library/DxeTpm2MeasureBootLib/DxeTpm2MeasureBootLib.c b/SecurityPkg/Library/DxeTpm2MeasureBootLib/DxeTpm2MeasureBootLib.c index 8482031e43..6b6a377b2d 100644 --- a/SecurityPkg/Library/DxeTpm2MeasureBootLib/DxeTpm2MeasureBootLib.c +++ b/SecurityPkg/Library/DxeTpm2MeasureBootLib/DxeTpm2MeasureBootLib.c @@ -509,7 +509,7 @@ DxeTpm2MeasureBootHandler ( // Measure GPT disk. // Status = TrEEMeasureGptTable (TreeProtocol, Handle); - DEBUG ((EFI_D_ERROR, "DxeTpm2MeasureBootHandler - TrEEMeasureGptTable - %r\n", Status)); + DEBUG ((EFI_D_INFO, "DxeTpm2MeasureBootHandler - TrEEMeasureGptTable - %r\n", Status)); if (!EFI_ERROR (Status)) { // // GPT disk check done. @@ -653,7 +653,7 @@ DxeTpm2MeasureBootHandler ( ImageContext.ImageType, DevicePathNode ); - DEBUG ((EFI_D_ERROR, "DxeTpm2MeasureBootHandler - TrEEMeasurePeImage - %r\n", Status)); + DEBUG ((EFI_D_INFO, "DxeTpm2MeasureBootHandler - TrEEMeasurePeImage - %r\n", Status)); } // @@ -664,7 +664,7 @@ Finish: FreePool (OrigDevicePathNode); } - DEBUG ((EFI_D_ERROR, "DxeTpm2MeasureBootHandler - %r\n", Status)); + DEBUG ((EFI_D_INFO, "DxeTpm2MeasureBootHandler - %r\n", Status)); return Status; } diff --git a/SecurityPkg/Library/DxeTrEEPhysicalPresenceLib/DxeTrEEPhysicalPresenceLib.c b/SecurityPkg/Library/DxeTrEEPhysicalPresenceLib/DxeTrEEPhysicalPresenceLib.c index ea9a29c0f0..89ce436b7c 100644 --- a/SecurityPkg/Library/DxeTrEEPhysicalPresenceLib/DxeTrEEPhysicalPresenceLib.c +++ b/SecurityPkg/Library/DxeTrEEPhysicalPresenceLib/DxeTrEEPhysicalPresenceLib.c @@ -91,15 +91,15 @@ TpmCommandClear ( CopyMem (LocalAuthSession.hmac.buffer, PlatformAuth->buffer, PlatformAuth->size); } - DEBUG ((EFI_D_ERROR, "Tpm2ClearControl ... \n")); + DEBUG ((EFI_D_INFO, "Tpm2ClearControl ... \n")); Status = Tpm2ClearControl (TPM_RH_PLATFORM, AuthSession, NO); - DEBUG ((EFI_D_ERROR, "Tpm2ClearControl - %r\n", Status)); + DEBUG ((EFI_D_INFO, "Tpm2ClearControl - %r\n", Status)); if (EFI_ERROR (Status)) { goto Done; } - DEBUG ((EFI_D_ERROR, "Tpm2Clear ... \n")); + DEBUG ((EFI_D_INFO, "Tpm2Clear ... \n")); Status = Tpm2Clear (TPM_RH_PLATFORM, AuthSession); - DEBUG ((EFI_D_ERROR, "Tpm2Clear - %r\n", Status)); + DEBUG ((EFI_D_INFO, "Tpm2Clear - %r\n", Status)); Done: ZeroMem (&LocalAuthSession.hmac, sizeof(LocalAuthSession.hmac)); diff --git a/SecurityPkg/Library/HashLibBaseCryptoRouter/HashLibBaseCryptoRouterPei.c b/SecurityPkg/Library/HashLibBaseCryptoRouter/HashLibBaseCryptoRouterPei.c index 1e7eb2d77e..5bc8d01ad5 100644 --- a/SecurityPkg/Library/HashLibBaseCryptoRouter/HashLibBaseCryptoRouterPei.c +++ b/SecurityPkg/Library/HashLibBaseCryptoRouter/HashLibBaseCryptoRouterPei.c @@ -3,7 +3,7 @@ hash handler registerd, such as SHA1, SHA256. Platform can use PcdTpm2HashMask to mask some hash engines. -Copyright (c) 2013, Intel Corporation. All rights reserved.
+Copyright (c) 2013 - 2014, Intel Corporation. All rights reserved.
This program and the accompanying materials are licensed and made available under the terms and conditions of the BSD License which accompanies this distribution. The full text of the license may be found at @@ -275,7 +275,7 @@ RegisterHashInterfaceLib ( // // In PEI phase, there will be shadow driver dispatched again. // - DEBUG ((EFI_D_ERROR, "RegisterHashInterfaceLib - Override\n")); + DEBUG ((EFI_D_INFO, "RegisterHashInterfaceLib - Override\n")); CopyMem (&HashInterfaceHob->HashInterface[Index], HashInterface, sizeof(*HashInterface)); return EFI_SUCCESS; } diff --git a/SecurityPkg/Library/Tpm2DeviceLibRouter/Tpm2DeviceLibRouterPei.c b/SecurityPkg/Library/Tpm2DeviceLibRouter/Tpm2DeviceLibRouterPei.c index a13bf80d62..8e75140123 100644 --- a/SecurityPkg/Library/Tpm2DeviceLibRouter/Tpm2DeviceLibRouterPei.c +++ b/SecurityPkg/Library/Tpm2DeviceLibRouter/Tpm2DeviceLibRouterPei.c @@ -129,7 +129,7 @@ Tpm2RegisterTpm2DeviceLib ( // // In PEI phase, there will be shadow driver dispatched again. // - DEBUG ((EFI_D_ERROR, "Tpm2RegisterTpm2DeviceLib - Override\n")); + DEBUG ((EFI_D_INFO, "Tpm2RegisterTpm2DeviceLib - Override\n")); CopyMem (Tpm2DeviceInterface, Tpm2Device, sizeof(*Tpm2Device)); return EFI_SUCCESS; } else { diff --git a/SecurityPkg/Tcg/TrEEConfig/TpmDetection.c b/SecurityPkg/Tcg/TrEEConfig/TpmDetection.c index 516f7c92a3..3706c2c967 100644 --- a/SecurityPkg/Tcg/TrEEConfig/TpmDetection.c +++ b/SecurityPkg/Tcg/TrEEConfig/TpmDetection.c @@ -46,7 +46,7 @@ IsDtpmPresent ( DEBUG ((EFI_D_ERROR, "DetectTpmDevice: Dtpm not present\n")); return FALSE; } else { - DEBUG ((EFI_D_ERROR, "DetectTpmDevice: Dtpm present\n")); + DEBUG ((EFI_D_INFO, "DetectTpmDevice: Dtpm present\n")); return TRUE; } } @@ -76,7 +76,7 @@ DetectTpmDevice ( // In S3, we rely on normal boot Detection, because we save to ReadOnly Variable in normal boot. // if (BootMode == BOOT_ON_S3_RESUME) { - DEBUG ((EFI_D_ERROR, "DetectTpmDevice: S3 mode\n")); + DEBUG ((EFI_D_INFO, "DetectTpmDevice: S3 mode\n")); Status = PeiServicesLocatePpi (&gEfiPeiReadOnlyVariable2PpiGuid, 0, NULL, (VOID **) &VariablePpi); ASSERT_EFI_ERROR (Status); @@ -99,7 +99,7 @@ DetectTpmDevice ( } } - DEBUG ((EFI_D_ERROR, "DetectTpmDevice:\n")); + DEBUG ((EFI_D_INFO, "DetectTpmDevice:\n")); if (!IsDtpmPresent ()) { // dTPM not available return TPM_DEVICE_NULL; diff --git a/SecurityPkg/Tcg/TrEEConfig/TrEEConfigPeim.c b/SecurityPkg/Tcg/TrEEConfig/TrEEConfigPeim.c index 9c1e917ab4..579d096be5 100644 --- a/SecurityPkg/Tcg/TrEEConfig/TrEEConfigPeim.c +++ b/SecurityPkg/Tcg/TrEEConfig/TrEEConfigPeim.c @@ -1,7 +1,7 @@ /** @file The module entry point for TrEE configuration module. -Copyright (c) 2013, Intel Corporation. All rights reserved.
+Copyright (c) 2013 - 2014, Intel Corporation. All rights reserved.
This program and the accompanying materials are licensed and made available under the terms and conditions of the BSD License which accompanies this distribution. The full text of the license may be found at @@ -101,11 +101,11 @@ TrEEConfigPeimEntryPoint ( // // Although we have SetupVariable info, we still need detect TPM device manually. // - DEBUG ((EFI_D_ERROR, "TrEEConfiguration.TpmDevice from Setup: %x\n", TrEEConfiguration.TpmDevice)); + DEBUG ((EFI_D_INFO, "TrEEConfiguration.TpmDevice from Setup: %x\n", TrEEConfiguration.TpmDevice)); if (PcdGetBool (PcdTpmAutoDetection)) { TpmDevice = DetectTpmDevice (TrEEConfiguration.TpmDevice); - DEBUG ((EFI_D_ERROR, "TpmDevice final: %x\n", TpmDevice)); + DEBUG ((EFI_D_INFO, "TpmDevice final: %x\n", TpmDevice)); if (TpmDevice != TPM_DEVICE_NULL) { TrEEConfiguration.TpmDevice = TpmDevice; } @@ -125,7 +125,7 @@ TrEEConfigPeimEntryPoint ( if (TpmDevice == mTpmInstanceId[Index].TpmDevice) { Size = sizeof(mTpmInstanceId[Index].TpmInstanceGuid); PcdSetPtr (PcdTpmInstanceGuid, &Size, &mTpmInstanceId[Index].TpmInstanceGuid); - DEBUG ((EFI_D_ERROR, "TpmDevice PCD: %g\n", &mTpmInstanceId[Index].TpmInstanceGuid)); + DEBUG ((EFI_D_INFO, "TpmDevice PCD: %g\n", &mTpmInstanceId[Index].TpmInstanceGuid)); break; } } diff --git a/SecurityPkg/Tcg/TrEEDxe/TrEEDxe.c b/SecurityPkg/Tcg/TrEEDxe/TrEEDxe.c index 8db0fa0df0..2f159e0c6b 100644 --- a/SecurityPkg/Tcg/TrEEDxe/TrEEDxe.c +++ b/SecurityPkg/Tcg/TrEEDxe/TrEEDxe.c @@ -364,7 +364,7 @@ TreeGetCapability ( IN OUT TREE_BOOT_SERVICE_CAPABILITY *ProtocolCapability ) { - DEBUG ((EFI_D_ERROR, "TreeGetCapability ...\n")); + DEBUG ((EFI_D_INFO, "TreeGetCapability ...\n")); if ((This == NULL) || (ProtocolCapability == NULL)) { return EFI_INVALID_PARAMETER; @@ -376,7 +376,7 @@ TreeGetCapability ( } CopyMem (ProtocolCapability, &mTcgDxeData.BsCap, mTcgDxeData.BsCap.Size); - DEBUG ((EFI_D_ERROR, "TreeGetCapability - %r\n", EFI_SUCCESS)); + DEBUG ((EFI_D_INFO, "TreeGetCapability - %r\n", EFI_SUCCESS)); return EFI_SUCCESS; } @@ -451,7 +451,7 @@ TreeGetEventLog ( { UINTN Index; - DEBUG ((EFI_D_ERROR, "TreeGetEventLog ...\n")); + DEBUG ((EFI_D_INFO, "TreeGetEventLog ...\n")); if (This == NULL) { return EFI_INVALID_PARAMETER; @@ -482,7 +482,7 @@ TreeGetEventLog ( if (EventLogLocation != NULL) { *EventLogLocation = mTcgDxeData.EventLogAreaStruct[Index].Lasa; - DEBUG ((EFI_D_ERROR, "TreeGetEventLog (EventLogLocation - %x)\n", *EventLogLocation)); + DEBUG ((EFI_D_INFO, "TreeGetEventLog (EventLogLocation - %x)\n", *EventLogLocation)); } if (EventLogLastEntry != NULL) { @@ -491,15 +491,15 @@ TreeGetEventLog ( } else { *EventLogLastEntry = (EFI_PHYSICAL_ADDRESS)(UINTN)mTcgDxeData.EventLogAreaStruct[Index].LastEvent; } - DEBUG ((EFI_D_ERROR, "TreeGetEventLog (EventLogLastEntry - %x)\n", *EventLogLastEntry)); + DEBUG ((EFI_D_INFO, "TreeGetEventLog (EventLogLastEntry - %x)\n", *EventLogLastEntry)); } if (EventLogTruncated != NULL) { *EventLogTruncated = mTcgDxeData.EventLogAreaStruct[Index].EventLogTruncated; - DEBUG ((EFI_D_ERROR, "TreeGetEventLog (EventLogTruncated - %x)\n", *EventLogTruncated)); + DEBUG ((EFI_D_INFO, "TreeGetEventLog (EventLogTruncated - %x)\n", *EventLogTruncated)); } - DEBUG ((EFI_D_ERROR, "TreeGetEventLog - %r\n", EFI_SUCCESS)); + DEBUG ((EFI_D_INFO, "TreeGetEventLog - %r\n", EFI_SUCCESS)); // Dump Event Log for debug purpose if ((EventLogLocation != NULL) && (EventLogLastEntry != NULL)) { @@ -860,7 +860,7 @@ TreeHashLogExtendEvent ( TCG_PCR_EVENT_HDR NewEventHdr; TPML_DIGEST_VALUES DigestList; - DEBUG ((EFI_D_ERROR, "TreeHashLogExtendEvent ...\n")); + DEBUG ((EFI_D_INFO, "TreeHashLogExtendEvent ...\n")); if ((This == NULL) || (DataToHash == 0) || (Event == NULL)) { return EFI_INVALID_PARAMETER; @@ -902,7 +902,7 @@ TreeHashLogExtendEvent ( Event->Event ); } - DEBUG ((EFI_D_ERROR, "TreeHashLogExtendEvent - %r\n", Status)); + DEBUG ((EFI_D_INFO, "TreeHashLogExtendEvent - %r\n", Status)); return Status; } @@ -932,7 +932,7 @@ TreeSubmitCommand ( { EFI_STATUS Status; - DEBUG ((EFI_D_ERROR, "TreeSubmitCommand ...\n")); + DEBUG ((EFI_D_INFO, "TreeSubmitCommand ...\n")); if ((This == NULL) || (InputParameterBlockSize == 0) || (InputParameterBlock == NULL) || @@ -957,7 +957,7 @@ TreeSubmitCommand ( &OutputParameterBlockSize, OutputParameterBlock ); - DEBUG ((EFI_D_ERROR, "TreeSubmitCommand - %r\n", Status)); + DEBUG ((EFI_D_INFO, "TreeSubmitCommand - %r\n", Status)); return Status; } @@ -1178,7 +1178,7 @@ MeasureSeparatorEvent ( TCG_PCR_EVENT_HDR TcgEvent; UINT32 EventData; - DEBUG ((EFI_D_ERROR, "MeasureSeparatorEvent Pcr - %x\n", PCRIndex)); + DEBUG ((EFI_D_INFO, "MeasureSeparatorEvent Pcr - %x\n", PCRIndex)); EventData = 0; TcgEvent.PCRIndex = PCRIndex; @@ -1223,8 +1223,8 @@ MeasureVariable ( UINTN VarNameLength; EFI_VARIABLE_DATA_TREE *VarLog; - DEBUG ((EFI_D_ERROR, "TrEEDxe: MeasureVariable (Pcr - %x, EventType - %x, ", (UINTN)PCRIndex, (UINTN)EventType)); - DEBUG ((EFI_D_ERROR, "VariableName - %s, VendorGuid - %g)\n", VarName, VendorGuid)); + DEBUG ((EFI_D_INFO, "TrEEDxe: MeasureVariable (Pcr - %x, EventType - %x, ", (UINTN)PCRIndex, (UINTN)EventType)); + DEBUG ((EFI_D_INFO, "VariableName - %s, VendorGuid - %g)\n", VarName, VendorGuid)); VarNameLength = StrLen (VarName); TcgEvent.PCRIndex = PCRIndex; @@ -1553,11 +1553,11 @@ MeasureSecureBootPolicy ( if (PcdGetBool (PcdFirmwareDebuggerInitialized)) { Status = MeasureLaunchOfFirmwareDebugger (); - DEBUG ((EFI_D_ERROR, "MeasureLaunchOfFirmwareDebugger - %r\n", Status)); + DEBUG ((EFI_D_INFO, "MeasureLaunchOfFirmwareDebugger - %r\n", Status)); } Status = MeasureAllSecureVariables (); - DEBUG ((EFI_D_ERROR, "MeasureAllSecureVariables - %r\n", Status)); + DEBUG ((EFI_D_INFO, "MeasureAllSecureVariables - %r\n", Status)); // // We need measure Separator(7) here, because this event must be between SecureBootPolicy (Configure) @@ -1566,7 +1566,7 @@ MeasureSecureBootPolicy ( // the Authority measurement happen before ReadToBoot event. // Status = MeasureSeparatorEvent (7); - DEBUG ((EFI_D_ERROR, "MeasureSeparatorEvent - %r\n", Status)); + DEBUG ((EFI_D_INFO, "MeasureSeparatorEvent - %r\n", Status)); return ; } @@ -1862,14 +1862,14 @@ DriverEntry ( // // Fill information // - DEBUG ((EFI_D_ERROR, "TrEE.ProtocolVersion - %02x.%02x\n", mTcgDxeData.BsCap.ProtocolVersion.Major, mTcgDxeData.BsCap.ProtocolVersion.Minor)); - DEBUG ((EFI_D_ERROR, "TrEE.StructureVersion - %02x.%02x\n", mTcgDxeData.BsCap.StructureVersion.Major, mTcgDxeData.BsCap.StructureVersion.Minor)); + DEBUG ((EFI_D_INFO, "TrEE.ProtocolVersion - %02x.%02x\n", mTcgDxeData.BsCap.ProtocolVersion.Major, mTcgDxeData.BsCap.ProtocolVersion.Minor)); + DEBUG ((EFI_D_INFO, "TrEE.StructureVersion - %02x.%02x\n", mTcgDxeData.BsCap.StructureVersion.Major, mTcgDxeData.BsCap.StructureVersion.Minor)); Status = Tpm2GetCapabilityManufactureID (&mTcgDxeData.BsCap.ManufacturerID); if (EFI_ERROR (Status)) { DEBUG ((EFI_D_ERROR, "Tpm2GetCapabilityManufactureID fail!\n")); } else { - DEBUG ((EFI_D_ERROR, "Tpm2GetCapabilityManufactureID - %08x\n", mTcgDxeData.BsCap.ManufacturerID)); + DEBUG ((EFI_D_INFO, "Tpm2GetCapabilityManufactureID - %08x\n", mTcgDxeData.BsCap.ManufacturerID)); } DEBUG_CODE ( @@ -1880,7 +1880,7 @@ DriverEntry ( if (EFI_ERROR (Status)) { DEBUG ((EFI_D_ERROR, "Tpm2GetCapabilityFirmwareVersion fail!\n")); } else { - DEBUG ((EFI_D_ERROR, "Tpm2GetCapabilityFirmwareVersion - %08x %08x\n", FirmwareVersion1, FirmwareVersion2)); + DEBUG ((EFI_D_INFO, "Tpm2GetCapabilityFirmwareVersion - %08x %08x\n", FirmwareVersion1, FirmwareVersion2)); } ); @@ -1890,7 +1890,7 @@ DriverEntry ( } else { mTcgDxeData.BsCap.MaxCommandSize = (UINT16)MaxCommandSize; mTcgDxeData.BsCap.MaxResponseSize = (UINT16)MaxResponseSize; - DEBUG ((EFI_D_ERROR, "Tpm2GetCapabilityMaxCommandResponseSize - %08x, %08x\n", MaxCommandSize, MaxResponseSize)); + DEBUG ((EFI_D_INFO, "Tpm2GetCapabilityMaxCommandResponseSize - %08x, %08x\n", MaxCommandSize, MaxResponseSize)); } Status = Tpm2GetCapabilityPcrs (&Pcrs); @@ -1898,10 +1898,10 @@ DriverEntry ( DEBUG ((EFI_D_ERROR, "Tpm2GetCapabilityPcrs fail!\n")); TpmHashAlgorithmBitmap = TREE_BOOT_HASH_ALG_SHA1; } else { - DEBUG ((EFI_D_ERROR, "Tpm2GetCapabilityPcrs Count - %08x\n", Pcrs.count)); + DEBUG ((EFI_D_INFO, "Tpm2GetCapabilityPcrs Count - %08x\n", Pcrs.count)); TpmHashAlgorithmBitmap = 0; for (Index = 0; Index < Pcrs.count; Index++) { - DEBUG ((EFI_D_ERROR, "hash - %x\n", Pcrs.pcrSelections[Index].hash)); + DEBUG ((EFI_D_INFO, "hash - %x\n", Pcrs.pcrSelections[Index].hash)); switch (Pcrs.pcrSelections[Index].hash) { case TPM_ALG_SHA1: TpmHashAlgorithmBitmap |= TREE_BOOT_HASH_ALG_SHA1; @@ -1921,11 +1921,11 @@ DriverEntry ( } } } - DEBUG ((EFI_D_ERROR, "TPM.HashAlgorithmBitmap - 0x%08x\n", TpmHashAlgorithmBitmap)); + DEBUG ((EFI_D_INFO, "TPM.HashAlgorithmBitmap - 0x%08x\n", TpmHashAlgorithmBitmap)); - DEBUG ((EFI_D_ERROR, "TrEE.SupportedEventLogs - 0x%08x\n", mTcgDxeData.BsCap.SupportedEventLogs)); + DEBUG ((EFI_D_INFO, "TrEE.SupportedEventLogs - 0x%08x\n", mTcgDxeData.BsCap.SupportedEventLogs)); mTcgDxeData.BsCap.HashAlgorithmBitmap = TpmHashAlgorithmBitmap; - DEBUG ((EFI_D_ERROR, "TrEE.HashAlgorithmBitmap - 0x%08x\n", mTcgDxeData.BsCap.HashAlgorithmBitmap)); + DEBUG ((EFI_D_INFO, "TrEE.HashAlgorithmBitmap - 0x%08x\n", mTcgDxeData.BsCap.HashAlgorithmBitmap)); if (mTcgDxeData.BsCap.TrEEPresentFlag) { // @@ -1982,7 +1982,7 @@ DriverEntry ( // Install TrEEProtocol // Status = InstallTrEE (); - DEBUG ((EFI_D_ERROR, "InstallTrEE - %r\n", Status)); + DEBUG ((EFI_D_INFO, "InstallTrEE - %r\n", Status)); return Status; } diff --git a/SecurityPkg/VariableAuthenticated/EsalVariableDxeSal/Variable.c b/SecurityPkg/VariableAuthenticated/EsalVariableDxeSal/Variable.c index e098c81df3..0b2775dac8 100644 --- a/SecurityPkg/VariableAuthenticated/EsalVariableDxeSal/Variable.c +++ b/SecurityPkg/VariableAuthenticated/EsalVariableDxeSal/Variable.c @@ -1,7 +1,7 @@ /** @file The implementation of Extended SAL variable services. -Copyright (c) 2009 - 2013, Intel Corporation. All rights reserved.
+Copyright (c) 2009 - 2014, Intel Corporation. All rights reserved.
This program and the accompanying materials are licensed and made available under the terms and conditions of the BSD License which accompanies this distribution. The full text of the license may be found at @@ -3244,7 +3244,7 @@ VariableCommonInitialize ( ); } else { Status = EFI_VOLUME_CORRUPTED; - DEBUG((EFI_D_INFO, "Variable Store header is corrupted\n")); + DEBUG((EFI_D_ERROR, "Variable Store header is corrupted\n")); } Done: diff --git a/SecurityPkg/VariableAuthenticated/RuntimeDxe/Measurement.c b/SecurityPkg/VariableAuthenticated/RuntimeDxe/Measurement.c index 41e9b6f5e0..1b625e7469 100644 --- a/SecurityPkg/VariableAuthenticated/RuntimeDxe/Measurement.c +++ b/SecurityPkg/VariableAuthenticated/RuntimeDxe/Measurement.c @@ -1,7 +1,7 @@ /** @file Measure TrEE required variable. -Copyright (c) 2013, Intel Corporation. All rights reserved.
+Copyright (c) 2013 - 2014, Intel Corporation. All rights reserved.
This program and the accompanying materials are licensed and made available under the terms and conditions of the BSD License which accompanies this distribution. The full text of the license may be found at @@ -118,8 +118,8 @@ MeasureVariable ( ); } - DEBUG ((EFI_D_ERROR, "AuthVariableDxe: MeasureVariable (Pcr - %x, EventType - %x, ", (UINTN)7, (UINTN)EV_EFI_VARIABLE_AUTHORITY)); - DEBUG ((EFI_D_ERROR, "VariableName - %s, VendorGuid - %g)\n", VarName, VendorGuid)); + DEBUG ((EFI_D_INFO, "AuthVariableDxe: MeasureVariable (Pcr - %x, EventType - %x, ", (UINTN)7, (UINTN)EV_EFI_VARIABLE_AUTHORITY)); + DEBUG ((EFI_D_INFO, "VariableName - %s, VendorGuid - %g)\n", VarName, VendorGuid)); Status = TpmMeasureAndLogData ( 7, @@ -245,7 +245,7 @@ SecureBootHook ( VariableData, VariableDataSize ); - DEBUG ((EFI_D_ERROR, "MeasureBootPolicyVariable - %r\n", Status)); + DEBUG ((EFI_D_INFO, "MeasureBootPolicyVariable - %r\n", Status)); if (VariableData != NULL) { FreePool (VariableData); diff --git a/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfigImpl.c b/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfigImpl.c index 8b9a8e26a3..c6356143bc 100644 --- a/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfigImpl.c +++ b/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfigImpl.c @@ -1920,7 +1920,7 @@ DeleteKeyExchangeKey ( ZeroMem (OldData, KekDataSize); while ((KekDataSize > 0) && (KekDataSize >= CertList->SignatureListSize)) { CertCount = (CertList->SignatureListSize - sizeof (EFI_SIGNATURE_LIST) - CertList->SignatureHeaderSize) / CertList->SignatureSize; - DEBUG ((DEBUG_ERROR, " CertCount = %x\n", CertCount)); + DEBUG ((DEBUG_INFO, " CertCount = %x\n", CertCount)); if (CertCount != 0) { CopyMem (OldData + Offset, CertList, CertList->SignatureListSize); Offset += CertList->SignatureListSize; @@ -2118,7 +2118,7 @@ DeleteSignature ( ZeroMem (OldData, ItemDataSize); while ((ItemDataSize > 0) && (ItemDataSize >= CertList->SignatureListSize)) { CertCount = (CertList->SignatureListSize - sizeof (EFI_SIGNATURE_LIST) - CertList->SignatureHeaderSize) / CertList->SignatureSize; - DEBUG ((DEBUG_ERROR, " CertCount = %x\n", CertCount)); + DEBUG ((DEBUG_INFO, " CertCount = %x\n", CertCount)); if (CertCount != 0) { CopyMem (OldData + Offset, (UINT8*)(CertList), CertList->SignatureListSize); Offset += CertList->SignatureListSize; -- 2.39.2