🚀 go-pugleaf

"Stefano Bonifazi" <stefboombastic@email.it> wrote in message
news:41d2f95b@newsgroups.borland.com...

>   //create pipe for the console stdout
>
if(!CreatePipe(&ChildSTDOUT_ReadPipeHandleTmp,&ChildSTDOUT_WritePipeHandle,&
sa,0))
>       return -1;
>   //create pipe for the console stderr
>
if(!CreatePipe(&ChildSTDERR_ReadPipeHandleTmp,&ChildSTDERR_WritePipeHandle,&
sa,0))
>       return -2;

You don't need separate pipes for stdout and stderr.  You can use the same
pipe for both of them.

>   if(!DuplicateHandle(hProcess, ChildSTDOUT_ReadPipeHandleTmp,hProcess,
> &ChildSTDOUT_ReadPipeHandle, 0, FALSE, DUPLICATE_SAME_ACCESS))
>    return -4;

Why are you duplicating the pipes you just created?

>   si.lpDesktop =  "WinSta0\\Default"; //also tried without anything
here!!!

Are you intending for the user logged into the service's machine to interact
with the console that you are creating?  If not, then get rid of that.  You
only need to specify the Desktop if you are going to make the console visual
for interaction with the user.  Otherwise, CreateProcessAsUser() creates the
console non-visually instead.

>   CloseHandle(TempTokenHandle);

You are closing the token while you are still impersonating the user and
while the console process is still running.  Wait until the console process
is terminated before doing that.  Also, if CreateProcessAsUser() fails, you
are not removing the impersonation.


Gambit