s3: smbd: Ensure we remove any pending aio values for named pipes on forced shutdown.

Matches file and directory closes.

Remove knownfail.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=15423

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Wed Sep 20 02:43:18 UTC 2023 on atb-devel-224

diff --git a/selftest/knownfail.d/smb2_pipe_read_async_disconnect b/selftest/knownfail.d/smb2_pipe_read_async_disconnect
deleted file mode 100644 (file)
index 342a308..0000000
--- a/selftest/knownfail.d/smb2_pipe_read_async_disconnect
+++ /dev/null
@@ -1 +0,0 @@
-^samba3.smbtorture_s3.plain.SMB2-PIPE-READ-ASYNC-DISCONNECT.check_panic\(fileserver\)

diff --git a/source3/smbd/close.c b/source3/smbd/close.c
index 2b180e0c7182026ea60444617004c692a0c29861..af5e78daa1005c6e68803f77ca9dc175bc927df3 100644 (file)
--- a/source3/smbd/close.c
+++ b/source3/smbd/close.c
@@ -1630,6 +1630,14 @@ NTSTATUS close_file_smb(struct smb_request *req,
        SMB_ASSERT(fsp->stream_fsp == NULL);
 
        if (fsp->fake_file_handle != NULL) {
+               /*
+                * Named pipes are opened as fake files and
+                * can have pending aio requests. Ensure
+                * we clear out all pending aio on force
+                * shutdown of named pipes also.
+                * BUG: https://bugzilla.samba.org/show_bug.cgi?id=15423
+                */
+               assert_no_pending_aio(fsp, close_type);
                status = close_fake_file(req, fsp);
        } else if (fsp->print_file != NULL) {
                /* FIXME: return spool errors */