From 25ac0334fe7f8ac6b5d5b8bdf26a003d4c225764 Mon Sep 17 00:00:00 2001
From: James Crosswell <jamescrosswell@users.noreply.github.com>
Date: Tue, 18 Jun 2024 22:10:31 +1200
Subject: [PATCH 1/3] Fix memory leak when tracing is enabled

---
 src/Sentry/TransactionTracer.cs | 25 +++++++++++++++++++++++++
 1 file changed, 25 insertions(+)

diff --git a/src/Sentry/TransactionTracer.cs b/src/Sentry/TransactionTracer.cs
index 6ce9295a76..6b459b767f 100644
--- a/src/Sentry/TransactionTracer.cs
+++ b/src/Sentry/TransactionTracer.cs
@@ -167,7 +167,11 @@ public IReadOnlyList<string> Fingerprint
     /// <inheritdoc />
     public IReadOnlyDictionary<string, string> Tags => _tags;
 
+#if NETSTANDARD2_1_OR_GREATER
     private readonly ConcurrentBag<ISpan> _spans = new();
+#else
+    private ConcurrentBag<ISpan> _spans = new();
+#endif
 
     /// <inheritdoc />
     public IReadOnlyCollection<ISpan> Spans => _spans;
@@ -337,6 +341,14 @@ public void Push(ISpan span)
                 return null;
             }
         }
+
+        public void Clear()
+        {
+            lock (_lock)
+            {
+                TrackedSpans.Clear();
+            }
+        }
     }
     private readonly LastActiveSpanTracker _activeSpanTracker = new LastActiveSpanTracker();
 
@@ -373,6 +385,9 @@ public void Finish()
 
         // Client decides whether to discard this transaction based on sampling
         _hub.CaptureTransaction(new SentryTransaction(this));
+
+        // Release tracked spans
+        ReleaseSpans();
     }
 
     /// <inheritdoc />
@@ -395,4 +410,14 @@ public void Finish(Exception exception) =>
 
     /// <inheritdoc />
     public SentryTraceHeader GetTraceHeader() => new(TraceId, SpanId, IsSampled);
+
+    private void ReleaseSpans()
+    {
+#if NETSTANDARD2_1_OR_GREATER
+        _spans.Clear();
+#else
+        _spans = new ConcurrentBag<ISpan>();
+#endif
+        _activeSpanTracker.Clear();
+    }
 }

From b33c94474e8d89012c06b263169fea4da4c2ceeb Mon Sep 17 00:00:00 2001
From: James Crosswell <jamescrosswell@users.noreply.github.com>
Date: Tue, 18 Jun 2024 22:13:49 +1200
Subject: [PATCH 2/3] Update CHANGELOG.md

---
 CHANGELOG.md | 1 +
 1 file changed, 1 insertion(+)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 365b3342e8..137b509794 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -7,6 +7,7 @@
 - Fixed Monitor duration calculation ([#3420]https://github.com/getsentry/sentry-dotnet/pull/3420)
 - Fixed null IServiceProvider in anonymous routes with OpenTelemetry ([#3401](https://github.com/getsentry/sentry-dotnet/pull/3401))
 - Fixed Trim warnings in Sentry.DiagnosticSource and WinUIUnhandledException integrations ([#3410](https://github.com/getsentry/sentry-dotnet/pull/3410))
+- Fix memory leak when tracing is enabled ([#3432](https://github.com/getsentry/sentry-dotnet/pull/3432))
 
 ### Dependencies
 

From 9ac327c8498ea2d7387d17b5d8d3c88444b96c57 Mon Sep 17 00:00:00 2001
From: James Crosswell <jamescrosswell@users.noreply.github.com>
Date: Tue, 18 Jun 2024 22:34:18 +1200
Subject: [PATCH 3/3] Update SentryTransactionTests.cs

---
 test/Sentry.Tests/Protocol/SentryTransactionTests.cs | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/test/Sentry.Tests/Protocol/SentryTransactionTests.cs b/test/Sentry.Tests/Protocol/SentryTransactionTests.cs
index 80012d33a8..6d33e19a2c 100644
--- a/test/Sentry.Tests/Protocol/SentryTransactionTests.cs
+++ b/test/Sentry.Tests/Protocol/SentryTransactionTests.cs
@@ -130,7 +130,8 @@ public void Redact_Redacts_Urls()
         child2.SetExtra("f222", "p111");
         child2.Finish(SpanStatus.OutOfRange);
 
-        txTracer.Finish(SpanStatus.Aborted);
+        // Don't finish the tracer - that would cause the spans to be released
+        // txTracer.Finish(SpanStatus.Aborted);
 
         // Act
         var transaction = new SentryTransaction(txTracer);