[advapi32/tests #2] Don't set last error to 0 so we can check if it has changed

Robert Shearman rob at codeweavers.com
Mon Feb 18 13:55:26 CST 2008

Alexandre Julliard wrote:
> "Paul Vriens" <paul.vriens.wine at gmail.com> writes:
>> I have no clue what the test does and what's wrong with it so can we
>> leave it like this? Or should we ask Rob to go
>> way back in his memory (June 2005) ;-)

I've sent a patch to cause this part of the test to now be run.

> Probably the last error check should simply be removed, since we expect
> the call to succeed.

AccessCheck can return TRUE but still set the last error. AccessCheck 
only returns FALSE if an invalid parameter is input or there is 
something wrong with the token being checked against. It can return TRUE 
but set last error if access to the object secured by the security 
descriptor was denied.

Rob Shearman

More information about the wine-devel mailing list