UefiCpuPkg/SecMigrationPei: Add initial PEIM (CVE-2019-11098)

REF:https://bugzilla.tianocore.org/show_bug.cgi?id=1614

Adds a PEIM that republishes structures produced in SEC. This
is done because SEC modules may not be shadowed in some platforms
due to space constraints or special alignment requirements. The
SecMigrationPei module locates interfaces that may be published in
SEC and reinstalls the interface with permanent memory addresses.

This is important if pre-memory address access is forbidden after
memory initialization and data such as a PPI descriptor, PPI GUID,
or PPI inteface reside in pre-memory.

Cc: Eric Dong <eric.dong@intel.com>
Cc: Ray Ni <ray.ni@intel.com>
Cc: Laszlo Ersek <lersek@redhat.com>
Cc: Rahul Kumar <rahul1.kumar@intel.com>
Cc: Debkumar De <debkumar.de@intel.com>
Cc: Harry Han <harry.han@intel.com>
Cc: Catharine West <catharine.west@intel.com>
Signed-off-by: Michael Kubacki <michael.a.kubacki@intel.com>
Acked-by: Laszlo Ersek <lersek@redhat.com>
Reviewed-by: Liming Gao <liming.gao@intel.com>

1 files changed, 24 insertions, 2 deletions

diff --git a/UefiCpuPkg/SecCore/SecMain.c b/UefiCpuPkg/SecCore/SecMain.c
index 5d5e7f17dc..155be49a60 100644
--- a/UefiCpuPkg/SecCore/SecMain.c
+++ b/UefiCpuPkg/SecCore/SecMain.c
@@ -370,7 +370,12 @@ SecTemporaryRamDone (
   VOID

   )

 {

-  BOOLEAN  State;

+  EFI_STATUS                    Status;

+  EFI_STATUS                    Status2;

+  UINTN                         Index;

+  BOOLEAN                       State;

+  EFI_PEI_PPI_DESCRIPTOR        *PeiPpiDescriptor;

+  REPUBLISH_SEC_PPI_PPI         *RepublishSecPpiPpi;

 

   //

   // Republish Sec Platform Information(2) PPI

@@ -378,6 +383,23 @@ SecTemporaryRamDone (
   RepublishSecPlatformInformationPpi ();

 

   //

+  // Re-install SEC PPIs using a PEIM produced service if published

+  //

+  for (Index = 0, Status = EFI_SUCCESS; Status == EFI_SUCCESS; Index++) {

+    Status = PeiServicesLocatePpi (

+               &gRepublishSecPpiPpiGuid,

+               Index,

+               &PeiPpiDescriptor,

+               (VOID **) &RepublishSecPpiPpi

+               );

+    if (!EFI_ERROR (Status)) {

+      DEBUG ((DEBUG_INFO, "Calling RepublishSecPpi instance %d.\n", Index));

+      Status2 = RepublishSecPpiPpi->RepublishSecPpis ();

+      ASSERT_EFI_ERROR (Status2);

+    }

+  }

+

+  //

   // Migrate DebugAgentContext.

   //

   InitializeDebugAgent (DEBUG_AGENT_INIT_POSTMEM_SEC, NULL, NULL);

@@ -385,7 +407,7 @@ SecTemporaryRamDone (
   //

   // Disable interrupts and save current interrupt state

   //

-  State = SaveAndDisableInterrupts();

+  State = SaveAndDisableInterrupts ();

 

   //

   // Disable Temporary RAM after Stack and Heap have been migrated at this point.