NamedPipe problem

[Oleg Prokhorov]

Hello ,

I've made the test program to check/test some createprocess behavior.
  But found that it doesn't work with wine at all (it's ok on windows).

In ConnectToNewClient
     ConnectNamedPipe fails and returns non zero.

The source is attached.
  Who is in charge for named pipes problems ? :)

[Mike McCormack]

Well, nobody is "in charge" if named pipes, but I might be able to help 
you fix the problem, since I wrote lots of the code :)

I'll have a look at it and get back to you.

Mike

Oleg Prokhorov wrote:
> Hello ,
>
> I've made the test program to check/test some createprocess behavior.
>   But found that it doesn't work with wine at all (it's ok on windows).
>
> In ConnectToNewClient
>      ConnectNamedPipe fails and returns non zero.
>
> The source is attached.
>   Who is in charge for named pipes problems ? :)

[Mike McCormack]

Hi Oleg,

This patch fixes the immediate problem, but there are more.

Mike

ChangeLog:
* always return FALSE when doing an overlapped ConnectNamedPipe

Oleg Prokhorov wrote:
> Hello ,
>
> I've made the test program to check/test some createprocess behavior.
>   But found that it doesn't work with wine at all (it's ok on windows).
>
> In ConnectToNewClient
>      ConnectNamedPipe fails and returns non zero.
>
> The source is attached.
>   Who is in charge for named pipes problems ? :)

Index: dlls/kernel/sync.c
===================================================================
RCS file: /home/wine/wine/dlls/kernel/sync.c,v
retrieving revision 1.32
diff -u -r1.32 sync.c
--- dlls/kernel/sync.c	20 May 2003 04:32:53 -0000	1.32
+++ dlls/kernel/sync.c	5 Jun 2003 04:37:44 -0000
@@ -911,7 +911,11 @@
     TRACE("(%p,%p)\n",hPipe, overlapped);

if(overlapped)
-        return SYNC_ConnectNamedPipe(hPipe,overlapped);
+    {
+        if(SYNC_ConnectNamedPipe(hPipe,overlapped))
+            SetLastError( ERROR_IO_PENDING );
+        return FALSE;
+    }

memset(&ov,0,sizeof ov);
     ov.hEvent = CreateEventA(NULL,0,0,NULL);

Re[2]: NamedPipe problem

[Oleg Prokhorov]

Hello Mike,

Thursday, June 5, 2003, 11:50:16 AM, you wrote:

MM> This patch fixes the immediate problem, but there are more.
What are they ?
If you need some help (including testing on windows), you are welcome.