Jakob Eriksson <jakov(a)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.
--
Alexandre Julliard
julliard(a)winehq.org