Merge pull request #361 from ionite34/fix-process-error

Fix process errors on platforms not supporting name or start time reports

Author: ionite34

CHANGELOG.md

@@ -5,6 +5,10 @@ All notable changes to Stability Matrix will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning 2.0](https://semver.org/spec/v2.0.0.html).
 
+## v2.6.3
+### Fixed
+- Fixed InvalidOperationException during prerequisite installs on certain platforms where process name and duration reporting are not supported
+
 ## v2.6.2
 ### Changed
 - Backend changes for auto-update schema v3, supporting customizable release channels and faster downloads with zip compression

StabilityMatrix.Core/Processes/ProcessRunner.cs

@@ -204,13 +204,29 @@ public static async Task<ProcessResult> GetProcessResultAsync(
 
         await process.WaitForExitAsync().ConfigureAwait(false);
 
+        string? processName = null;
+        TimeSpan elapsed = default;
+
+        // Accessing these properties may throw an exception if the process has already exited
+        try
+        {
+            processName = process.ProcessName;
+        }
+        catch (SystemException) { }
+
+        try
+        {
+            elapsed = process.ExitTime - process.StartTime;
+        }
+        catch (SystemException) { }
+
         return new ProcessResult
         {
             ExitCode = process.ExitCode,
             StandardOutput = stdout,
             StandardError = stderr,
-            ProcessName = process.MachineName,
-            Elapsed = process.ExitTime - process.StartTime
+            ProcessName = processName,
+            Elapsed = elapsed
         };
     }