Dmitry Timoshkov : crypt32: Fix conversion of CRYPT_KEY_PROV_INFO between store and certificate property.

Alexandre Julliard julliard at winehq.org
Fri Sep 25 14:52:54 CDT 2020


Module: wine
Branch: master
Commit: 7f8f424f38717d6a26a2b55e2ea7582a98dbc419
URL:    https://source.winehq.org/git/wine.git/?a=commit;h=7f8f424f38717d6a26a2b55e2ea7582a98dbc419

Author: Dmitry Timoshkov <dmitry at baikal.ru>
Date:   Tue Sep 15 19:00:11 2020 +0300

crypt32: Fix conversion of CRYPT_KEY_PROV_INFO between store and certificate property.

Store uses relative offsets while certificate property uses pointers.

Signed-off-by: Dmitry Timoshkov <dmitry at baikal.ru>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 dlls/crypt32/cert.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/dlls/crypt32/cert.c b/dlls/crypt32/cert.c
index 13d8df9258..ef871155b9 100644
--- a/dlls/crypt32/cert.c
+++ b/dlls/crypt32/cert.c
@@ -566,7 +566,7 @@ void CRYPT_FixKeyProvInfoPointers(PCRYPT_KEY_PROV_INFO buf)
 
     if (store->pwszContainerName)
     {
-        info.pwszContainerName = (LPWSTR)p;
+        info.pwszContainerName = (LPWSTR)((BYTE *)store + store->pwszContainerName);
         p += (lstrlenW(info.pwszContainerName) + 1) * sizeof(WCHAR);
     }
     else
@@ -574,7 +574,7 @@ void CRYPT_FixKeyProvInfoPointers(PCRYPT_KEY_PROV_INFO buf)
 
     if (store->pwszProvName)
     {
-        info.pwszProvName = (LPWSTR)p;
+        info.pwszProvName = (LPWSTR)((BYTE *)store + store->pwszProvName);
         p += (lstrlenW(info.pwszProvName) + 1) * sizeof(WCHAR);
     }
     else
@@ -705,6 +705,7 @@ static void CRYPT_CopyKeyProvInfo(store_CRYPT_KEY_PROV_INFO *to, const CRYPT_KEY
     to->dwProvType = from->dwProvType;
     to->dwFlags = from->dwFlags;
     to->cProvParam = from->cProvParam;
+    to->rgProvParam = 0;
     to->dwKeySpec = from->dwKeySpec;
 
     for (i = 0; i < to->cProvParam; i++)
@@ -713,6 +714,7 @@ static void CRYPT_CopyKeyProvInfo(store_CRYPT_KEY_PROV_INFO *to, const CRYPT_KEY
         p += sizeof(*param);
 
         param->dwParam = from->rgProvParam[i].dwParam;
+        param->pbData = 0;
         param->cbData = from->rgProvParam[i].cbData;
         param->dwFlags = from->rgProvParam[i].dwFlags;
         memcpy(p, from->rgProvParam[i].pbData, from->rgProvParam[i].cbData);




More information about the wine-cvs mailing list