Fix

Author: jkotas

src/coreclr/nativeaot/System.Private.CoreLib/src/System/Threading/Thread.NativeAot.Windows.cs

@@ -15,7 +15,7 @@ namespace System.Threading
     public sealed partial class Thread
     {
         [ThreadStatic]
-        private static ApartmentType t_apartmentType;
+        private static ApartmentState t_apartmentState;
 
         [ThreadStatic]
         private static ComState t_comState;
@@ -334,14 +334,15 @@ public ApartmentState GetApartmentState()
                 return _initialApartmentState;
             }
 
-            switch (GetCurrentApartmentType())
+            switch (GetCurrentApartmentState())
             {
-                case ApartmentType.STA:
+                case ApartmentState.STA:
                     return ApartmentState.STA;
-                case ApartmentType.MTA:
+                case ApartmentState.MTA:
                     return ApartmentState.MTA;
                 default:
-                    return ApartmentState.Unknown;
+                    // If COM is uninitialized on the current thread, it is assumed to be implicit MTA.
+                    return ApartmentState.MTA;
             }
         }
 
@@ -379,9 +380,9 @@ private bool SetApartmentStateUnchecked(ApartmentState state, bool throwOnError)
                 }
 
                 // Clear the cache and check whether new state matches the desired state
-                t_apartmentType = ApartmentType.Unknown;
+                t_apartmentState = ApartmentState.Unknown;
 
-                retState = GetApartmentState();
+                retState = GetCurrentApartmentState();
             }
 
             if (retState != state)
@@ -527,49 +528,53 @@ internal static void CheckForPendingInterrupt()
         }
 
         internal static bool ReentrantWaitsEnabled =>
-            GetCurrentApartmentType() == ApartmentType.STA;
+            GetCurrentApartmentState() == ApartmentState.STA;
 
-        internal static ApartmentType GetCurrentApartmentType()
+        internal static ApartmentState GetCurrentApartmentState()
         {
-            ApartmentType currentThreadType = t_apartmentType;
-            if (currentThreadType != ApartmentType.Unknown)
+            ApartmentState currentThreadType = t_apartmentState;
+            if (currentThreadType != ApartmentState.Unknown)
                 return currentThreadType;
 
             Interop.APTTYPE aptType;
             Interop.APTTYPEQUALIFIER aptTypeQualifier;
             int result = Interop.Ole32.CoGetApartmentType(out aptType, out aptTypeQualifier);
 
-            ApartmentType type = ApartmentType.Unknown;
+            ApartmentState type = ApartmentState.Unknown;
 
             switch (result)
             {
                 case HResults.CO_E_NOTINITIALIZED:
-                    type = ApartmentType.None;
+                    Debug.Fail("COM is not initialized");
+                    type = ApartmentState.Unknown;
                     break;
 
                 case HResults.S_OK:
                     switch (aptType)
                     {
                         case Interop.APTTYPE.APTTYPE_STA:
                         case Interop.APTTYPE.APTTYPE_MAINSTA:
-                            type = ApartmentType.STA;
+                            type = ApartmentState.STA;
                             break;
 
                         case Interop.APTTYPE.APTTYPE_MTA:
-                            type = ApartmentType.MTA;
+                            type = ApartmentState.MTA;
                             break;
 
                         case Interop.APTTYPE.APTTYPE_NA:
                             switch (aptTypeQualifier)
                             {
                                 case Interop.APTTYPEQUALIFIER.APTTYPEQUALIFIER_NA_ON_MTA:
+                                    type = ApartmentState.MTA;
+                                    break;
+
                                 case Interop.APTTYPEQUALIFIER.APTTYPEQUALIFIER_NA_ON_IMPLICIT_MTA:
-                                    type = ApartmentType.MTA;
+                                    type = ApartmentState.Unknown;
                                     break;
 
                                 case Interop.APTTYPEQUALIFIER.APTTYPEQUALIFIER_NA_ON_STA:
                                 case Interop.APTTYPEQUALIFIER.APTTYPEQUALIFIER_NA_ON_MAINSTA:
-                                    type = ApartmentType.STA;
+                                    type = ApartmentState.STA;
                                     break;
 
                                 default:
@@ -585,19 +590,11 @@ internal static ApartmentType GetCurrentApartmentType()
                     break;
             }
 
-            if (type != ApartmentType.Unknown)
-                t_apartmentType = type;
+            if (type != ApartmentState.Unknown)
+                t_apartmentState = type;
             return type;
         }
 
-        internal enum ApartmentType : byte
-        {
-            Unknown = 0,
-            None,
-            STA,
-            MTA
-        }
-
         [Flags]
         internal enum ComState : byte
         {