From: Johan Lorensson <lateralusx.github@gmail.com>
Date: Mon, 23 Oct 2023 23:46:55 +0000 (+0200)
Subject: Fix ModuleLoadUnloadTraceData.ModuleID cast to be unchecked since its really a ulong... 
X-Git-Tag: accepted/tizen/unified/riscv/20231226.055542~35^2~1^2~1
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=093a0ea50d7c04c520d18ced9fab4849d2c7ff9e;p=platform%2Fcore%2Fdotnet%2Fdiagnostics.git

Fix ModuleLoadUnloadTraceData.ModuleID cast to be unchecked since its really a ulong member. (#4354)

The ModuleLoadUnload events ModuleID is typed as a uint64 in the
EventPipe manifest and emitted as a uint64 in the event payload.
However, the parsing logic in ModuleLoadUnloadTraceDataevent in trace
event:


https://github.com/microsoft/perfview/blob/a6c87911fe1aef8f59c9ce54aa4e16a1be6db91e/src/TraceEvent/Parsers/ClrEtwAll.cs.base#L4963

handles it as a long. Android devices running arm64 can use pointer
tagging meaning that high bits can be set in 64-bit addresses and since
module id is a memory address, it will be returned as a negative long
from ModuleLoadUnload event and since all diagnostic tooling is build
with overflow checking enabled by default, casting it to a ulong will
trigger and overflow exception when high bit is set.

Fix is to do an unchecked cast in this case since the value should be
threated as a ulong in first place.

Fixes https://github.com/dotnet/diagnostics/issues/4348.
---

diff --git a/src/Tools/dotnet-gcdump/DotNetHeapDump/DotNetHeapDumpGraphReader.cs b/src/Tools/dotnet-gcdump/DotNetHeapDump/DotNetHeapDumpGraphReader.cs
index c26488fce..328febb54 100644
--- a/src/Tools/dotnet-gcdump/DotNetHeapDump/DotNetHeapDumpGraphReader.cs
+++ b/src/Tools/dotnet-gcdump/DotNetHeapDump/DotNetHeapDumpGraphReader.cs
@@ -117,12 +117,13 @@ public class DotNetHeapDumpGraphReader
                 return;
             }
 
-            if (!m_moduleID2Name.ContainsKey((ulong)data.ModuleID))
+            ulong moduleID = unchecked((ulong)data.ModuleID);
+            if (!m_moduleID2Name.ContainsKey(moduleID))
             {
-                m_moduleID2Name[(ulong)data.ModuleID] = data.ModuleILPath;
+                m_moduleID2Name[moduleID] = data.ModuleILPath;
             }
 
-            m_log.WriteLine("Found Module {0} ID 0x{1:x}", data.ModuleILFileName, (ulong)data.ModuleID);
+            m_log.WriteLine("Found Module {0} ID 0x{1:x}", data.ModuleILFileName, moduleID);
         };
         source.Clr.AddCallbackForEvents(moduleCallback); // Get module events for clr provider
         // TODO should not be needed if we use CAPTURE_STATE when collecting.