Reviewed-by: jiewen....@intel.com > -----Original Message----- > From: Wu, Hao A > Sent: Monday, February 13, 2017 6:43 PM > To: edk2-devel@lists.01.org > Cc: Wu, Hao A <hao.a...@intel.com>; Yao, Jiewen <jiewen....@intel.com>; > Gao, Liming <liming....@intel.com>; Kinney, Michael D > <michael.d.kin...@intel.com> > Subject: [PATCH] MdePkg/BaseLib: Refine logic for (Ascii)StrnLenS to handle > MaxSize = 0 > > https://bugzilla.tianocore.org/show_bug.cgi?id=378 > > Cc: Jiewen Yao <jiewen....@intel.com> > Cc: Liming Gao <liming....@intel.com> > Cc: Michael Kinney <michael.d.kin...@intel.com> > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Hao Wu <hao.a...@intel.com> > --- > MdePkg/Library/BaseLib/SafeString.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/MdePkg/Library/BaseLib/SafeString.c > b/MdePkg/Library/BaseLib/SafeString.c > index 315059e..79c79e0 100644 > --- a/MdePkg/Library/BaseLib/SafeString.c > +++ b/MdePkg/Library/BaseLib/SafeString.c > @@ -128,9 +128,9 @@ StrnLenS ( > ASSERT (((UINTN) String & BIT0) == 0); > > // > - // If String is a null pointer, then the StrnLenS function returns zero. > + // If String is a null pointer or MaxSize is 0, then the StrnLenS function > returns > zero. > // > - if (String == NULL) { > + if ((String == NULL) || (MaxSize == 0)) { > return 0; > } > > @@ -1097,9 +1097,9 @@ AsciiStrnLenS ( > UINTN Length; > > // > - // If String is a null pointer, then the AsciiStrnLenS function returns > zero. > + // If String is a null pointer or MaxSize is 0, then the AsciiStrnLenS > function > returns zero. > // > - if (String == NULL) { > + if ((String == NULL) || (MaxSize == 0)) { > return 0; > } > > -- > 1.9.5.msysgit.0
_______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel
