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 8009546acdb1a1a91bfaf472f0dc6d5ca402aab8..4150b338b211847a17cf2d2968a11b11e1e46976 100644 (file)
--- a/MdeModulePkg/Core/Pei/FwVol/FwVol.c
+++ b/MdeModulePkg/Core/Pei/FwVol/FwVol.c
@@ -495,12 +495,13 @@ PeiInitializeFv (
   //\r
   // Get handle of BFV\r
   //\r
   //\r
   // Get handle of BFV\r
   //\r

-  FvPpi->ProcessVolume (\r
-           FvPpi, \r
-           SecCoreData->BootFirmwareVolumeBase,\r
-           (UINTN)BfvHeader->FvLength,\r
-           &FvHandle\r
-           );\r
+  Status = FvPpi->ProcessVolume (\r
+                    FvPpi,\r
+                    SecCoreData->BootFirmwareVolumeBase,\r
+                    (UINTN)BfvHeader->FvLength,\r
+                    &FvHandle\r
+                    );\r
+  ASSERT_EFI_ERROR (Status);\r

 \r
   //\r
   // Update internal PEI_CORE_FV array.\r
 \r
   //\r
   // Update internal PEI_CORE_FV array.\r