ShellPkg/HandleParsingLib: Correct format specifier for LoadedImage

The format specifier for the LoadOptions field of the LoadedImage protocol
is "%s". However, the data in LoadOptions is often generic binary data. A
format specifier of "%x" is more appropriate for this field.

Using "dh -v" with format specifier "%s" on BIOS images based on EDK II
source before commit 891d844 can cause a crash.

Cc: Ruiyu Ni <ruiyu.ni@intel.com>
Cc: Jaben Carsey <jaben.carsey@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Jeff Westfahl <jeff.westfahl@ni.com>
Reviewed-by: Ruiyu Ni <ruiyu.ni@intel.com>
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com>
This commit is contained in:
Jeff Westfahl 2017-03-15 05:02:01 +08:00 committed by Ruiyu Ni
parent 56dee83a55
commit 7c6f0d14c3
1 changed files with 1 additions and 1 deletions

View File

@ -354,7 +354,7 @@
" DeviceHandle..: %%H%x%%N\r\n" " DeviceHandle..: %%H%x%%N\r\n"
" FilePath......: %%H%x%%N\r\n" " FilePath......: %%H%x%%N\r\n"
" OptionsSize...: %%H%x%%N\r\n" " OptionsSize...: %%H%x%%N\r\n"
" LoadOptions...: %%H%s%%N\r\n" " LoadOptions...: %%H%x%%N\r\n"
" ImageBase.....: %%H%x%%N\r\n" " ImageBase.....: %%H%x%%N\r\n"
" ImageSize.....: %%H%Lx%%N\r\n" " ImageSize.....: %%H%Lx%%N\r\n"
" CodeType......: %%H%s%%N\r\n" " CodeType......: %%H%s%%N\r\n"