Executor: Don't use Process.ExitCode, unless the process has exited (#…

…1947)
dotnet · Mar 18, 2022 · 01ead56 · 01ead56
1 parent 1d34f85
commit 01ead56
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/src/BenchmarkDotNet/Toolchains/Executor.cs b/src/BenchmarkDotNet/Toolchains/Executor.cs
@@ -81,7 +81,12 @@ private ExecuteResult Execute(Process process, BenchmarkCase benchmarkCase, Sync
             if (loggerWithDiagnoser.LinesWithResults.Any(line => line.Contains("BadImageFormatException")))
                 logger.WriteLineError("You are probably missing <PlatformTarget>AnyCPU</PlatformTarget> in your .csproj file.");
 
-            return new ExecuteResult(true, process.ExitCode, process.Id, loggerWithDiagnoser.LinesWithResults, loggerWithDiagnoser.LinesWithExtraOutput, launchIndex);
+            return new ExecuteResult(true,
+                process.HasExited ? process.ExitCode : null,
+                process.Id,
+                loggerWithDiagnoser.LinesWithResults,
+                loggerWithDiagnoser.LinesWithExtraOutput,
+                launchIndex);
         }
 
         private ProcessStartInfo CreateStartInfo(BenchmarkCase benchmarkCase, ArtifactsPaths artifactsPaths, string args, IResolver resolver)