crypt32: Always return TRUE from CertFreeCRLContext.

This commit is contained in:
Jacek Caban 2013-10-18 10:50:12 +02:00 committed by Alexandre Julliard
parent f329de4df4
commit 351314145f
2 changed files with 12 additions and 6 deletions

View File

@ -363,13 +363,11 @@ PCCRL_CONTEXT WINAPI CertDuplicateCRLContext(PCCRL_CONTEXT pCrlContext)
BOOL WINAPI CertFreeCRLContext(PCCRL_CONTEXT pCrlContext) BOOL WINAPI CertFreeCRLContext(PCCRL_CONTEXT pCrlContext)
{ {
BOOL ret = TRUE;
TRACE("(%p)\n", pCrlContext); TRACE("(%p)\n", pCrlContext);
if (pCrlContext) if (pCrlContext)
ret = Context_Release(&crl_from_ptr(pCrlContext)->base); Context_Release(&crl_from_ptr(pCrlContext)->base);
return ret; return TRUE;
} }
DWORD WINAPI CertEnumCRLContextProperties(PCCRL_CONTEXT pCRLContext, DWORD WINAPI CertEnumCRLContextProperties(PCCRL_CONTEXT pCRLContext,

View File

@ -120,6 +120,7 @@ static void testCreateCRL(void)
static void testDupCRL(void) static void testDupCRL(void)
{ {
PCCRL_CONTEXT context, dupContext; PCCRL_CONTEXT context, dupContext;
BOOL res;
context = CertDuplicateCRLContext(NULL); context = CertDuplicateCRLContext(NULL);
ok(context == NULL, "expected NULL\n"); ok(context == NULL, "expected NULL\n");
@ -128,8 +129,15 @@ static void testDupCRL(void)
dupContext = CertDuplicateCRLContext(context); dupContext = CertDuplicateCRLContext(context);
ok(dupContext != NULL, "expected a context\n"); ok(dupContext != NULL, "expected a context\n");
ok(dupContext == context, "expected identical context addresses\n"); ok(dupContext == context, "expected identical context addresses\n");
CertFreeCRLContext(dupContext);
CertFreeCRLContext(context); res = CertFreeCRLContext(dupContext);
ok(res, "CertFreeCRLContext failed\n");
res = CertFreeCRLContext(context);
ok(res, "CertFreeCRLContext failed\n");
res = CertFreeCRLContext(NULL);
ok(res, "CertFreeCRLContext failed\n");
} }
static void testAddCRL(void) static void testAddCRL(void)