summaryrefslogtreecommitdiffstats
path: root/BaseTools/Source/Python/GenFds/RuleSimpleFile.py
diff options
context:
space:
mode:
authorWang, Jian J <jian.j.wang@intel.com>2018-01-02 16:20:20 +0800
committerLiming Gao <liming.gao@intel.com>2018-01-02 16:31:19 +0800
commit701e8cc29a283d6d598695592ca2469cdacddf64 (patch)
tree03b0db465853639ed1bd76c57e303f31398b9f81 /BaseTools/Source/Python/GenFds/RuleSimpleFile.py
parentcd16355bbf3267f1ad7f678e91dc60cf31f3cb29 (diff)
downloadedk2-701e8cc29a283d6d598695592ca2469cdacddf64.tar.gz
edk2-701e8cc29a283d6d598695592ca2469cdacddf64.tar.bz2
edk2-701e8cc29a283d6d598695592ca2469cdacddf64.zip
MdePkg/BasePrintLib: Fix incomplete print output
This is caused by previous patch which tried to fix string over-read, which breaks UEFI menu rendering: the following /------------------------------------------------------------------------------\ | Device Manager | \------------------------------------------------------------------------------/ is rendered as /\ | Device Manager | \/.0 2.00 GHz (the spurious digits are SMBIOS data from the home screen) The problem appears to be that the CHAR16 value of BOXDRAW_HORIZONTAL equals 0x2500, which means that testing ArgumentString[] != '\0' (which tests the low byte only) will yield FALSE and terminate the loop prematurely. Cc: Michael D Kinney <michael.d.kinney@intel.com> Cc: Liming Gao <liming.gao@intel.com> Cc: Star Zeng <star.zeng@intel.com> Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org> Reviewed-by: Liming Gao <liming.gao@intel.com>
Diffstat (limited to 'BaseTools/Source/Python/GenFds/RuleSimpleFile.py')
0 files changed, 0 insertions, 0 deletions