Juan Lang : crypt32: CertFindCRLInStore with find type CRL_FIND_ISSUED_FOR shouldn' t check whether the CRL is valid for the subject certificate.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 9d9070ae3cb6ca7a13847fa5ff00663e42af855b
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=9d9070ae3cb6ca7a13847fa5ff...

Author: Juan Lang <juan.lang(a)gmail.com>
Date:   Thu Nov 19 11:50:32 2009 -0800

crypt32: CertFindCRLInStore with find type CRL_FIND_ISSUED_FOR shouldn't check whether the CRL is valid for the subject certificate.

---

 dlls/crypt32/crl.c       |    3 ---
 dlls/crypt32/tests/crl.c |    2 --
 2 files changed, 0 insertions(+), 5 deletions(-)

diff --git a/dlls/crypt32/crl.c b/dlls/crypt32/crl.c
index 5b93d3a..9e7fe9d 100644
--- a/dlls/crypt32/crl.c
+++ b/dlls/crypt32/crl.c
@@ -177,9 +177,6 @@ static BOOL compare_crl_issued_for(PCCRL_CONTEXT pCrlContext, DWORD dwType,
 
     ret = CertCompareCertificateName(para->pIssuerCert->dwCertEncodingType,
      ¶->pIssuerCert->pCertInfo->Issuer, &pCrlContext->pCrlInfo->Issuer);
-    if (ret)
-        ret = CertIsValidCRLForCertificate(para->pSubjectCert, pCrlContext,
-         0, NULL);
     return ret;
 }
 
diff --git a/dlls/crypt32/tests/crl.c b/dlls/crypt32/tests/crl.c
index dfb9356..4a96a46 100644
--- a/dlls/crypt32/tests/crl.c
+++ b/dlls/crypt32/tests/crl.c
@@ -530,7 +530,6 @@ static void testFindCRL(void)
      * match cert's issuer, but verisignCRL does not, so the expected count
      * is 0.
      */
-    todo_wine {
     ok(count == 3 || broken(count == 0 /* NT4, Win9x */),
      "expected 3 matching CRLs, got %d\n", count);
     /* Only v1CRLWithIssuerAndEntry and v2CRLWithIssuingDistPoint contain
@@ -538,7 +537,6 @@ static void testFindCRL(void)
      */
     ok(revoked_count == 2 || broken(revoked_count == 0 /* NT4, Win9x */),
      "expected 2 matching CRL entries, got %d\n", revoked_count);
-    }
 
     CertFreeCertificateContext(cert);