October 2007 - Wine-Commits

Juan Lang : crypt32: Use GetSystemTimeAsFileTime rather than GetSystemTime/ SystemTimeToFileTime.
by Alexandre Julliard Oct. 19, 2007

Oct. 19, 2007

Module: wine Branch: master Commit: cf904c2bc4a314625d6dfb7a581d1e1de8e1bada URL: http://source.winehq.org/git/wine.git/?a=commit;h=cf904c2bc4a314625d6dfb7a5… Author: Juan Lang <juan.lang(a)gmail.com> Date: Thu Oct 18 11:14:12 2007 -0700 crypt32: Use GetSystemTimeAsFileTime rather than GetSystemTime/SystemTimeToFileTime. --- dlls/crypt32/cert.c | 5 +---- dlls/crypt32/crl.c | 5 +---- 2 files changed, 2 insertions(+), 8 deletions(-) diff --git a/dlls/crypt32/cert.c b/dlls/crypt32/cert.c index dcad7fa..2451973 100644 --- a/dlls/crypt32/cert.c +++ b/dlls/crypt32/cert.c @@ -1305,10 +1305,7 @@ LONG WINAPI CertVerifyTimeValidity(LPFILETIME pTimeToVerify, if (!pTimeToVerify) { - SYSTEMTIME sysTime; - - GetSystemTime(&sysTime); - SystemTimeToFileTime(&sysTime, &fileTime); + GetSystemTimeAsFileTime(&fileTime); pTimeToVerify = &fileTime; } if ((ret = CompareFileTime(pTimeToVerify, &pCertInfo->NotBefore)) >= 0) diff --git a/dlls/crypt32/crl.c b/dlls/crypt32/crl.c index bc7f73e..ada06fc 100644 --- a/dlls/crypt32/crl.c +++ b/dlls/crypt32/crl.c @@ -519,10 +519,7 @@ LONG WINAPI CertVerifyCRLTimeValidity(LPFILETIME pTimeToVerify, if (!pTimeToVerify) { - SYSTEMTIME sysTime; - - GetSystemTime(&sysTime); - SystemTimeToFileTime(&sysTime, &fileTime); + GetSystemTimeAsFileTime(&fileTime); pTimeToVerify = &fileTime; } if ((ret = CompareFileTime(pTimeToVerify, &pCrlInfo->ThisUpdate)) >= 0)

1 0

Juan Lang : crypt32: Pass reg store' s key and memory store to CRYPT_RegReadFromReg, rather than passing a private type.
by Alexandre Julliard Oct. 19, 2007

Oct. 19, 2007

Module: wine Branch: master Commit: 4d203fe9028c8e17ab60a794ca9bef0ee04513fb URL: http://source.winehq.org/git/wine.git/?a=commit;h=4d203fe9028c8e17ab60a794c… Author: Juan Lang <juan.lang(a)gmail.com> Date: Thu Oct 18 11:12:32 2007 -0700 crypt32: Pass reg store's key and memory store to CRYPT_RegReadFromReg, rather than passing a private type. --- dlls/crypt32/regstore.c | 22 +++++++++++----------- 1 files changed, 11 insertions(+), 11 deletions(-) diff --git a/dlls/crypt32/regstore.c b/dlls/crypt32/regstore.c index e7de5fd..3754011 100644 --- a/dlls/crypt32/regstore.c +++ b/dlls/crypt32/regstore.c @@ -63,8 +63,8 @@ static const WCHAR CRLsW[] = { 'C','R','L','s',0 }; static const WCHAR CTLsW[] = { 'C','T','L','s',0 }; static const WCHAR BlobW[] = { 'B','l','o','b',0 }; -static void CRYPT_RegReadSerializedFromReg(const WINE_REGSTOREINFO *store, HKEY key, - DWORD contextType) +static void CRYPT_RegReadSerializedFromReg(HKEY key, DWORD contextType, + HCERTSTORE store) { LONG rc; DWORD index = 0; @@ -136,7 +136,7 @@ static void CRYPT_RegReadSerializedFromReg(const WINE_REGSTOREINFO *store, HKEY { TRACE("hash matches, adding\n"); contextInterface->addContextToStore( - store->memStore, context, + store, context, CERT_STORE_ADD_REPLACE_EXISTING, NULL); } else @@ -156,7 +156,7 @@ static void CRYPT_RegReadSerializedFromReg(const WINE_REGSTOREINFO *store, HKEY } while (!rc); } -static void CRYPT_RegReadFromReg(const WINE_REGSTOREINFO *store) +static void CRYPT_RegReadFromReg(HKEY key, HCERTSTORE store) { static const WCHAR * const subKeys[] = { CertsW, CRLsW, CTLsW }; static const DWORD contextFlags[] = { CERT_STORE_CERTIFICATE_CONTEXT_FLAG, @@ -165,15 +165,15 @@ static void CRYPT_RegReadFromReg(const WINE_REGSTOREINFO *store) for (i = 0; i < sizeof(subKeys) / sizeof(subKeys[0]); i++) { - HKEY key; + HKEY hKey; LONG rc; - rc = RegCreateKeyExW(store->key, subKeys[i], 0, NULL, 0, KEY_READ, NULL, - &key, NULL); + rc = RegCreateKeyExW(key, subKeys[i], 0, NULL, 0, KEY_READ, NULL, + &hKey, NULL); if (!rc) { - CRYPT_RegReadSerializedFromReg(store, key, contextFlags[i]); - RegCloseKey(key); + CRYPT_RegReadSerializedFromReg(hKey, contextFlags[i], store); + RegCloseKey(hKey); } } } @@ -444,7 +444,7 @@ static BOOL WINAPI CRYPT_RegControl(HCERTSTORE hCertStore, DWORD dwFlags, case CERT_STORE_CTRL_RESYNC: CRYPT_RegFlushStore(store, FALSE); CRYPT_EmptyStore(store->memStore); - CRYPT_RegReadFromReg(store); + CRYPT_RegReadFromReg(store->key, store->memStore); ret = TRUE; break; case CERT_STORE_CTRL_COMMIT: @@ -523,7 +523,7 @@ PWINECRYPT_CERTSTORE CRYPT_RegOpenStore(HCRYPTPROV hCryptProv, DWORD dwFlags, regInfo->cs.DebugInfo->Spare[0] = (DWORD_PTR)(__FILE__ ": PWINE_REGSTOREINFO->cs"); list_init(&regInfo->certsToDelete); list_init(&regInfo->crlsToDelete); - CRYPT_RegReadFromReg(regInfo); + CRYPT_RegReadFromReg(regInfo->key, regInfo->memStore); regInfo->dirty = FALSE; provInfo.cbSize = sizeof(provInfo); provInfo.cStoreProvFunc = sizeof(regProvFuncs) /

1 0

Juan Lang : crypt32: Don' t keep a pointer to the lower quality chains when choosing a higher quality one , otherwise they'll get double-freed.
by Alexandre Julliard Oct. 19, 2007

Oct. 19, 2007

Module: wine Branch: master Commit: 5d6feab0e2aba6f71679df24f46c0251e63f2f7b URL: http://source.winehq.org/git/wine.git/?a=commit;h=5d6feab0e2aba6f71679df24f… Author: Juan Lang <juan.lang(a)gmail.com> Date: Thu Oct 18 20:44:18 2007 -0700 crypt32: Don't keep a pointer to the lower quality chains when choosing a higher quality one, otherwise they'll get double-freed. --- dlls/crypt32/chain.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/dlls/crypt32/chain.c b/dlls/crypt32/chain.c index ff555bb..9832f0e 100644 --- a/dlls/crypt32/chain.c +++ b/dlls/crypt32/chain.c @@ -1272,6 +1272,8 @@ static PCertificateChain CRYPT_ChooseHighestQualityChain( chain->context.rgpLowerQualityChainContext; alternate->context.rgpLowerQualityChainContext[i] = (PCCERT_CHAIN_CONTEXT)chain; + chain->context.cLowerQualityChainContext = 0; + chain->context.rgpLowerQualityChainContext = NULL; chain = alternate; } }

1 0

Juan Lang : crypt32: Use I_CertUpdateStore in registry stores.
by Alexandre Julliard Oct. 19, 2007

Oct. 19, 2007

Module: wine Branch: master Commit: abc2aec88c98fffbebfb94d797eb7ad30c26ebc9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=abc2aec88c98fffbebfb94d79… Author: Juan Lang <juan.lang(a)gmail.com> Date: Thu Oct 18 11:13:20 2007 -0700 crypt32: Use I_CertUpdateStore in registry stores. --- dlls/crypt32/crypt32_private.h | 1 - dlls/crypt32/regstore.c | 10 ++++++++-- dlls/crypt32/store.c | 2 +- 3 files changed, 9 insertions(+), 4 deletions(-) diff --git a/dlls/crypt32/crypt32_private.h b/dlls/crypt32/crypt32_private.h index 63d7c2d..8975522 100644 --- a/dlls/crypt32/crypt32_private.h +++ b/dlls/crypt32/crypt32_private.h @@ -233,7 +233,6 @@ typedef struct WINE_CRYPTCERTSTORE void CRYPT_InitStore(WINECRYPT_CERTSTORE *store, DWORD dwFlags, CertStoreType type); void CRYPT_FreeStore(PWINECRYPT_CERTSTORE store); -void CRYPT_EmptyStore(HCERTSTORE store); BOOL WINAPI I_CertUpdateStore(HCERTSTORE store1, HCERTSTORE store2, DWORD unk0, DWORD unk1); diff --git a/dlls/crypt32/regstore.c b/dlls/crypt32/regstore.c index 3754011..35928b8 100644 --- a/dlls/crypt32/regstore.c +++ b/dlls/crypt32/regstore.c @@ -442,11 +442,17 @@ static BOOL WINAPI CRYPT_RegControl(HCERTSTORE hCertStore, DWORD dwFlags, switch (dwCtrlType) { case CERT_STORE_CTRL_RESYNC: + { + HCERTSTORE memStore = CertOpenStore(CERT_STORE_PROV_MEMORY, 0, 0, + CERT_STORE_CREATE_NEW_FLAG, NULL); + CRYPT_RegFlushStore(store, FALSE); - CRYPT_EmptyStore(store->memStore); - CRYPT_RegReadFromReg(store->key, store->memStore); + CRYPT_RegReadFromReg(store->key, memStore); + I_CertUpdateStore(store->memStore, memStore, 0, 0); + CertCloseStore(memStore, 0); ret = TRUE; break; + } case CERT_STORE_CTRL_COMMIT: ret = CRYPT_RegFlushStore(store, dwFlags & CERT_STORE_CTRL_COMMIT_FORCE_FLAG); diff --git a/dlls/crypt32/store.c b/dlls/crypt32/store.c index 28c37d2..57a2bd8 100644 --- a/dlls/crypt32/store.c +++ b/dlls/crypt32/store.c @@ -199,7 +199,7 @@ static BOOL CRYPT_MemDeleteCrl(PWINECRYPT_CERTSTORE store, void *pCrlContext) return TRUE; } -void CRYPT_EmptyStore(HCERTSTORE store) +static void CRYPT_EmptyStore(HCERTSTORE store) { PCCERT_CONTEXT cert; PCCRL_CONTEXT crl;

1 0

Juan Lang : crypt32: Implement I_CertUpdateStore.
by Alexandre Julliard Oct. 19, 2007

Oct. 19, 2007

Module: wine Branch: master Commit: e857f383a428f07e7b0f1e0dbbec7ffa62d5e54b URL: http://source.winehq.org/git/wine.git/?a=commit;h=e857f383a428f07e7b0f1e0db… Author: Juan Lang <juan.lang(a)gmail.com> Date: Thu Oct 18 11:10:55 2007 -0700 crypt32: Implement I_CertUpdateStore. --- dlls/crypt32/main.c | 10 ---------- dlls/crypt32/store.c | 32 ++++++++++++++++++++++++++++++++ dlls/crypt32/tests/store.c | 11 ----------- 3 files changed, 32 insertions(+), 21 deletions(-) diff --git a/dlls/crypt32/main.c b/dlls/crypt32/main.c index fa8b782..12f2885 100644 --- a/dlls/crypt32/main.c +++ b/dlls/crypt32/main.c @@ -60,16 +60,6 @@ HCRYPTPROV CRYPT_GetDefaultProvider(void) return hDefProv; } -/* Appears to be called to update the contents of store1 with those in store2. - * The second two parameters are unknown. - */ -BOOL WINAPI I_CertUpdateStore(HCERTSTORE store1, HCERTSTORE store2, DWORD unk0, - DWORD unk1) -{ - FIXME("(%p, %p, %08x, %08x)\n", store1, store2, unk0, unk1); - return FALSE; -} - typedef void * HLRUCACHE; /* this function is called by Internet Explorer when it is about to verify a diff --git a/dlls/crypt32/store.c b/dlls/crypt32/store.c index c32ae62..28c37d2 100644 --- a/dlls/crypt32/store.c +++ b/dlls/crypt32/store.c @@ -216,6 +216,38 @@ void CRYPT_EmptyStore(HCERTSTORE store) } while (crl); } +BOOL WINAPI I_CertUpdateStore(HCERTSTORE store1, HCERTSTORE store2, DWORD unk0, + DWORD unk1) +{ + static BOOL warned = FALSE; + PCCERT_CONTEXT cert = NULL; + PCCRL_CONTEXT crl = NULL; + + TRACE("(%p, %p, %08x, %08x)\n", store1, store2, unk0, unk1); + if (!warned) + { + FIXME("semi-stub\n"); + warned = TRUE; + } + + /* Poor-man's resync: empty first store, then add everything from second + * store to it. + */ + CRYPT_EmptyStore(store1); + do { + cert = CertEnumCertificatesInStore(store2, cert); + if (cert) + CertAddCertificateContextToStore(store1, cert, + CERT_STORE_ADD_ALWAYS, NULL); + } while (cert); + do { + crl = CertEnumCRLsInStore(store2, crl); + if (crl) + CertAddCRLContextToStore(store1, crl, CERT_STORE_ADD_ALWAYS, NULL); + } while (crl); + return TRUE; +} + static void WINAPI CRYPT_MemCloseStore(HCERTSTORE hCertStore, DWORD dwFlags) { WINE_MEMSTORE *store = (WINE_MEMSTORE *)hCertStore; diff --git a/dlls/crypt32/tests/store.c b/dlls/crypt32/tests/store.c index 99fac18..05ea5e0 100644 --- a/dlls/crypt32/tests/store.c +++ b/dlls/crypt32/tests/store.c @@ -1840,32 +1840,25 @@ static void test_I_UpdateStore(void) ret = pI_CertUpdatestore(NULL, store2, 0, 0); */ ret = pI_CertUpdatestore(store1, store2, 0, 0); - todo_wine ok(ret, "I_CertUpdateStore failed: %08x\n", GetLastError()); CertAddEncodedCertificateToStore(store2, X509_ASN_ENCODING, bigCert, sizeof(bigCert), CERT_STORE_ADD_ALWAYS, &cert); /* I_CertUpdateStore adds the contexts from store2 to store1 */ ret = pI_CertUpdatestore(store1, store2, 0, 0); - todo_wine ok(ret, "I_CertUpdateStore failed: %08x\n", GetLastError()); certs = countCertsInStore(store1); - todo_wine ok(certs == 1, "Expected 1 cert, got %d\n", certs); /* Calling it a second time has no effect */ ret = pI_CertUpdatestore(store1, store2, 0, 0); - todo_wine ok(ret, "I_CertUpdateStore failed: %08x\n", GetLastError()); certs = countCertsInStore(store1); - todo_wine ok(certs == 1, "Expected 1 cert, got %d\n", certs); /* The last parameters to I_CertUpdateStore appear to be ignored */ ret = pI_CertUpdatestore(store1, store2, 1, 0); - todo_wine ok(ret, "I_CertUpdateStore failed: %08x\n", GetLastError()); ret = pI_CertUpdatestore(store1, store2, 0, 1); - todo_wine ok(ret, "I_CertUpdateStore failed: %08x\n", GetLastError()); CertAddEncodedCRLToStore(store2, X509_ASN_ENCODING, signedCRL, @@ -1873,13 +1866,10 @@ static void test_I_UpdateStore(void) /* I_CertUpdateStore also adds the CRLs from store2 to store1 */ ret = pI_CertUpdatestore(store1, store2, 0, 0); - todo_wine ok(ret, "I_CertUpdateStore failed: %08x\n", GetLastError()); certs = countCertsInStore(store1); - todo_wine ok(certs == 1, "Expected 1 cert, got %d\n", certs); certs = countCRLsInStore(store1); - todo_wine ok(certs == 1, "Expected 1 CRL, got %d\n", certs); CertDeleteCertificateFromStore(cert); @@ -1887,7 +1877,6 @@ static void test_I_UpdateStore(void) * from store1 */ ret = pI_CertUpdatestore(store1, store2, 0, 0); - todo_wine ok(ret, "I_CertUpdateStore failed: %08x\n", GetLastError()); certs = countCertsInStore(store1); ok(certs == 0, "Expected 0 certs, got %d\n", certs);

1 0

Juan Lang : crypt32: Use I_CertUpdateStore in file stores.
by Alexandre Julliard Oct. 19, 2007

Oct. 19, 2007

Module: wine Branch: master Commit: 889449630dcba07351a58b07cb874974ed34f68f URL: http://source.winehq.org/git/wine.git/?a=commit;h=889449630dcba07351a58b07c… Author: Juan Lang <juan.lang(a)gmail.com> Date: Thu Oct 18 11:11:54 2007 -0700 crypt32: Use I_CertUpdateStore in file stores. --- dlls/crypt32/crypt32_private.h | 2 ++ dlls/crypt32/filestore.c | 35 ++++++++++++++--------------------- 2 files changed, 16 insertions(+), 21 deletions(-) diff --git a/dlls/crypt32/crypt32_private.h b/dlls/crypt32/crypt32_private.h index c0246d0..63d7c2d 100644 --- a/dlls/crypt32/crypt32_private.h +++ b/dlls/crypt32/crypt32_private.h @@ -234,6 +234,8 @@ void CRYPT_InitStore(WINECRYPT_CERTSTORE *store, DWORD dwFlags, CertStoreType type); void CRYPT_FreeStore(PWINECRYPT_CERTSTORE store); void CRYPT_EmptyStore(HCERTSTORE store); +BOOL WINAPI I_CertUpdateStore(HCERTSTORE store1, HCERTSTORE store2, DWORD unk0, + DWORD unk1); PWINECRYPT_CERTSTORE CRYPT_CollectionOpenStore(HCRYPTPROV hCryptProv, DWORD dwFlags, const void *pvPara); diff --git a/dlls/crypt32/filestore.c b/dlls/crypt32/filestore.c index 8294e82..4c8c4a1 100644 --- a/dlls/crypt32/filestore.c +++ b/dlls/crypt32/filestore.c @@ -118,11 +118,20 @@ static BOOL WINAPI CRYPT_FileControl(HCERTSTORE hCertStore, DWORD dwFlags, switch (dwCtrlType) { case CERT_STORE_CTRL_RESYNC: - CRYPT_EmptyStore(store->memStore); store->dirty = FALSE; if (store->type == CERT_STORE_SAVE_AS_STORE) - ret = CRYPT_ReadSerializedStoreFromFile(store->file, - store->memStore); + { + HCERTSTORE memStore = CertOpenStore(CERT_STORE_PROV_MEMORY, 0, 0, + CERT_STORE_CREATE_NEW_FLAG, NULL); + + /* FIXME: if I could translate a handle to a path, I could use + * CryptQueryObject instead, but there's no API to do so yet. + */ + ret = CRYPT_ReadSerializedStoreFromFile(store->file, memStore); + if (ret) + I_CertUpdateStore(store->memStore, memStore, 0, 0); + CertCloseStore(memStore, 0); + } else if (store->type == CERT_STORE_SAVE_AS_PKCS7) { CERT_BLOB blob = { 0, NULL }; @@ -136,24 +145,8 @@ static BOOL WINAPI CRYPT_FileControl(HCERTSTORE hCertStore, DWORD dwFlags, CERT_QUERY_CONTENT_FLAG_PKCS7_SIGNED, CERT_QUERY_FORMAT_FLAG_BINARY, 0, NULL, NULL, NULL, &messageStore, NULL, NULL); - if (ret) - { - PCCERT_CONTEXT cert = NULL; - PCCRL_CONTEXT crl = NULL; - - do { - cert = CertEnumCertificatesInStore(messageStore, cert); - if (cert) - CertAddCertificateContextToStore(store->memStore, - cert, CERT_STORE_ADD_ALWAYS, NULL); - } while (cert); - do { - crl = CertEnumCRLsInStore(messageStore, crl); - if (crl) - CertAddCRLContextToStore(store->memStore, crl, - CERT_STORE_ADD_ALWAYS, NULL); - } while (crl); - } + I_CertUpdateStore(store->memStore, messageStore, 0, 0); + CertCloseStore(messageStore, 0); CryptMemFree(blob.pbData); } }

1 0

Juan Lang : crypt32: Add tests for I_CertUpdateStore.
by Alexandre Julliard Oct. 19, 2007

Oct. 19, 2007

Module: wine Branch: master Commit: 542af8aeeb6d023f8b27b4b12e3cf9cddd2cfb41 URL: http://source.winehq.org/git/wine.git/?a=commit;h=542af8aeeb6d023f8b27b4b12… Author: Juan Lang <juan.lang(a)gmail.com> Date: Thu Oct 18 11:10:23 2007 -0700 crypt32: Add tests for I_CertUpdateStore. --- dlls/crypt32/tests/store.c | 111 ++++++++++++++++++++++++++++++++++++++++++++ 1 files changed, 111 insertions(+), 0 deletions(-) diff --git a/dlls/crypt32/tests/store.c b/dlls/crypt32/tests/store.c index b14dbc2..99fac18 100644 --- a/dlls/crypt32/tests/store.c +++ b/dlls/crypt32/tests/store.c @@ -1788,6 +1788,115 @@ static void testAddSerialized(void) CertCloseStore(store, 0); } +static DWORD countCertsInStore(HCERTSTORE store) +{ + PCCERT_CONTEXT cert = NULL; + DWORD certs = 0; + + do { + cert = CertEnumCertificatesInStore(store, cert); + if (cert) + certs++; + } while (cert); + return certs; +} + +static DWORD countCRLsInStore(HCERTSTORE store) +{ + PCCRL_CONTEXT crl = NULL; + DWORD crls = 0; + + do { + crl = CertEnumCRLsInStore(store, crl); + if (crl) + crls++; + } while (crl); + return crls; +} + +static void test_I_UpdateStore(void) +{ + HMODULE lib = GetModuleHandleA("crypt32"); + BOOL (WINAPI *pI_CertUpdatestore)(HCERTSTORE, HCERTSTORE, DWORD, DWORD) = + (void *)GetProcAddress(lib, "I_CertUpdateStore"); + BOOL ret; + HCERTSTORE store1, store2; + PCCERT_CONTEXT cert; + DWORD certs; + + if (!pI_CertUpdatestore) + { + skip("No I_CertUpdateStore\n"); + return; + } + store1 = CertOpenStore(CERT_STORE_PROV_MEMORY, 0, 0, + CERT_STORE_CREATE_NEW_FLAG, NULL); + store2 = CertOpenStore(CERT_STORE_PROV_MEMORY, 0, 0, + CERT_STORE_CREATE_NEW_FLAG, NULL); + + /* Crash + ret = pI_CertUpdatestore(NULL, NULL, 0, 0); + ret = pI_CertUpdatestore(store1, NULL, 0, 0); + ret = pI_CertUpdatestore(NULL, store2, 0, 0); + */ + ret = pI_CertUpdatestore(store1, store2, 0, 0); + todo_wine + ok(ret, "I_CertUpdateStore failed: %08x\n", GetLastError()); + + CertAddEncodedCertificateToStore(store2, X509_ASN_ENCODING, bigCert, + sizeof(bigCert), CERT_STORE_ADD_ALWAYS, &cert); + /* I_CertUpdateStore adds the contexts from store2 to store1 */ + ret = pI_CertUpdatestore(store1, store2, 0, 0); + todo_wine + ok(ret, "I_CertUpdateStore failed: %08x\n", GetLastError()); + certs = countCertsInStore(store1); + todo_wine + ok(certs == 1, "Expected 1 cert, got %d\n", certs); + /* Calling it a second time has no effect */ + ret = pI_CertUpdatestore(store1, store2, 0, 0); + todo_wine + ok(ret, "I_CertUpdateStore failed: %08x\n", GetLastError()); + certs = countCertsInStore(store1); + todo_wine + ok(certs == 1, "Expected 1 cert, got %d\n", certs); + + /* The last parameters to I_CertUpdateStore appear to be ignored */ + ret = pI_CertUpdatestore(store1, store2, 1, 0); + todo_wine + ok(ret, "I_CertUpdateStore failed: %08x\n", GetLastError()); + ret = pI_CertUpdatestore(store1, store2, 0, 1); + todo_wine + ok(ret, "I_CertUpdateStore failed: %08x\n", GetLastError()); + + CertAddEncodedCRLToStore(store2, X509_ASN_ENCODING, signedCRL, + sizeof(signedCRL), CERT_STORE_ADD_ALWAYS, NULL); + + /* I_CertUpdateStore also adds the CRLs from store2 to store1 */ + ret = pI_CertUpdatestore(store1, store2, 0, 0); + todo_wine + ok(ret, "I_CertUpdateStore failed: %08x\n", GetLastError()); + certs = countCertsInStore(store1); + todo_wine + ok(certs == 1, "Expected 1 cert, got %d\n", certs); + certs = countCRLsInStore(store1); + todo_wine + ok(certs == 1, "Expected 1 CRL, got %d\n", certs); + + CertDeleteCertificateFromStore(cert); + /* If a context is deleted from store2, I_CertUpdateStore delets it + * from store1 + */ + ret = pI_CertUpdatestore(store1, store2, 0, 0); + todo_wine + ok(ret, "I_CertUpdateStore failed: %08x\n", GetLastError()); + certs = countCertsInStore(store1); + ok(certs == 0, "Expected 0 certs, got %d\n", certs); + + CertFreeCertificateContext(cert); + CertCloseStore(store1, 0); + CertCloseStore(store2, 0); +} + START_TEST(store) { /* various combinations of CertOpenStore */ @@ -1805,4 +1914,6 @@ START_TEST(store) testStoreProperty(); testAddSerialized(); + + test_I_UpdateStore(); }

1 0

Juan Lang : cryptnet: Add URL_OID_GET_OBJECT_URL_FUNC definition.
by Alexandre Julliard Oct. 19, 2007

Oct. 19, 2007

Module: wine Branch: master Commit: 72688bd239641c1bbfbb4ade3b1c5f51bc06313f URL: http://source.winehq.org/git/wine.git/?a=commit;h=72688bd239641c1bbfbb4ade3… Author: Juan Lang <juan.lang(a)gmail.com> Date: Thu Oct 18 10:28:59 2007 -0700 cryptnet: Add URL_OID_GET_OBJECT_URL_FUNC definition. --- include/wincrypt.h | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/include/wincrypt.h b/include/wincrypt.h index 7563051..e45a38b 100644 --- a/include/wincrypt.h +++ b/include/wincrypt.h @@ -1216,6 +1216,8 @@ typedef struct _CRYPT_URL_INFO { #define URL_OID_CRL_FRESHEST_CRL ((LPCSTR)7) #define URL_OID_CROSS_CERT_DIST_POINT ((LPCSTR)8) +#define URL_OID_GET_OBJECT_URL_FUNC "UrlDllGetObjectUrl" + typedef HANDLE HCRYPTASYNC, *PHCRYPTASYNC; typedef void (WINAPI *PFN_CRYPT_ASYNC_PARAM_FREE_FUNC)(LPSTR pszParamOid,

1 0

Juan Lang : cryptnet: Add stub for CryptGetObjectUrl.
by Alexandre Julliard Oct. 19, 2007

Oct. 19, 2007

Module: wine Branch: master Commit: 016a26ddae0d0341e7e058e092d1b48dcbaa53ce URL: http://source.winehq.org/git/wine.git/?a=commit;h=016a26ddae0d0341e7e058e09… Author: Juan Lang <juan.lang(a)gmail.com> Date: Thu Oct 18 10:20:02 2007 -0700 cryptnet: Add stub for CryptGetObjectUrl. --- dlls/cryptnet/cryptnet.spec | 2 +- dlls/cryptnet/cryptnet_main.c | 12 ++++++++++++ 2 files changed, 13 insertions(+), 1 deletions(-) diff --git a/dlls/cryptnet/cryptnet.spec b/dlls/cryptnet/cryptnet.spec index 9d89c15..bcf35e6 100644 --- a/dlls/cryptnet/cryptnet.spec +++ b/dlls/cryptnet/cryptnet.spec @@ -4,7 +4,7 @@ @ stub LdapProvOpenStore @ stub CryptCancelAsyncRetrieval @ stub CryptFlushTimeValidObject -@ stub CryptGetObjectUrl +@ stdcall CryptGetObjectUrl(ptr ptr long ptr ptr ptr ptr ptr) @ stub CryptGetTimeValidObject @ stub CryptInstallCancelRetrieval @ stub CryptRetrieveObjectByUrlA diff --git a/dlls/cryptnet/cryptnet_main.c b/dlls/cryptnet/cryptnet_main.c index d8f072e..67a1ebb 100644 --- a/dlls/cryptnet/cryptnet_main.c +++ b/dlls/cryptnet/cryptnet_main.c @@ -79,6 +79,18 @@ HRESULT WINAPI DllUnregisterServer(void) } /*********************************************************************** + * CryptGetObjectUrl (CRYPTNET.@) + */ +BOOL WINAPI CryptGetObjectUrl(LPCSTR pszUrlOid, LPVOID pvPara, DWORD dwFlags, + PCRYPT_URL_ARRAY pUrlArray, DWORD *pcbUrlArray, PCRYPT_URL_INFO pUrlInfo, + DWORD *pcbUrlInfo, LPVOID pvReserved) +{ + FIXME("(%s, %p, %08x, %p, %p, %p, %p, %p): stub\n", debugstr_a(pszUrlOid), + pvPara, dwFlags, pUrlArray, pcbUrlArray, pUrlInfo, pcbUrlInfo, pvReserved); + return FALSE; +} + +/*********************************************************************** * CertDllVerifyRevocation (CRYPTNET.@) */ BOOL WINAPI CertDllVerifyRevocation(DWORD dwEncodingType, DWORD dwRevType,

1 0

Juan Lang : cryptnet: Add CertDllVerifyRevocation stub.
by Alexandre Julliard Oct. 19, 2007

Oct. 19, 2007

Module: wine Branch: master Commit: 6d431c7337f6b010f82b633464bea04d91b292a9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=6d431c7337f6b010f82b63346… Author: Juan Lang <juan.lang(a)gmail.com> Date: Thu Oct 18 10:19:12 2007 -0700 cryptnet: Add CertDllVerifyRevocation stub. --- dlls/cryptnet/cryptnet.spec | 2 +- dlls/cryptnet/cryptnet_main.c | 12 ++++++++++++ 2 files changed, 13 insertions(+), 1 deletions(-) diff --git a/dlls/cryptnet/cryptnet.spec b/dlls/cryptnet/cryptnet.spec index 62a8823..9d89c15 100644 --- a/dlls/cryptnet/cryptnet.spec +++ b/dlls/cryptnet/cryptnet.spec @@ -1,5 +1,5 @@ @ stub CertDllVerifyCTLUsage -@ stub CertDllVerifyRevocation +@ stdcall CertDllVerifyRevocation(long long long ptr long ptr ptr) @ stub CryptnetWlxLogoffEvent @ stub LdapProvOpenStore @ stub CryptCancelAsyncRetrieval diff --git a/dlls/cryptnet/cryptnet_main.c b/dlls/cryptnet/cryptnet_main.c index 17b62f0..d8f072e 100644 --- a/dlls/cryptnet/cryptnet_main.c +++ b/dlls/cryptnet/cryptnet_main.c @@ -77,3 +77,15 @@ HRESULT WINAPI DllUnregisterServer(void) CERT_STORE_PROV_LDAP_W); return S_OK; } + +/*********************************************************************** + * CertDllVerifyRevocation (CRYPTNET.@) + */ +BOOL WINAPI CertDllVerifyRevocation(DWORD dwEncodingType, DWORD dwRevType, + DWORD cContext, void *rgpvContext[], DWORD dwFlags, + PCERT_REVOCATION_PARA pRevPara, PCERT_REVOCATION_STATUS pRevStatus) +{ + FIXME("(%08x, %d, %d, %p, %08x, %p, %p): stub\n", dwEncodingType, dwRevType, + cContext, rgpvContext, dwFlags, pRevPara, pRevStatus); + return FALSE; +}

1 0