[edk2-test][PATCH] uefi-sct/SctPkg: Framework: Enhanced Null Pointer Checks.

[edk2-test][PATCH] uefi-sct/SctPkg: Framework: Enhanced Null Pointer Checks.

[Sakar Arora]

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 <sakar.arora@arm.com>
Reported-by: Chandni Cherukuri <Chandni.Cherukuri@arm.com>
---
 .../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