SecurityPkg: Update final event log calculation.

This patch updated final event log calculation based on
TCG EFI PROTOCOL rev9, 7.1.2 "All options implementation".
The previous logic is to save event to final event log
once GetEventLog() is called. 7.1.1 shows that we should
save event log to both normal even log area and final event
log area after GetEventLog() is called.
And there is no need to record TCG12 format log to final
event log area, and normal event log area can be BootServices
memory because OS will not access it directly.

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

git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@19661 6f19259b-4bc3-4df7-8a09-765794883524

1 files changed, 84 insertions, 38 deletions

diff --git a/SecurityPkg/Tcg/Tcg2Dxe/Tcg2Dxe.c b/SecurityPkg/Tcg/Tcg2Dxe/Tcg2Dxe.c
index c4926f63ba..b1c885e407 100644
--- a/SecurityPkg/Tcg/Tcg2Dxe/Tcg2Dxe.c
+++ b/SecurityPkg/Tcg/Tcg2Dxe/Tcg2Dxe.c
@@ -817,11 +817,10 @@ TcgDxeLogEvent (
     return EFI_INVALID_PARAMETER;

   }

 

-  if (!mTcgDxeData.GetEventLogCalled[Index]) {

-    EventLogAreaStruct = &mTcgDxeData.EventLogAreaStruct[Index];

-  } else {

-    EventLogAreaStruct = &mTcgDxeData.FinalEventLogAreaStruct[Index];

-  }

+  //

+  // Record to normal event log

+  //

+  EventLogAreaStruct = &mTcgDxeData.EventLogAreaStruct[Index];

 

   if (EventLogAreaStruct->EventLogTruncated) {

     return EFI_VOLUME_FULL;

@@ -838,15 +837,50 @@ TcgDxeLogEvent (
              NewEventSize

              );

   

-  if (Status == EFI_DEVICE_ERROR) {

-    return EFI_DEVICE_ERROR;

-  } else if (Status == EFI_OUT_OF_RESOURCES) {

+  if (Status == EFI_OUT_OF_RESOURCES) {

     EventLogAreaStruct->EventLogTruncated = TRUE;

     return EFI_VOLUME_FULL;

   } else if (Status == EFI_SUCCESS) {

     EventLogAreaStruct->EventLogStarted = TRUE;

-    if (mTcgDxeData.GetEventLogCalled[Index]) {

+  }

+

+  //

+  // If GetEventLog is called, record to FinalEventsTable, too.

+  //

+  if (mTcgDxeData.GetEventLogCalled[Index]) {

+    if (mTcgDxeData.FinalEventsTable[Index] == NULL) {

+      //

+      // no need for FinalEventsTable

+      //

+      return EFI_SUCCESS;

+    }

+    EventLogAreaStruct = &mTcgDxeData.FinalEventLogAreaStruct[Index];

+

+    if (EventLogAreaStruct->EventLogTruncated) {

+      return EFI_VOLUME_FULL;

+    }

+

+    EventLogAreaStruct->LastEvent = (UINT8*)(UINTN)EventLogAreaStruct->Lasa;

+    Status = TcgCommLogEvent (

+               &EventLogAreaStruct->LastEvent,

+               &EventLogAreaStruct->EventLogSize,

+               (UINTN)EventLogAreaStruct->Laml,

+               NewEventHdr,

+               NewEventHdrSize,

+               NewEventData,

+               NewEventSize

+               );

+    if (Status == EFI_OUT_OF_RESOURCES) {

+      EventLogAreaStruct->EventLogTruncated = TRUE;

+      return EFI_VOLUME_FULL;

+    } else if (Status == EFI_SUCCESS) {

+      EventLogAreaStruct->EventLogStarted = TRUE;

+      //

+      // Increase the NumberOfEvents in FinalEventsTable

+      //

       (mTcgDxeData.FinalEventsTable[Index])->NumberOfEvents ++;

+      DEBUG ((EFI_D_INFO, "FinalEventsTable->NumberOfEvents - 0x%x\n", (mTcgDxeData.FinalEventsTable[Index])->NumberOfEvents));

+      DEBUG ((EFI_D_INFO, "  Size - 0x%x\n", (UINTN)EventLogAreaStruct->LastEvent - (UINTN)mTcgDxeData.FinalEventsTable[Index]));

     }

   }

 

@@ -1466,7 +1500,7 @@ SetupEventLog (
       Lasa = (EFI_PHYSICAL_ADDRESS) (SIZE_4GB - 1);

       Status = gBS->AllocatePages (

                       AllocateMaxAddress,

-                      EfiACPIMemoryNVS,

+                      EfiBootServicesData,

                       EFI_SIZE_TO_PAGES (PcdGet32 (PcdTcgLogAreaMinLen)),

                       &Lasa

                       );

@@ -1564,41 +1598,53 @@ SetupEventLog (
   //

   for (Index = 0; Index < sizeof(mTcg2EventInfo)/sizeof(mTcg2EventInfo[0]); Index++) {

     if ((mTcgDxeData.BsCap.SupportedEventLogs & mTcg2EventInfo[Index].LogFormat) != 0) {

-      Lasa = (EFI_PHYSICAL_ADDRESS) (SIZE_4GB - 1);

-      Status = gBS->AllocatePages (

-                      AllocateMaxAddress,

-                      EfiACPIMemoryNVS,

-                      EFI_SIZE_TO_PAGES (PcdGet32 (PcdTcg2FinalLogAreaLen)),

-                      &Lasa

-                      );

-      if (EFI_ERROR (Status)) {

-        return Status;

-      }

-      SetMem ((VOID *)(UINTN)Lasa, PcdGet32 (PcdTcg2FinalLogAreaLen), 0xFF);

+      if (mTcg2EventInfo[Index].LogFormat == EFI_TCG2_EVENT_LOG_FORMAT_TCG_2) {

+        Lasa = (EFI_PHYSICAL_ADDRESS) (SIZE_4GB - 1);

+        Status = gBS->AllocatePages (

+                        AllocateMaxAddress,

+                        EfiACPIMemoryNVS,

+                        EFI_SIZE_TO_PAGES (PcdGet32 (PcdTcg2FinalLogAreaLen)),

+                        &Lasa

+                        );

+        if (EFI_ERROR (Status)) {

+          return Status;

+        }

+        SetMem ((VOID *)(UINTN)Lasa, PcdGet32 (PcdTcg2FinalLogAreaLen), 0xFF);

 

-      //

-      // Initialize

-      //

-      mTcgDxeData.FinalEventsTable[Index] = (VOID *)(UINTN)Lasa;

-      (mTcgDxeData.FinalEventsTable[Index])->Version = EFI_TCG2_FINAL_EVENTS_TABLE_VERSION;

-      (mTcgDxeData.FinalEventsTable[Index])->NumberOfEvents = 0;

-

-      mTcgDxeData.FinalEventLogAreaStruct[Index].EventLogFormat = mTcg2EventInfo[Index].LogFormat;

-      mTcgDxeData.FinalEventLogAreaStruct[Index].Lasa = Lasa + sizeof(EFI_TCG2_FINAL_EVENTS_TABLE);

-      mTcgDxeData.FinalEventLogAreaStruct[Index].Laml = PcdGet32 (PcdTcg2FinalLogAreaLen) - sizeof(EFI_TCG2_FINAL_EVENTS_TABLE);

-      mTcgDxeData.FinalEventLogAreaStruct[Index].EventLogSize = 0;

-      mTcgDxeData.FinalEventLogAreaStruct[Index].LastEvent = (VOID *)(UINTN)mTcgDxeData.FinalEventLogAreaStruct[Index].Lasa;

-      mTcgDxeData.FinalEventLogAreaStruct[Index].EventLogStarted = FALSE;

-      mTcgDxeData.FinalEventLogAreaStruct[Index].EventLogTruncated = FALSE;

+        //

+        // Initialize

+        //

+        mTcgDxeData.FinalEventsTable[Index] = (VOID *)(UINTN)Lasa;

+        (mTcgDxeData.FinalEventsTable[Index])->Version = EFI_TCG2_FINAL_EVENTS_TABLE_VERSION;

+        (mTcgDxeData.FinalEventsTable[Index])->NumberOfEvents = 0;

+

+        mTcgDxeData.FinalEventLogAreaStruct[Index].EventLogFormat = mTcg2EventInfo[Index].LogFormat;

+        mTcgDxeData.FinalEventLogAreaStruct[Index].Lasa = Lasa + sizeof(EFI_TCG2_FINAL_EVENTS_TABLE);

+        mTcgDxeData.FinalEventLogAreaStruct[Index].Laml = PcdGet32 (PcdTcg2FinalLogAreaLen) - sizeof(EFI_TCG2_FINAL_EVENTS_TABLE);

+        mTcgDxeData.FinalEventLogAreaStruct[Index].EventLogSize = 0;

+        mTcgDxeData.FinalEventLogAreaStruct[Index].LastEvent = (VOID *)(UINTN)mTcgDxeData.FinalEventLogAreaStruct[Index].Lasa;

+        mTcgDxeData.FinalEventLogAreaStruct[Index].EventLogStarted = FALSE;

+        mTcgDxeData.FinalEventLogAreaStruct[Index].EventLogTruncated = FALSE;

 

-      if (mTcg2EventInfo[Index].LogFormat == EFI_TCG2_EVENT_LOG_FORMAT_TCG_2) {

         //

-        // Install to configuration table

+        // Install to configuration table for EFI_TCG2_EVENT_LOG_FORMAT_TCG_2 

         //

-        Status = gBS->InstallConfigurationTable (&gEfiTcg2FinalEventsTableGuid, (VOID *)mTcgDxeData.FinalEventsTable[1]);

+        Status = gBS->InstallConfigurationTable (&gEfiTcg2FinalEventsTableGuid, (VOID *)mTcgDxeData.FinalEventsTable[Index]);

         if (EFI_ERROR (Status)) {

           return Status;

         }

+      } else {

+        //

+        // No need to handle EFI_TCG2_EVENT_LOG_FORMAT_TCG_1_2

+        //

+        mTcgDxeData.FinalEventsTable[Index] = NULL;

+        mTcgDxeData.FinalEventLogAreaStruct[Index].EventLogFormat = mTcg2EventInfo[Index].LogFormat;

+        mTcgDxeData.FinalEventLogAreaStruct[Index].Lasa = 0;

+        mTcgDxeData.FinalEventLogAreaStruct[Index].Laml = 0;

+        mTcgDxeData.FinalEventLogAreaStruct[Index].EventLogSize = 0;

+        mTcgDxeData.FinalEventLogAreaStruct[Index].LastEvent = 0;

+        mTcgDxeData.FinalEventLogAreaStruct[Index].EventLogStarted = FALSE;

+        mTcgDxeData.FinalEventLogAreaStruct[Index].EventLogTruncated = FALSE;

       }

     }

   }