[3/4] secur32/tests: Add a test for QueryCredentialsAttributes(SECPKG_CRED_ATTR_NAMES). Take 2.

Dmitry Timoshkov dmitry at baikal.ru
Thu Dec 12 00:21:09 CST 2013


Nikolay Sivov <bunglehead at gmail.com> wrote:

> > +    st = pQueryCredentialsAttributesA(&cred, SECPKG_CRED_ATTR_NAMES, &names);
> > +    ok(st == SEC_E_NO_CREDENTIALS || st == SEC_E_UNSUPPORTED_FUNCTION /* before Vista */, "expected SEC_E_NO_CREDENTIALS, got %08x\n", st);
> > +
> Second possible return code should be broken(), otherwise your code 
> change in next patch won't make any effect on tests.

SEC_E_UNSUPPORTED_FUNCTION is a perfectly valid error code, broken is not
appropriate here. My next patch just changes Wine code to follow more recent
Windows behaviour.

-- 
Dmitry.



More information about the wine-devel mailing list