Added error trapping when attepting to get the last exit code from a …

…remote machine. Microsoft does not allow you to retrieve information from remote computers due to security and architecture.
Keyfactor · Apr 17, 2024 · 3eb72af · 3eb72af
1 parent 4b7b26e
commit 3eb72af
Showing 1 changed file with 12 additions and 2 deletions.
diff --git a/IISU/ClientPSCertStoreManager.cs b/IISU/ClientPSCertStoreManager.cs
@@ -147,8 +147,18 @@ public JobResult ImportPFXFile(string filePath, string privateKeyPassword, strin
                     // Invoke the script
                     var results = ps.Invoke();
 
-                    //Get the last exist code returned from the script
-                    int lastExitCode = (int)(ps.Runspace.SessionStateProxy.PSVariable.GetValue("c"));
+                    // Get the last exist code returned from the script
+                    // This statement is in a try/catch block because PSVariable.GetValue() is not a valid method on a remote PS Session and throws an exception.
+                    // Due to security reasons and Windows architecture, retreiving values from a remote system is not supported.
+                    int lastExitCode = 0;
+                    try
+                    {
+                        lastExitCode = (int)ps.Runspace.SessionStateProxy.PSVariable.GetValue("c");
+                    }
+                    catch (Exception)
+                    {
+                    }
+
 
                     bool isError = false;
                     if (lastExitCode != 0)