Brendan Shanks : server: Don't block ConnectNamedPipe for pipe in PIPE_NOWAIT mode.
Alexandre Julliard
julliard at winehq.org
Wed Oct 16 16:59:30 CDT 2019
Module: wine
Branch: master
Commit: 41374ef1967e2c0ec2a74c4d47a5d8fc8bb5b4b8
URL: https://source.winehq.org/git/wine.git/?a=commit;h=41374ef1967e2c0ec2a74c4d47a5d8fc8bb5b4b8
Author: Brendan Shanks <bshanks at codeweavers.com>
Date: Wed Oct 16 14:50:45 2019 +0200
server: Don't block ConnectNamedPipe for pipe in PIPE_NOWAIT mode.
Signed-off-by: Brendan Shanks <bshanks at codeweavers.com>
Signed-off-by: Jacek Caban <jacek at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>
---
dlls/kernel32/tests/pipe.c | 4 ++--
server/named_pipe.c | 5 +++++
2 files changed, 7 insertions(+), 2 deletions(-)
diff --git a/dlls/kernel32/tests/pipe.c b/dlls/kernel32/tests/pipe.c
index de30e6a1c9..a192779e1d 100644
--- a/dlls/kernel32/tests/pipe.c
+++ b/dlls/kernel32/tests/pipe.c
@@ -4026,7 +4026,7 @@ static void test_nowait(void)
ol.hEvent = CreateEventW(NULL, TRUE, FALSE, NULL);
SetLastError(0xdeadbeef);
ok(ConnectNamedPipe(pipewrite, &ol) == FALSE, "ConnectNamedPipe should fail\n");
- todo_wine ok(GetLastError() == ERROR_PIPE_LISTENING, "got %d should be ERROR_PIPE_LISTENING\n", GetLastError());
+ ok(GetLastError() == ERROR_PIPE_LISTENING, "got %d should be ERROR_PIPE_LISTENING\n", GetLastError());
if (GetLastError() == ERROR_IO_PENDING)
CancelIo(pipewrite);
@@ -4044,7 +4044,7 @@ static void test_nowait(void)
ok(DisconnectNamedPipe(pipewrite) == TRUE, "DisconnectNamedPipe should succeed\n");
SetLastError(0xdeadbeef);
ok(ConnectNamedPipe(pipewrite,&ol) == FALSE, "ConnectNamedPipe should fail\n");
- todo_wine ok(GetLastError() == ERROR_PIPE_LISTENING, "got %d should be ERROR_PIPE_LISTENING\n", GetLastError());
+ ok(GetLastError() == ERROR_PIPE_LISTENING, "got %d should be ERROR_PIPE_LISTENING\n", GetLastError());
if (GetLastError() == ERROR_IO_PENDING)
CancelIo(pipewrite);
ok(CloseHandle(ol.hEvent), "CloseHandle for the event failed\n");
diff --git a/server/named_pipe.c b/server/named_pipe.c
index 9c5438b77a..f97b141641 100644
--- a/server/named_pipe.c
+++ b/server/named_pipe.c
@@ -1106,6 +1106,11 @@ static int pipe_server_ioctl( struct fd *fd, ioctl_code_t code, struct async *as
return 0;
}
+ if (server->pipe_end.flags & NAMED_PIPE_NONBLOCKING_MODE)
+ {
+ set_error( STATUS_PIPE_LISTENING );
+ return 0;
+ }
queue_async( &server->listen_q, async );
async_wake_up( &server->pipe_end.pipe->waiters, STATUS_SUCCESS );
set_error( STATUS_PENDING );
More information about the wine-cvs
mailing list