Jacek Caban : ntdll: Allow FileAccessInformation to be queried on files without fds.

[Alexandre Julliard]

Module: wine
Branch: stable
Commit: 587c6000e8927056466a8a8dffd39f2fba95475c
URL:    https://source.winehq.org/git/wine.git/?a=commit;h=587c6000e8927056466a8a8df...

Author: Jacek Caban 
Date:   Mon Aug 20 19:29:16 2018 +0200

ntdll: Allow FileAccessInformation to be queried on files without fds.

Signed-off-by: Jacek Caban 
Signed-off-by: Alexandre Julliard 
(cherry picked from commit 8db78eccab18c90474e9f625e5aecf6daa10be44)
Signed-off-by: Michael Stefaniuc

---

dlls/kernel32/tests/pipe.c | 38 ++++++++++++++++++++++++++++++++++++++
 dlls/ntdll/file.c          |  2 +-
 2 files changed, 39 insertions(+), 1 deletion(-)

diff --git a/dlls/kernel32/tests/pipe.c b/dlls/kernel32/tests/pipe.c
index 922eed9..1824b05 100644
--- a/dlls/kernel32/tests/pipe.c
+++ b/dlls/kernel32/tests/pipe.c
@@ -140,6 +140,20 @@ static void _test_pipe_info(unsigned line, HANDLE pipe, DWORD ex_flags, DWORD ex
     ok_(__FILE__,line)(max_instances == ex_max_instances, "max_instances = %x, expected %u\n", max_instances, ex_max_instances);
 }

+#define test_file_access(a,b) _test_file_access(__LINE__,a,b)
+static void _test_file_access(unsigned line, HANDLE handle, DWORD expected_access)
+{
+    FILE_ACCESS_INFORMATION info;
+    IO_STATUS_BLOCK io;
+    NTSTATUS status;
+
+    memset(&info, 0x11, sizeof(info));
+    status = NtQueryInformationFile(handle, &io, &info, sizeof(info), FileAccessInformation);
+    ok_(__FILE__,line)(status == STATUS_SUCCESS, "expected STATUS_SUCCESS, got %08x\n", status);
+    ok_(__FILE__,line)(info.AccessFlags == expected_access, "got access %08x expected %08x\n",
+                       info.AccessFlags, expected_access);
+}
+
 static void test_CreateNamedPipe(int pipemode)
 {
     HANDLE hnp;
@@ -209,6 +223,10 @@ static void test_CreateNamedPipe(int pipemode)
     ok(hnp != INVALID_HANDLE_VALUE, "CreateNamedPipe failed\n");
     test_signaled(hnp);

+    test_file_access(hnp, SYNCHRONIZE | READ_CONTROL | FILE_WRITE_ATTRIBUTES
+                     | FILE_READ_ATTRIBUTES | FILE_WRITE_PROPERTIES | FILE_READ_PROPERTIES
+                     | FILE_APPEND_DATA | FILE_WRITE_DATA | FILE_READ_DATA);
+
     ret = PeekNamedPipe(hnp, NULL, 0, NULL, &readden, NULL);
     todo_wine
     ok(!ret && GetLastError() == ERROR_BAD_PIPE, "PeekNamedPipe returned %x (%u)\n",
@@ -627,6 +645,26 @@ static void test_CreateNamedPipe(int pipemode)

ok(CloseHandle(hnp), "CloseHandle\n");

+    hnp = CreateNamedPipeA(PIPENAME, PIPE_ACCESS_INBOUND, pipemode | PIPE_WAIT,
+                           1, 1024, 1024, NMPWAIT_USE_DEFAULT_WAIT, NULL);
+    ok(hnp != INVALID_HANDLE_VALUE, "CreateNamedPipe failed\n");
+    test_signaled(hnp);
+
+    test_file_access(hnp, SYNCHRONIZE | READ_CONTROL | FILE_READ_ATTRIBUTES | FILE_READ_PROPERTIES
+                     | FILE_READ_DATA);
+
+    CloseHandle(hnp);
+
+    hnp = CreateNamedPipeA(PIPENAME, PIPE_ACCESS_OUTBOUND, pipemode | PIPE_WAIT,
+                           1, 1024, 1024, NMPWAIT_USE_DEFAULT_WAIT, NULL);
+    ok(hnp != INVALID_HANDLE_VALUE, "CreateNamedPipe failed\n");
+    test_signaled(hnp);
+
+    test_file_access(hnp, SYNCHRONIZE | READ_CONTROL | FILE_WRITE_ATTRIBUTES
+                     | FILE_WRITE_PROPERTIES | FILE_APPEND_DATA | FILE_WRITE_DATA);
+
+    CloseHandle(hnp);
+
     if (winetest_debug > 1) trace("test_CreateNamedPipe returning\n");
 }

diff --git a/dlls/ntdll/file.c b/dlls/ntdll/file.c
index b9d2425..1517b28 100644
--- a/dlls/ntdll/file.c
+++ b/dlls/ntdll/file.c
@@ -2320,7 +2320,7 @@ NTSTATUS WINAPI NtQueryInformationFile( HANDLE hFile, PIO_STATUS_BLOCK io,
     if (len < info_sizes[class])
         return io->u.Status = STATUS_INFO_LENGTH_MISMATCH;

-    if (class != FilePipeInformation && class != FilePipeLocalInformation)
+    if (class != FilePipeInformation && class != FilePipeLocalInformation && class != FileAccessInformation)
     {
         if ((io->u.Status = server_get_unix_fd( hFile, 0, &fd, &needs_close, NULL, NULL )))
         {