Wait to complete the test before releasing the agile reference. (#112387

) Otherwise we can end up with a use-after-free
dotnet · Feb 11, 2025 · 7e82ad9 · 7e82ad9
1 parent b0ad53e
commit 7e82ad9
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/src/tests/Interop/COM/ComWrappers/API/Program.cs b/src/tests/Interop/COM/ComWrappers/API/Program.cs
@@ -1084,6 +1084,7 @@ public unsafe void CrossApartmentQueryInterface_NoDeadlock()
 
                 staThread.Start();
                 mtaThread.Start();
+                testCompleted.WaitOne();
             }
             finally
             {
@@ -1092,8 +1093,6 @@ public unsafe void CrossApartmentQueryInterface_NoDeadlock()
                     Marshal.Release(agileReference);
                 }
             }
-
-            testCompleted.WaitOne();
         }
 
         [DllImport("ole32.dll")]