dlls/advapi32/tests/security.c - Bugfix in test!
Filip Navara
xnavara at volny.cz
Sat Mar 19 08:44:05 CST 2005
Jakob Eriksson wrote:
> Jakob Eriksson wrote:
>
>> Alexandre Julliard wrote:
>>
>>> Jakob Eriksson <jakov at vmlinux.org> writes:
>>>
>>>
>>>
>>>> --- dlls/advapi32/tests/security.c 14 Mar 2005 17:20:58 -0000
>>>> 1.12
>>>> +++ dlls/advapi32/tests/security.c 16 Mar 2005 09:32:28 -0000
>>>> @@ -289,8 +289,8 @@
>>>> luid.LowPart = i;
>>>> cchName = sizeof(buf);
>>>> ret = pLookupPrivilegeNameA(NULL, &luid, buf, &cchName);
>>>> - ok( ret && GetLastError() != ERROR_NO_SUCH_PRIVILEGE,
>>>> - "LookupPrivilegeNameA(0.%ld) failed: %ld\n", i,
>>>> GetLastError());
>>>> + if (GetLastError() != ERROR_NO_SUCH_PRIVILEGE)
>>>> + ok( ret, "LookupPrivilegeNameA(0.%ld) failed: %ld\n",
>>>> i, GetLastError());
>>>>
>>>
>>>
>>>
>>> It doesn't really make sense to check the last error if the function
>>> succeeded.
>>>
>>>
>>
>> True. Don't know what I was thinking.
>
> Now I know. If the error was ERROR_NO_SUCH_PRIVILEGE, it's ok, we
> don't care.
> Move on. NT4 has this behaviour.
What about: "ok( ret || GetLastError() == ERROR_NO_SUCH_PRIVILEGE,"?
- Filip
More information about the wine-devel
mailing list