[PATCH 0/1] MR8171: wintrust: Initialize all cert fields in WINTRUST_AddCert.

[Alex Henrie (＠alexhenrie)]

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=58293

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/8171

[PATCH 1/1] wintrust: Initialize all cert fields in WINTRUST_AddCert.

[Alex Henrie]

From: Alex Henrie <alexhenrie24(a)gmail.com>

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=58293
---
 dlls/wintrust/wintrust_main.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/dlls/wintrust/wintrust_main.c b/dlls/wintrust/wintrust_main.c
index 66e274a24ae..22921006d4a 100644
--- a/dlls/wintrust/wintrust_main.c
+++ b/dlls/wintrust/wintrust_main.c
@@ -1058,6 +1058,7 @@ BOOL WINAPI WINTRUST_AddCert(CRYPT_PROVIDER_DATA *data, DWORD idxSigner,
         CRYPT_PROVIDER_CERT *cert = &data->pasSigners[idxSigner].pasCertChain[
          data->pasSigners[idxSigner].csCertChain];
 
+        memset(cert, 0, sizeof(*cert));
         cert->cbStruct = sizeof(CRYPT_PROVIDER_CERT);
         cert->pCert = CertDuplicateCertificateContext(pCert2Add);
         data->pasSigners[idxSigner].csCertChain++;
-- 
GitLab

https://gitlab.winehq.org/wine/wine/-/merge_requests/8171

[Tyson Whitehead (＠twhitehead)]

Tried it, and it resolves the issue. :thumbsup:

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/8171#note_104953