diff options
| author | Ard Biesheuvel <ard.biesheuvel@arm.com> | 2020-06-09 10:16:05 +0200 |
|---|---|---|
| committer | mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> | 2020-06-16 16:15:00 +0000 |
| commit | bd7c73ba7706d82a974799aac39e48e05dccd169 (patch) | |
| tree | 2cd9c665202e7bc22f7a8ea8f905e8583eb9bf77 /StandaloneMmPkg/Core | |
| parent | a2433243fbe471c250d7eddc2c7da325d91265fd (diff) | |
| download | edk2-bd7c73ba7706d82a974799aac39e48e05dccd169.tar.gz edk2-bd7c73ba7706d82a974799aac39e48e05dccd169.tar.bz2 edk2-bd7c73ba7706d82a974799aac39e48e05dccd169.zip | |
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>
Diffstat (limited to 'StandaloneMmPkg/Core')
| -rw-r--r-- | StandaloneMmPkg/Core/Dispatcher.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/StandaloneMmPkg/Core/Dispatcher.c b/StandaloneMmPkg/Core/Dispatcher.c index 2f795d01dc..c99e6c04c8 100644 --- a/StandaloneMmPkg/Core/Dispatcher.c +++ b/StandaloneMmPkg/Core/Dispatcher.c @@ -816,7 +816,7 @@ FvIsBeingProcessed ( {
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);
|