fix: Initialize linked list at entry

Correct issue where linked list is not intiialized and if Finished is called before TestInit crash will occur. Signed-off-by: Jeff Brasen <[email protected]> Reviewed-by: Ashish Singhal <[email protected]>
NVIDIA · Sep 6, 2023 · 3bc2c62 · 3bc2c62
1 parent 925f0cd
commit 3bc2c62
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/Silicon/NVIDIA/Drivers/GenericMpMemoryTestDxe/GenericMpMemoryTestDxe.c b/Silicon/NVIDIA/Drivers/GenericMpMemoryTestDxe/GenericMpMemoryTestDxe.c
@@ -215,7 +215,6 @@ ConstructMemoryTestRanges (
   MEMORY_TEST_RANGE       *MemoryTestRange;
   UINTN                   Offset;
 
-  InitializeListHead (&Private->MemoryTestList);
   NonTestedRangeNode = GetFirstNode (&Private->NonTestedMemList);
 
   while (NonTestedRangeNode != &Private->NonTestedMemList) {
@@ -804,9 +803,10 @@ GenericMemoryTestEntryPoint (
   }
 
   //
-  // Initialize several internal link list
+  // Initialize several internal link lists
   //
   InitializeListHead (&mGenericMemoryTestPrivate.NonTestedMemList);
+  InitializeListHead (&mGenericMemoryTestPrivate.MemoryTestList);
 
   //
   // Install the protocol