Re: [PATCH] advapi32/tests: Fix resource leak.

[Alexandre Julliard]

Andrey Gusev  writes:
> Signed-off-by: Andrey Gusev
>
> dlls/advapi32/tests/security.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/dlls/advapi32/tests/security.c b/dlls/advapi32/tests/security.c
> index 51dcf90851..4e584eb08f 100644
> --- a/dlls/advapi32/tests/security.c
> +++ b/dlls/advapi32/tests/security.c
> @@ -3036,6 +3036,7 @@ static void test_process_security_child(void)
>      handle = OpenProcess( PROCESS_VM_READ, FALSE, GetCurrentProcessId() );
>      todo_wine
>      ok(handle == NULL, "OpenProcess(PROCESS_VM_READ) should have failed\n");
> - CloseHandle( handle );

That's not very useful since it's supposed to fail. If the goal is to
close it because it wrongly succeeds on Wine, there are more places that
would need the same thing.