Use GetExitCodeProcess() because WaitForSingleObject needs more access rights on the handle.

KN4CK3R · KN4CK3R · commit b5374ea3573f · 2017-09-01T23:48:31.000+02:00
diff --git a/PipeServer/Messages.cpp b/PipeServer/Messages.cpp
@@ -22,7 +22,8 @@ bool CloseProcessRequest::Handle(MessageClient& client)
 //---------------------------------------------------------------------------
 bool IsValidRequest::Handle(MessageClient& client)
 {
-	const auto success = WaitForSingleObject(GetRemoteId(), 0) == WAIT_TIMEOUT;
+	DWORD exitCode = 0;
+	const auto success = GetExitCodeProcess(GetRemoteId(), &exitCode) != FALSE && exitCode == STILL_ACTIVE;
 
 	client.Send(StatusResponse(GetRemoteId(), success));
 

Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,8 @@ bool CloseProcessRequest::Handle(MessageClient& client)`
`22`	`22`	`//---------------------------------------------------------------------------`
`23`	`23`	`bool IsValidRequest::Handle(MessageClient& client)`
`24`	`24`	`{`
`25`		`- const auto success = WaitForSingleObject(GetRemoteId(), 0) == WAIT_TIMEOUT;`
	`25`	`+ DWORD exitCode = 0;`
	`26`	`+ const auto success = GetExitCodeProcess(GetRemoteId(), &exitCode) != FALSE && exitCode == STILL_ACTIVE;`
`26`	`27`
`27`	`28`	`client.Send(StatusResponse(GetRemoteId(), success));`
`28`	`29`