Paul Gofman : server: Cancel pipe asyncs when the last handle in process is closed.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 03c1930b74e9a41da84fd712a3ebdbb4f7fcbcb5
URL:    https://gitlab.winehq.org/wine/wine/-/commit/03c1930b74e9a41da84fd712a3ebdbb...

Author: Paul Gofman 
Date:   Fri Jul 14 20:21:50 2023 -0600

server: Cancel pipe asyncs when the last handle in process is closed.

---

dlls/ntdll/tests/pipe.c | 5 ++++-
 server/named_pipe.c     | 4 ++--
 2 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/dlls/ntdll/tests/pipe.c b/dlls/ntdll/tests/pipe.c
index c9cfdb03826..a1115bf47b9 100644
--- a/dlls/ntdll/tests/pipe.c
+++ b/dlls/ntdll/tests/pipe.c
@@ -3067,9 +3067,12 @@ static void test_async_cancel_on_handle_close(void)
             ok(bret, "failed, error %lu.\n", GetLastError());

CloseHandle(read);
+            /* Canceled asyncs with completion port and no event do not update IOSB before removing completion. */
+            todo_wine_if(other_process && tests[i].apc_context && !tests[i].event)
             ok(io.Status == 0xcccccccc, "got %#lx.\n", io.Status);
+
             if (other_process && tests[i].apc_context && !tests[i].event)
-                todo_wine test_queued_completion(port, &io, STATUS_CANCELLED, 0);
+                test_queued_completion(port, &io, STATUS_CANCELLED, 0);
             else
                 test_no_queued_completion(port);

diff --git a/server/named_pipe.c b/server/named_pipe.c
index 9a10c2bf041..3adc79068ab 100644
--- a/server/named_pipe.c
+++ b/server/named_pipe.c
@@ -179,7 +179,7 @@ static const struct object_ops pipe_server_ops =
     NULL,                         /* unlink_name */
     pipe_server_open_file,        /* open_file */
     no_kernel_obj_list,           /* get_kernel_obj_list */
-    no_close_handle,              /* close_handle */
+    async_close_obj_handle,       /* close_handle */
     pipe_server_destroy           /* destroy */
 };

@@ -223,7 +223,7 @@ static const struct object_ops pipe_client_ops =
     NULL,                         /* unlink_name */
     no_open_file,                 /* open_file */
     no_kernel_obj_list,           /* get_kernel_obj_list */
-    no_close_handle,              /* close_handle */
+    async_close_obj_handle,       /* close_handle */
     pipe_end_destroy              /* destroy */
 };