diff options
author | Hao Wu <hao.a.wu@intel.com> | 2017-05-22 15:03:24 +0800 |
---|---|---|
committer | Hao Wu <hao.a.wu@intel.com> | 2017-06-01 08:46:40 +0800 |
commit | 213da0b540891ba614753673ce3ed54940434497 (patch) | |
tree | 5e5e5229efd8704123421b29abe277ad73548776 /BaseTools/Source/Python/CommonDataClass/CommonClass.py | |
parent | b1d4b9651e1143245eccd99af0e2940635c2058f (diff) | |
download | edk2-213da0b540891ba614753673ce3ed54940434497.tar.gz edk2-213da0b540891ba614753673ce3ed54940434497.tar.bz2 edk2-213da0b540891ba614753673ce3ed54940434497.zip |
MdeModulePkg/PrintLib: Avoid reading content beyond the format string
https://bugzilla.tianocore.org/show_bug.cgi?id=567
In functions DxePrintLibPrint2ProtocolVaListToBaseList() and
InternalPrintLibSPrintMarker(), when processing ASCII format strings, if
the format string walker pointer 'Format' is pointing at the end of the
format string (i.e. '\0'), the following expression:
*(Format + 1)
will read an undefined value.
Though this value won't affect the functionality, since it will be masked
by variable 'FormatMask':
(*(Format + 1) << 8)) & FormatMask
(FormatMask is 0xff for ASCII format string)
This commit adds additional logic to avoid reading undefined content.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Hao Wu <hao.a.wu@intel.com>
Reviewed-by: Liming Gao <liming.gao@intel.com>
Diffstat (limited to 'BaseTools/Source/Python/CommonDataClass/CommonClass.py')
0 files changed, 0 insertions, 0 deletions