Fix createdump on alpine (dotnet/coreclr#24623)

Issue # https://github.com/dotnet/coreclr/issues/24599 Commit migrated from dotnet/coreclr@33043c0
Rheoz · May 18, 2019 · d927e22 · d927e22
1 parent 30a188d
commit d927e22
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/coreclr/src/debug/createdump/crashinfo.cpp b/src/coreclr/src/debug/createdump/crashinfo.cpp
@@ -198,12 +198,14 @@ CrashInfo::GatherCrashInfo(MINIDUMP_TYPE minidumpType)
             }
         }
     }
-    // Add all the heap (read/write) memory regions (m_otherMappings contains the heaps)
+    // Add all the heap read/write memory regions (m_otherMappings contains the heaps). On Alpine
+    // the heap regions are marked RWX instead of just RW.
     else if (minidumpType & MiniDumpWithPrivateReadWriteMemory)
     {
         for (const MemoryRegion& region : m_otherMappings)
         {
-            if (region.Permissions() == (PF_R | PF_W))
+            uint32_t permissions = region.Permissions();
+            if (permissions == (PF_R | PF_W) || permissions == (PF_R | PF_W | PF_X))
             {
                 InsertMemoryBackedRegion(region);
             }