Fix PerformanceCounterLib cache races

src/libraries/System.Diagnostics.PerformanceCounter/src/System/Diagnostics/PerformanceCounterLib.cs

@@ -124,7 +124,8 @@ internal Hashtable CategoryTable
         {
             get
             {
-                if (_categoryTable == null)
+                Hashtable categoryTable = _categoryTable;
+                if (categoryTable == null)
                 {
                     lock (_categoryTableLock)
                     {
@@ -194,26 +195,29 @@ internal Hashtable CategoryTable
 
                             _categoryTable = tempCategoryTable;
                         }
+
+                        categoryTable = _categoryTable;
                     }
                 }
 
-                return _categoryTable;
+                return categoryTable;
             }
         }
 
         internal Hashtable HelpTable
         {
             get
             {
-                if (_helpTable == null)
+                Hashtable helpTable = _helpTable;
+                if (helpTable == null)
                 {
                     lock (_helpTableLock)
                     {
-                        _helpTable ??= GetStringTable(true);
+                        helpTable = _helpTable ??= GetStringTable(true);
                     }
                 }
 
-                return _helpTable;
+                return helpTable;
             }
         }
 
@@ -246,15 +250,16 @@ internal Hashtable NameTable
         {
             get
             {
-                if (_nameTable == null)
+                Hashtable nameTable = _nameTable;
+                if (nameTable == null)
                 {
                     lock (_nameTableLock)
                     {
-                        _nameTable ??= GetStringTable(false);
+                        nameTable = _nameTable ??= GetStringTable(false);
                     }
                 }
 
-                return _nameTable;
+                return nameTable;
             }
         }
 
@@ -329,10 +334,13 @@ internal static void CloseAllLibraries()
 
         internal static void CloseAllTables()
         {
-            if (s_libraryTable != null)
+            lock (InternalSyncObject)
             {
-                foreach (PerformanceCounterLib library in s_libraryTable.Values)
-                    library.CloseTables();
+                if (s_libraryTable != null)
+                {
+                    foreach (PerformanceCounterLib library in s_libraryTable.Values)
+                        library.CloseTables();
+                }
             }
         }