Free the alternate signal stack if this PAL's thread allocated it. (#19539)

Free the alternate signal stack if this PAL's thread allocated it.

Move Ensure/FreeSignalAlternateStack to CPalThread class.

Frees the alternate stack a little sooner during thread termination.

1 files changed, 7 insertions, 7 deletions

diff --git a/src/pal/src/init/sxs.cpp b/src/pal/src/init/sxs.cpp
index 3f323c6a6e..55645c941b 100644
--- a/src/pal/src/init/sxs.cpp
+++ b/src/pal/src/init/sxs.cpp
@@ -109,10 +109,16 @@ AllocatePalThread(CPalThread **ppThread)
     CPalThread *pThread = NULL;
     PAL_ERROR palError;
 
+    palError = CreateThreadData(&pThread);
+    if (NO_ERROR != palError)
+    {
+        goto exit;
+    }
+
 #if !HAVE_MACH_EXCEPTIONS
     // Ensure alternate stack for SIGSEGV handling. Our SIGSEGV handler is set to
     // run on an alternate stack and the stack needs to be allocated per thread.
-    if (!EnsureSignalAlternateStack())
+    if (!pThread->EnsureSignalAlternateStack())
     {
         ERROR("Cannot allocate alternate stack for SIGSEGV handler!\n");
         palError = ERROR_NOT_ENOUGH_MEMORY;
@@ -120,12 +126,6 @@ AllocatePalThread(CPalThread **ppThread)
     }
 #endif // !HAVE_MACH_EXCEPTIONS
 
-    palError = CreateThreadData(&pThread);
-    if (NO_ERROR != palError)
-    {
-        goto exit;
-    }
-
     HANDLE hThread;
     palError = CreateThreadObject(pThread, pThread, &hThread);
     if (NO_ERROR != palError)