commit bd7c73ba7706d82a974799aac39e48e05dccd169
author Ard Biesheuvel <ard.biesheuvel@arm.com> Tue Jun 09 10:16:05 2020 +0200
committer mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> Tue Jun 16 16:15:00 2020 +0000
tree 2cd9c665202e7bc22f7a8ea8f905e8583eb9bf77
parent a2433243fbe471c250d7eddc2c7da325d91265fd

StandaloneMmPkg/Core: fix bogus FV pointer in DEBUG string

FvIsBeingProcessed () emits a DEBUG print with the intent to print
the memory address of the FV that is being processed, but instead,
it prints the contents of an uninitialized stack variable.

Signed-off-by: Ard Biesheuvel <ard.biesheuvel@arm.com>
Reviewed-by: Jiewen Yao <Jiewen.yao@intel.com>
Reviewed-by: Sami Mujawar <sami.mujawar@arm.com>
Tested-by: Ilias Apalodimas <ilias.apalodimas@linaro.org>

StandaloneMmPkg/Core/Dispatcher.c

diff --git a/StandaloneMmPkg/Core/Dispatcher.c b/StandaloneMmPkg/Core/Dispatcher.c
index 2f795d0..c99e6c0 100644
--- a/StandaloneMmPkg/Core/Dispatcher.c
+++ b/StandaloneMmPkg/Core/Dispatcher.c
@@ -816,7 +816,7 @@
 {

   KNOWN_FWVOL   *KnownFwVol;

 

-  DEBUG ((DEBUG_INFO, "FvIsBeingProcessed - 0x%08x\n", KnownFwVol));

+  DEBUG ((DEBUG_INFO, "FvIsBeingProcessed - 0x%08x\n", FwVolHeader));

 

   KnownFwVol = AllocatePool (sizeof (KNOWN_FWVOL));

   ASSERT (KnownFwVol != NULL);