server: Require FILE_READ_ATTRIBUTES access in get_named_pipe_info.

Signed-off-by: Jacek Caban <jacek@codeweavers.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>

server: Require FILE_READ_ATTRIBUTES access in get_named_pipe_info.
d4e0f0a1 · Jacek Caban · Alexandre Julliard · 58233b47 · d4e0f0a1 · d4e0f0a1
Commit d4e0f0a1 authored Aug 20, 2018 by Jacek Caban Committed by Alexandre Julliard Aug 21, 2018
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 3 deletions

pipe.c dlls/ntdll/tests/pipe.c +4 -2

named_pipe.c server/named_pipe.c +1 -1

No files found.
--- a/dlls/ntdll/tests/pipe.c
+++ b/dlls/ntdll/tests/pipe.c
@@ -756,16 +756,18 @@ static void test_filepipeinfo(void)

    res = pNtQueryInformationFile(hClient, &iosb, &local_info, sizeof(local_info),
                                  FilePipeLocalInformation);
-    todo_wine
    ok(res == STATUS_ACCESS_DENIED,
       "NtQueryInformationFile(FilePipeLocalInformation) returned: %x\n", res);

    res = pNtQueryInformationFile(hClient, &iosb, &local_info, sizeof(local_info),
                                  FilePipeInformation);
-    todo_wine
    ok(res == STATUS_ACCESS_DENIED,
       "NtQueryInformationFile(FilePipeInformation) returned: %x\n", res);

+    res = pNtQueryInformationFile(hClient, &iosb, &local_info, sizeof(local_info),
+                                  FileNameInformation);
+    ok(res == STATUS_SUCCESS, "NtQueryInformationFile(FileNameInformation) returned: %x\n", res);
+
    CloseHandle(hClient);
    CloseHandle(hServer);
 }

--- a/server/named_pipe.c
+++ b/server/named_pipe.c
@@ -1284,7 +1284,7 @@ DECL_HANDLER(get_named_pipe_info)

        clear_error();
        pipe_end = (struct pipe_end *)get_handle_obj( current->process, req->handle,
-                                                      0, &pipe_client_ops );
+                                                      FILE_READ_ATTRIBUTES, &pipe_client_ops );
        if (!pipe_end) return;
    }