Skip to content

Commit

Permalink
Fix the error when using Start-Process -Credential without the admi…
Browse files Browse the repository at this point in the history
…n privilege (#21393)
  • Loading branch information
jborean93 committed Apr 15, 2024
1 parent 6e99fd4 commit 588dff6
Show file tree
Hide file tree
Showing 2 changed files with 18 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -2109,7 +2109,21 @@ protected override void BeginProcessing()
// "Process was not started by this object, so requested information cannot be determined."
// Fetching the process handle will trigger the `Process` object to update its internal state by calling `SetProcessHandle`,
// the result is discarded as it's not used later in this code.
_ = process.Handle;
try
{
_ = process.Handle;
}
catch (Win32Exception e)
{
// If the caller was not an admin and the process was started with another user's credentials .NET
// won't be able to retrieve the process handle. As this is not a critical failure we treat this as
// a warning.
if (PassThru)
{
string msg = StringUtil.Format(ProcessResources.FailedToCreateProcessObject, e.Message);
WriteDebug(msg);
}
}

// Resume the process now that is has been set up.
processInfo.Resume();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -189,6 +189,9 @@
<data name="CannotStarttheProcess" xml:space="preserve">
<value>This command cannot be run completely because the system cannot find all the information required.</value>
</data>
<data name="FailedToCreateProcessObject" xml:space="preserve">
<value>Failed to retrieve the new process handle: "{0}". The Process object outputted may have some properties and methods that do not work properly.</value>
</data>
<data name="InvalidUserError" xml:space="preserve">
<value>This command cannot be run due to error 1783. The possible cause of this error can be using of a non-existing user "{0}". Please give a valid user and run your command again.</value>
</data>
Expand Down

0 comments on commit 588dff6

Please sign in to comment.