[try2] advapi32: Implement CredReadDomainCredentials stub and tests

James Hawkins truiken at gmail.com
Thu Oct 9 17:54:21 CDT 2008


On Thu, Oct 9, 2008 at 4:14 PM, Andrey Turkin <andrey.turkin at gmail.com> wrote:
>
> [now with tests, fixed and reformatted as per James' suggestions]
>
> Implement CredReadDomainCredentialsA and CredReadDomainCredentialsW
> stubs and few tests for them.  Required for MSN Messenger 7.0
> ---
>  dlls/advapi32/advapi32.spec |    4 +-
>  dlls/advapi32/cred.c        |  160 +++++++++++++++++++++++++++++++++++++++++++
>  dlls/advapi32/tests/cred.c  |   45 ++++++++++++
>  include/wincred.h           |   39 +++++++++++
>  4 files changed, 246 insertions(+), 2 deletions(-)
>

+    *Size = 0;

+    *Credentials = NULL;

+    if (!TargetInformation)

What if Size or Credentials is NULL?  You should add a test for that
and handle it appropriately.  If the tests crash, you should comment
out the tests and note that there is no handling in Windows.

-- 
James Hawkins



More information about the wine-devel mailing list