On some AARCH64 Platforms, after all the tests are completed there was a synchronous exception and it was found to be NULL pointer access issue.
Add additional NULL Pointer checks to resolve the exception. Signed-off-by: Sakar Arora <[email protected]> Reported-by: Chandni Cherukuri <[email protected]> --- .../TestInfrastructure/SCT/Framework/Data/SkippedCase.c | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/uefi-sct/SctPkg/TestInfrastructure/SCT/Framework/Data/SkippedCase.c b/uefi-sct/SctPkg/TestInfrastructure/SCT/Framework/Data/SkippedCase.c index b0cb1e9..c4a16c0 100644 --- a/uefi-sct/SctPkg/TestInfrastructure/SCT/Framework/Data/SkippedCase.c +++ b/uefi-sct/SctPkg/TestInfrastructure/SCT/Framework/Data/SkippedCase.c @@ -99,7 +99,10 @@ Returns: EFI_STATUS Status; UINT32 Order = 0; - if (ExecuteInfo == NULL) { + if ( (ExecuteInfo == NULL) || + (ExecuteInfo->TestCase == NULL) || + (ExecuteInfo->Category == NULL) || + (ExecuteInfo->SkippedCase == NULL) ) { return EFI_INVALID_PARAMETER; } @@ -302,10 +305,16 @@ Returns: // // Check parameters // - if ((DevicePath == NULL) || (FileName == NULL) || (ExecuteInfo == NULL)) { + if ( (DevicePath == NULL) || (FileName == NULL) ) { return EFI_INVALID_PARAMETER; } + if ( (ExecuteInfo == NULL) || + (ExecuteInfo->TestCase == NULL) || + (ExecuteInfo->Category == NULL) || + (ExecuteInfo->SkippedCase == NULL) ) { + return EFI_INVALID_PARAMETER; + } // // Debug information // -- 2.7.4 _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel
