From 58770427553db77563782c3b257fabe73de12d88 Mon Sep 17 00:00:00 2001
From: MattMo <matt@montoyatech.com>
Date: Thu, 12 Sep 2024 20:37:25 -0700
Subject: [PATCH] Added filename null or empty check. Start now calls the
 SetProcessHandle and SetProcessId function so that events are setup
 correctly. Bumped package version to 1.0.3

---
 Process.Net/Process.Net.csproj |  2 +-
 Process.Net/WinProcess.cs      | 12 ++++++++----
 2 files changed, 9 insertions(+), 5 deletions(-)

diff --git a/Process.Net/Process.Net.csproj b/Process.Net/Process.Net.csproj
index 2079958..b290994 100644
--- a/Process.Net/Process.Net.csproj
+++ b/Process.Net/Process.Net.csproj
@@ -15,7 +15,7 @@
     <PackageTags>MontoyaTech;Process.Net</PackageTags>
     <AssemblyName>MontoyaTech.Process.Net</AssemblyName>
     <RootNamespace>MontoyaTech.Process.Net</RootNamespace>
-    <Version>1.0.2</Version>
+    <Version>1.0.3</Version>
     <Company>MontoyaTech</Company>
     <PackageLicenseExpression>MIT</PackageLicenseExpression>
     <PackageReadmeFile>README.md</PackageReadmeFile>
diff --git a/Process.Net/WinProcess.cs b/Process.Net/WinProcess.cs
index 1a00547..0e92352 100644
--- a/Process.Net/WinProcess.cs
+++ b/Process.Net/WinProcess.cs
@@ -427,6 +427,9 @@ namespace MontoyaTech.Process.Net
             if (startInfo == null)
                 throw new InvalidOperationException("Process.StartInfo cannot be null.");
 
+            if (string.IsNullOrEmpty(startInfo.FileName))
+                throw new InvalidOperationException("FileName cannot be null or empty.");
+
             var processType = process.GetType();
 
             var commandLine = new StringBuilder();
@@ -538,10 +541,11 @@ namespace MontoyaTech.Process.Net
                 childErrorHandle?.Dispose();
             }
 
-            //Unsafely setup the process info.
-            processType.GetField("_processId", System.Reflection.BindingFlags.NonPublic | System.Reflection.BindingFlags.Instance).SetValue(process, (int)processInfo.dwProcessId);
-            processType.GetField("_haveProcessId", System.Reflection.BindingFlags.NonPublic | System.Reflection.BindingFlags.Instance).SetValue(process, true);
-            processType.GetField("_processHandle", System.Reflection.BindingFlags.NonPublic | System.Reflection.BindingFlags.Instance).SetValue(process, processHandle);
+            //Invoke SetProcessHandle
+            processType.GetMethod("SetProcessHandle", System.Reflection.BindingFlags.NonPublic | System.Reflection.BindingFlags.Instance).Invoke(process, new object[] { processHandle });
+
+            //Invoke SetProcessId
+            processType.GetMethod("SetProcessId", System.Reflection.BindingFlags.NonPublic | System.Reflection.BindingFlags.Instance).Invoke(process, new object[] { (int)processInfo.dwProcessId });
 
             //Set the standard input writer if needed
             if (startInfo.RedirectStandardInput)