Re: Add tests for ntdll event functions

[Andrew Cook]

While implemeting NtQueryEvent i stumbled across an odd permissions
issue, which is really what this patch tests

    if ((event = create_event( root, &name, req->attributes,
req->manual_reset, req->initial_state, sd )))
    {
        if (get_error() == STATUS_OBJECT_NAME_EXISTS)
            reply->handle = alloc_handle( current->process, event,
req->access, req->attributes );
        else
            reply->handle = alloc_handle_no_access_check(
current->process, event, req->access, req->attributes );
        release_object( event );
    }

when creating an event, the handle is allocated with
alloc_handle_no_access_check, which does not call event_map_access
this causes event_op and event_query to fail, as the handle only has
GENERIC_ALL, not the actual event permissions,
which does not match windows behaviour (or at least windows 7 anyway),

On 04/06/13 11:13, Andrew Cook wrote:
> ---
>  dlls/ntdll/tests/Makefile.in |  1 +
>  dlls/ntdll/tests/event.c     | 69
> ++++++++++++++++++++++++++++++++++++++++++++
>  2 files changed, 70 insertions(+)
>  create mode 100644 dlls/ntdll/tests/event.c
>
> diff --git a/dlls/ntdll/tests/Makefile.in b/dlls/ntdll/tests/Makefile.in
> index 10d6674..a44b880 100644
> --- a/dlls/ntdll/tests/Makefile.in
> +++ b/dlls/ntdll/tests/Makefile.in
> @@ -7,6 +7,7 @@ C_SRCS = \
>   directory.c \
>   env.c \
>   error.c \
> +	event.c \
>   exception.c \
>   file.c \
>   generated.c \
> diff --git a/dlls/ntdll/tests/event.c b/dlls/ntdll/tests/event.c
> new file mode 100644
> index 0000000..f29abc3
> --- /dev/null
> +++ b/dlls/ntdll/tests/event.c
> @@ -0,0 +1,69 @@
> +/*
> + * Unit test suite for event functions
> + *
> + * Copyright 2005 Robert Shearman
> + * Copyright 2005 Vitaliy Margolen
> + *
> + * This library is free software; you can redistribute it and/or
> + * modify it under the terms of the GNU Lesser General Public
> + * License as published by the Free Software Foundation; either
> + * version 2.1 of the License, or (at your option) any later version.
> + *
> + * This library is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
> + * Lesser General Public License for more details.
> + *
> + * You should have received a copy of the GNU Lesser General Public
> + * License along with this library; if not, write to the Free Software
> + * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA
> 02110-1301, USA
> + */
> +
> +#include "ntdll_test.h"
> +#include "winternl.h"
> +#include "winnt.h"
> +
> +static NTSTATUS (WINAPI *pNtCreateEvent) ( PHANDLE, ACCESS_MASK, const
> POBJECT_ATTRIBUTES, BOOLEAN, BOOLEAN);
> +static NTSTATUS (WINAPI *pNtPulseEvent)  ( HANDLE, PULONG );
> +static NTSTATUS (WINAPI *pNtQueryEvent)  ( HANDLE,
> EVENT_INFORMATION_CLASS, PVOID, ULONG, PULONG );
> +static NTSTATUS (WINAPI *pNtClose)       ( HANDLE );
> +static VOID     (WINAPI *pRtlInitUnicodeString)( PUNICODE_STRING,
> LPCWSTR );
> +
> +START_TEST(event)
> +{
> +    HANDLE Event;
> +    NTSTATUS status;
> +    UNICODE_STRING str;
> +    OBJECT_ATTRIBUTES attr;
> +    EVENT_BASIC_INFORMATION info;
> +    HMODULE hntdll = GetModuleHandleA("ntdll.dll");
> +    HMODULE hkernel32 = GetModuleHandleA("kernel32.dll");
> +    static const WCHAR eventName[] =
> {'\\','B','a','s','e','N','a','m','e','d','O','b','j','e','c','t','s','\\','t','e','s','t','E','v','e','n','t',0};
> +
> +    if (!hntdll)
> +    {
> +        skip("not running on NT, skipping test\n");
> +        return;
> +    }
> +
> +    pNtCreateEvent = (void *)GetProcAddress(hntdll, "NtCreateEvent");
> +    pNtQueryEvent  = (void *)GetProcAddress(hntdll, "NtQueryEvent");
> +    pNtPulseEvent  = (void *)GetProcAddress(hntdll, "NtPulseEvent");
> +    pNtClose       = (void *)GetProcAddress(hntdll, "NtClose");
> +    pRtlInitUnicodeString = (void *)GetProcAddress(hntdll,
> "RtlInitUnicodeString");
> +
> +    pRtlInitUnicodeString(&str, eventName);
> +    InitializeObjectAttributes(&attr, &str, 0, 0, NULL);
> +
> +    status = pNtCreateEvent(&Event, GENERIC_ALL, &attr, 0, 0);
> +    ok( status == STATUS_SUCCESS, "NtCreateEvent failed %08x\n", status );
> +
> +    status = pNtPulseEvent(Event, 0);
> +    todo_wine ok( status == STATUS_SUCCESS, "NtPulseEvent failed
> %08x\n", status );
> +
> +    status = pNtQueryEvent(Event, EventBasicInformation, &info,
> sizeof(info), NULL);
> +    todo_wine ok( status == STATUS_SUCCESS, "NtQueryEvent failed
> %08x\n", status );
> +
> +    status = pNtClose(Event);
> +    ok( status == STATUS_SUCCESS, "NtClose failed %08x\n", status );
> +}