Ensure GetPESectionBlock uses its cache (#90369)

dotnet · Aug 11, 2023 · e8c46c2 · e8c46c2
1 parent d339063
commit e8c46c2
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/...libraries/System.Reflection.Metadata/src/System/Reflection/PortableExecutable/PEReader.cs b/...libraries/System.Reflection.Metadata/src/System/Reflection/PortableExecutable/PEReader.cs
@@ -385,6 +385,12 @@ private AbstractMemoryBlock GetPESectionBlock(int index)
                 Interlocked.CompareExchange(ref _lazyPESectionBlocks, new AbstractMemoryBlock[PEHeaders.SectionHeaders.Length], null);
             }
 
+            AbstractMemoryBlock? existingBlock = Volatile.Read(ref _lazyPESectionBlocks[index]);
+            if (existingBlock != null)
+            {
+                return existingBlock;
+            }
+
             AbstractMemoryBlock newBlock;
             if (IsLoadedImage)
             {