Re: [edk2] [PATCH] MdePkg/BasePrintLib: Fix incorrect Precision position calculation

2017-12-27 Thread Wang, Jian J
..@intel.com>; Zeng, Star <star.z...@intel.com> > Subject: RE: [edk2] [PATCH] MdePkg/BasePrintLib: Fix incorrect Precision > position calculation > > Jian: > MdePkg/Library/BasePrintLib/PrintLibInternal.c line 1171 has the similar > issue. > Could you fix it a

Re: [edk2] [PATCH] MdePkg/BasePrintLib: Fix incorrect Precision position calculation

2017-12-27 Thread Gao, Liming
; > > > From: edk2-devel [mailto:edk2-devel- > > > boun...@lists.01.org] On Behalf Of Jian J > > > > Wang > > > > Sent: Monday, December 25, 2017 10:09 AM > > > > To: edk2-devel@lists.01.org > > > > Cc: Kinney, Michael D <michael.d.kin.

Re: [edk2] [PATCH] MdePkg/BasePrintLib: Fix incorrect Precision position calculation

2017-12-27 Thread Wang, Jian J
om>; edk2-devel@lists.01.org; Kinney, > Michael D <michael.d.kin...@intel.com> > Cc: Yao, Jiewen <jiewen@intel.com>; Zeng, Star <star.z...@intel.com>; > Gao, Liming <liming@intel.com> > Subject: RE: [edk2] [PATCH] MdePkg/BasePrintLib: Fix incorrect P

Re: [edk2] [PATCH] MdePkg/BasePrintLib: Fix incorrect Precision position calculation

2017-12-27 Thread Kinney, Michael D
Michael D <michael.d.kin...@intel.com>; Yao, > Jiewen <jiewen@intel.com>; Zeng, Star > <star.z...@intel.com>; Gao, Liming <liming....@intel.com> > Subject: RE: [edk2] [PATCH] MdePkg/BasePrintLib: Fix > incorrect Precision position calculation > > Mike

[edk2] [PATCH] MdePkg/BasePrintLib: Fix incorrect Precision position calculation

2017-12-24 Thread Jian J Wang
Due to the a potential hole in the stop condition of for-loop, the two continuous access to ArgumentString (index, index+1) inside the loop might cause the string ending character ('\0') to be read. Cc: Michael D Kinney Cc: Liming Gao Cc: Jiewen