Jakob Eriksson wrote:
Jakob Eriksson wrote:
Alexandre Julliard wrote:
Jakob Eriksson <[EMAIL PROTECTED]> 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
