Fix debugger collectible thread statics access (#21175)

The CordbClass::GetStaticFieldValue2 was missing support for accessing
thread statics in collectible classes. Fortunately the fix was simple,
we can use the same code path as for non-collectible thread statics.

1 files changed, 16 insertions, 13 deletions

diff --git a/src/debug/di/rsclass.cpp b/src/debug/di/rsclass.cpp
index 987f001f80..bfd02c75ec 100644
--- a/src/debug/di/rsclass.cpp
+++ b/src/debug/di/rsclass.cpp
@@ -240,25 +240,28 @@ HRESULT CordbClass::GetStaticFieldValue2(CordbModule * pModule,
     CORDB_ADDRESS pRmtStaticValue = NULL;
     CordbProcess * pProcess = pModule->GetProcess();
 
-    if (pFieldData->m_fFldIsCollectibleStatic)
+    if (!pFieldData->m_fFldIsTLS)
     {
-        EX_TRY
+        if (pFieldData->m_fFldIsCollectibleStatic)
         {
-            pRmtStaticValue = pProcess->GetDAC()->GetCollectibleTypeStaticAddress(pFieldData->m_vmFieldDesc, 
-                                                                                  pModule->GetAppDomain()->GetADToken());
+            EX_TRY
+            {
+                pRmtStaticValue = pProcess->GetDAC()->GetCollectibleTypeStaticAddress(pFieldData->m_vmFieldDesc, 
+                                                                                      pModule->GetAppDomain()->GetADToken());
+            }
+            EX_CATCH_HRESULT(hr);
+            if(FAILED(hr))
+            {
+                return hr;
+            }
         }
-        EX_CATCH_HRESULT(hr);
-        if(FAILED(hr)) 
+        else
         {
-            return hr;
+            // Statics never move, so we always address them using their absolute address.
+            _ASSERTE(pFieldData->OkToGetOrSetStaticAddress());
+            pRmtStaticValue = pFieldData->GetStaticAddress();
         }
     }
-    else if (!pFieldData->m_fFldIsTLS)
-    {
-        // Statics never move, so we always address them using their absolute address.
-        _ASSERTE(pFieldData->OkToGetOrSetStaticAddress());
-        pRmtStaticValue = pFieldData->GetStaticAddress();
-    }
     else
     {
         // We've got a thread local static