Weird error for the advapi32/eventlog tests on Vista+ when running winetest

[Reece Dunn]

2009/11/4 Paul Vriens <paul.vriens.wine at gmail.com>:
> Hi,
>
> Just noticed that we have 1 test failure on Vista+ for the advapi32/eventlog
> tests. I was sure I've tested these from Win95 up to Win7.
>
> When I run tests I want to do this as Administrator, so on Vista+ I select
> 'Run as Administrator' to start a command prompt, or to launch winetest.
>
> Running the mentioned crosscompiled tests is no problem unless you do this
> from the actual home directory (or any subdirectory) of the user that logged
> in.
>
> Winetest runs most of it's test in the %TEMP% directory and hence we see
> this failure. But I have no clue why (yet).
>
> Anyone?

Ah, I see. This is the failure on test.winehq.org, correct?

    eventlog.c:586: Test failed: Expected ERROR_INVALID_HANDLE, got 5

5 is ERROR_ACCESS_DENIED.

The create_backup function does not have any error checking -- what
happens when you put the return values for these calls, and the
GetLastError in ok checks, e.g.

    ok(DeleteFileA(filename), "error deleting the backup event log file");
    ok(GetLastError() == 0, "GetLastError -- expected ERROR_SUCCESS,
got %d", GetLastError());

- Reece