Hi Michael, Sean, Bret, Could help review the patch?
Thanks > -----Original Message----- > From: [email protected] [mailto:[email protected]] On Behalf Of > Zhang, Shenglei > Sent: Wednesday, March 4, 2020 3:59 PM > To: [email protected]; [email protected] > Subject: Re: [edk2-devel] [PATCH] UnitTestFrameworkPkg: Invalid index may > be used. > > Reviewed-by: Shenglei Zhang <[email protected]> > > > -----Original Message----- > > From: [email protected] [mailto:[email protected]] On Behalf Of > > GuoMinJ > > Sent: Thursday, February 20, 2020 9:42 AM > > To: [email protected] > > Cc: GuoMinJ <[email protected]> > > Subject: [edk2-devel] [PATCH] UnitTestFrameworkPkg: Invalid index may > > be used. > > > > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2535 > > > > The UINT_TEST_STATUS and FAILURE_TYPE have used 0 as status, so use 0 > > as unknown is confused, remove it from array enumeration but keep it > > location in the array. > > > > Signed-off-by: GuoMinJ <[email protected]> > > --- > > .../Library/UnitTestResultReportLib/UnitTestResultReportLib.c | 4 > > ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > diff --git > > > a/UnitTestFrameworkPkg/Library/UnitTestResultReportLib/UnitTestResultR > > eportLib.c > > > b/UnitTestFrameworkPkg/Library/UnitTestResultReportLib/UnitTestResultR > > eportLib.c > > index 687a04f55d..eba68e330c 100644 > > --- > > > a/UnitTestFrameworkPkg/Library/UnitTestResultReportLib/UnitTestResultR > > eportLib.c > > +++ > > > b/UnitTestFrameworkPkg/Library/UnitTestResultReportLib/UnitTestResultR > > eportLib.c > > @@ -65,7 +65,7 @@ GetStringForUnitTestStatus ( { > > UINTN Index; > > > > - for (Index = 0; Index < ARRAY_SIZE (mStatusStrings); Index++) { > > + for (Index = 0; Index < ARRAY_SIZE (mStatusStrings) - 1; Index++) { > > if (mStatusStrings[Index].Status == Status) { > > // > > // Return string from matching entry @@ -87,7 +87,7 @@ > > GetStringForFailureType ( { > > UINTN Index; > > > > - for (Index = 0; Index < ARRAY_SIZE (mFailureTypeStrings); Index++) > > { > > + for (Index = 0; Index < ARRAY_SIZE (mFailureTypeStrings) - 1; > > + Index++) { > > if (mFailureTypeStrings[Index].Type == Failure) { > > // > > // Return string from matching entry > > -- > > 2.17.1 > > > > > > > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#56090): https://edk2.groups.io/g/devel/message/56090 Mute This Topic: https://groups.io/mt/71411719/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
