[compiler-rt] [asan][win] Fix CreateThread leak (PR #126738)

[via llvm-commits]

llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-compiler-rt-sanitizer

Author: None (GkvJwa)

<details>
<summary>Changes</summary>

Fix #<!-- -->126541

Since ```t->Destroy``` cannot be called after ```start_routine```(When calling standard thread_start in crt), the func is run in advance to avoid memory leaks and remain the same as before.

---
Full diff: https://github.com/llvm/llvm-project/pull/126738.diff


1 Files Affected:

- (modified) compiler-rt/lib/asan/asan_win.cpp (+3-1) 


``````````diff
diff --git a/compiler-rt/lib/asan/asan_win.cpp b/compiler-rt/lib/asan/asan_win.cpp
index 09a13b11cff1f53..d043bb822d2cf15 100644
--- a/compiler-rt/lib/asan/asan_win.cpp
+++ b/compiler-rt/lib/asan/asan_win.cpp
@@ -143,9 +143,11 @@ static thread_return_t THREAD_CALLING_CONV asan_thread_start(void *arg) {
 
   ThreadStartParams params;
   t->GetStartData(params);
+  // The ExitThread will end the current thread, causing destroy to be unable to
+  // be called.
+  t->Destroy();
 
   auto res = (*params.start_routine)(params.arg);
-  t->Destroy();  // POSIX calls this from TSD destructor.
   return res;
 }
 

``````````

</details>


https://github.com/llvm/llvm-project/pull/126738