From: Marcin Wojtas <mw@semihalf.com>
Date: Fri, 1 Jun 2018 13:58:13 +0000 (+0800)
Subject: MdeModulePkg PeiCore: Check error status when processing boot FV
X-Git-Tag: edk2-stable201903~1650
X-Git-Url: https://git.proxmox.com/?p=mirror_edk2.git;a=commitdiff_plain;h=38c977c148e92e2af17c5d346d9b4b2e7a18680a;hp=c4061d18ef531147a58075f7f011a25b598d6aee

MdeModulePkg PeiCore: Check error status when processing boot FV

Until now the possible errors returned from processing
boot firmware volume were not checked, which could cause
misbehavior in further booting stages. Add relevant assert.

Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Marcin Wojtas <mw@semihalf.com>
Signed-off-by: Jan Dabros <jsd@semihalf.com>
Reviewed-by: Liming Gao <liming.gao@intel.com>
Reviewed-by: Star Zeng <star.zeng@intel.com>
---

diff --git a/MdeModulePkg/Core/Pei/FwVol/FwVol.c b/MdeModulePkg/Core/Pei/FwVol/FwVol.c
index 8009546acd..4150b338b2 100644
--- a/MdeModulePkg/Core/Pei/FwVol/FwVol.c
+++ b/MdeModulePkg/Core/Pei/FwVol/FwVol.c
@@ -495,12 +495,13 @@ PeiInitializeFv (
   //
   // Get handle of BFV
   //
-  FvPpi->ProcessVolume (
-           FvPpi, 
-           SecCoreData->BootFirmwareVolumeBase,
-           (UINTN)BfvHeader->FvLength,
-           &FvHandle
-           );
+  Status = FvPpi->ProcessVolume (
+                    FvPpi,
+                    SecCoreData->BootFirmwareVolumeBase,
+                    (UINTN)BfvHeader->FvLength,
+                    &FvHandle
+                    );
+  ASSERT_EFI_ERROR (Status);
 
   //
   // Update internal PEI_CORE_FV array.