Merge pull request #16240 from tamasvajk/fix/cleanup

C#: Fix cleanup logic in dependency manager
2025-12-17 01:03:14 +01:00 · 2024-04-17 14:53:40 +02:00
parent 902f0f91b0 6f782f3f62
commit dc4cc74cbb
1 changed files with 5 additions and 2 deletions
--- a/csharp/extractor/Semmle.Extraction.CSharp.DependencyFetching/DependencyManager.cs
+++ b/csharp/extractor/Semmle.Extraction.CSharp.DependencyFetching/DependencyManager.cs
@@ -564,9 +564,12 @@ namespace Semmle.Extraction.CSharp.DependencyFetching

        public void Dispose()
        {
-            tempWorkingDirectory?.Dispose();
-            diagnosticsWriter?.Dispose();
            nugetPackageRestorer?.Dispose();
+            if (cleanupTempWorkingDirectory)
+            {
+                tempWorkingDirectory?.Dispose();
+            }
+            diagnosticsWriter?.Dispose();
        }
    }
 }