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

[Paul Vriens]

On 11/04/2009 09:45 PM, Reece Dunn wrote:
> 2009/11/4 Paul Vriens:
> > 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?

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

Yes. I'm in the process of writing loads of eventlog tests while keeping 
track of possible failures on test.winehq.org.
>  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

Deleting the file works just fine. both in create_backup() and 
test_clear().

What makes it even more weird is that I have the same error when I run 
these tests as the user (not admin).

I also don't get any UAC prompts in either case. Turning UAC off doesn't 
make a difference.

I don't see any logical explanation why the function returns a different 
error. The fact that the function fails anyway makes me consider using a 
second possible last error for that particular test.