From 6fb09da89f88000a7592171a0ce08cf1feaa0646 Mon Sep 17 00:00:00 2001 From: Chris Jones Date: Tue, 14 Dec 2021 20:37:36 +0800 Subject: [PATCH] ShellPkg: Fix incorrect PPTT FlagName dereference Bugzilla: 3770 (https://bugzilla.tianocore.org/show_bug.cgi?id=3770) The PPTT parser in AcpiView incorrectly dereferences a pointer to FlagName when trying to log an error with the PPTT cache flags, which can lead to random crashes and other errors. Also fix some spacing in the error message to ensure the message is printed cleanly. Signed-off-by: Chris Jones Reviewed-by: Rebecca Cran Reviewed-by: Sami Mujawar --- .../UefiShellAcpiViewCommandLib/Parsers/Pptt/PpttParser.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Pptt/PpttParser.c b/ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Pptt/PpttParser.c index 8d52bb5e48..5377764458 100644 --- a/ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Pptt/PpttParser.c +++ b/ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Pptt/PpttParser.c @@ -40,9 +40,9 @@ LogCacheFlagError ( IncrementErrorCount (); Print ( L"\nERROR: On Arm based systems, all cache properties must be" - L"provided in the cache type structure." - L"Missing '%s' flag.", - *FlagName + L" provided in the cache type structure." + L" Missing '%s' flag.", + FlagName ); }