November 2007 - Wine-commits

Juan Lang : rsaenh: Use helper function to read key pairs from registry.
by Alexandre Julliard 07 Nov '07

07 Nov '07

Module: wine Branch: master Commit: 5408471b23e903c308c256284cf16c367c75b9f2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=5408471b23e903c308c256284… Author: Juan Lang <juan.lang(a)gmail.com> Date: Tue Nov 6 13:05:23 2007 -0800 rsaenh: Use helper function to read key pairs from registry. --- dlls/rsaenh/rsaenh.c | 106 +++++++++++++++++++++++++------------------------ 1 files changed, 54 insertions(+), 52 deletions(-) diff --git a/dlls/rsaenh/rsaenh.c b/dlls/rsaenh/rsaenh.c index 4c154c1..2d89751 100644 --- a/dlls/rsaenh/rsaenh.c +++ b/dlls/rsaenh/rsaenh.c @@ -1100,6 +1100,51 @@ static HCRYPTPROV new_key_container(PCCH pszContainerName, DWORD dwFlags, const } /****************************************************************************** + * read_key_value [Internal] + * + * Reads a key pair value from the registry + * + * PARAMS + * hKeyContainer [I] Crypt provider to use to import the key + * hKey [I] Registry key from which to read the key pair + * szValueName [I] Registry value from which to read the key pair's value + * dwFlags [I] Flags for unprotecting the key + * phCryptKey [O] Returned key + */ +static BOOL read_key_value(HCRYPTPROV hKeyContainer, HKEY hKey, LPCSTR szValueName, DWORD dwFlags, HCRYPTKEY *phCryptKey) +{ + DWORD dwValueType, dwLen; + BYTE *pbKey; + DATA_BLOB blobIn, blobOut; + BOOL ret = FALSE; + + if (RegQueryValueExA(hKey, szValueName, 0, &dwValueType, NULL, &dwLen) == + ERROR_SUCCESS) + { + pbKey = HeapAlloc(GetProcessHeap(), 0, dwLen); + if (pbKey) + { + if (RegQueryValueExA(hKey, szValueName, 0, &dwValueType, pbKey, &dwLen) == + ERROR_SUCCESS) + { + blobIn.pbData = pbKey; + blobIn.cbData = dwLen; + + if (CryptUnprotectData(&blobIn, NULL, NULL, NULL, NULL, + dwFlags, &blobOut)) + { + ret = RSAENH_CPImportKey(hKeyContainer, blobOut.pbData, blobOut.cbData, 0, 0, + phCryptKey); + HeapFree(GetProcessHeap(), 0, blobOut.pbData); + } + } + HeapFree(GetProcessHeap(), 0, pbKey); + } + } + return ret; +} + +/****************************************************************************** * read_key_container [Internal] * * Tries to read the persistent state of the key container (mainly the signature @@ -1115,12 +1160,9 @@ static HCRYPTPROV new_key_container(PCCH pszContainerName, DWORD dwFlags, const */ static HCRYPTPROV read_key_container(PCHAR pszContainerName, DWORD dwFlags, const VTableProvStruc *pVTable) { - BYTE *pbKey; HKEY hKey; - DWORD dwValueType, dwLen; KEYCONTAINER *pKeyContainer; HCRYPTPROV hKeyContainer; - DATA_BLOB blobIn, blobOut; HCRYPTKEY hCryptKey; if (!open_container_key(pszContainerName, dwFlags, &hKey)) @@ -1132,59 +1174,19 @@ static HCRYPTPROV read_key_container(PCHAR pszContainerName, DWORD dwFlags, cons hKeyContainer = new_key_container(pszContainerName, dwFlags, pVTable); if (hKeyContainer != (HCRYPTPROV)INVALID_HANDLE_VALUE) { + DWORD dwProtectFlags = (dwFlags & CRYPT_MACHINE_KEYSET) ? + CRYPTPROTECT_LOCAL_MACHINE : 0; + if (!lookup_handle(&handle_table, hKeyContainer, RSAENH_MAGIC_CONTAINER, (OBJECTHDR**)&pKeyContainer)) return (HCRYPTPROV)INVALID_HANDLE_VALUE; - if (RegQueryValueExA(hKey, "KeyExchangeKeyPair", 0, &dwValueType, NULL, &dwLen) == - ERROR_SUCCESS) - { - pbKey = HeapAlloc(GetProcessHeap(), 0, dwLen); - if (pbKey) - { - if (RegQueryValueExA(hKey, "KeyExchangeKeyPair", 0, &dwValueType, pbKey, &dwLen) == - ERROR_SUCCESS) - { - blobIn.pbData = pbKey; - blobIn.cbData = dwLen; - - if (CryptUnprotectData(&blobIn, NULL, NULL, NULL, NULL, - (dwFlags & CRYPT_MACHINE_KEYSET) ? CRYPTPROTECT_LOCAL_MACHINE : 0, &blobOut)) - { - if(RSAENH_CPImportKey(hKeyContainer, blobOut.pbData, blobOut.cbData, 0, 0, - &hCryptKey)) - pKeyContainer->hKeyExchangeKeyPair = hCryptKey; - HeapFree(GetProcessHeap(), 0, blobOut.pbData); - } - } - HeapFree(GetProcessHeap(), 0, pbKey); - } - } - - if (RegQueryValueExA(hKey, "SignatureKeyPair", 0, &dwValueType, NULL, &dwLen) == - ERROR_SUCCESS) - { - pbKey = HeapAlloc(GetProcessHeap(), 0, dwLen); - if (pbKey) - { - if (RegQueryValueExA(hKey, "SignatureKeyPair", 0, &dwValueType, pbKey, &dwLen) == - ERROR_SUCCESS) - { - blobIn.pbData = pbKey; - blobIn.cbData = dwLen; - - if (CryptUnprotectData(&blobIn, NULL, NULL, NULL, NULL, - (dwFlags & CRYPT_MACHINE_KEYSET) ? CRYPTPROTECT_LOCAL_MACHINE : 0, &blobOut)) - { - if(RSAENH_CPImportKey(hKeyContainer, blobOut.pbData, blobOut.cbData, 0, 0, - &hCryptKey)) - pKeyContainer->hSignatureKeyPair = hCryptKey; - HeapFree(GetProcessHeap(), 0, blobOut.pbData); - } - } - HeapFree(GetProcessHeap(), 0, pbKey); - } - } + if (read_key_value(hKeyContainer, hKey, "KeyExchangeKeyPair", + dwProtectFlags, &hCryptKey)) + pKeyContainer->hKeyExchangeKeyPair = hCryptKey; + if (read_key_value(hKeyContainer, hKey, "SignatureKeyPair", + dwProtectFlags, &hCryptKey)) + pKeyContainer->hSignatureKeyPair = hCryptKey; } return hKeyContainer;

1 0

Juan Lang : rsaenh: Use LocalFree to free memory return by Crypt(Un) ProtectData.
by Alexandre Julliard 07 Nov '07

07 Nov '07

Module: wine Branch: master Commit: 8788cca0eff2e64e541123c6f3869543dfe462b4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=8788cca0eff2e64e541123c6f… Author: Juan Lang <juan.lang(a)gmail.com> Date: Tue Nov 6 13:58:55 2007 -0800 rsaenh: Use LocalFree to free memory return by Crypt(Un)ProtectData. --- dlls/rsaenh/rsaenh.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/rsaenh/rsaenh.c b/dlls/rsaenh/rsaenh.c index 2d89751..776c6e7 100644 --- a/dlls/rsaenh/rsaenh.c +++ b/dlls/rsaenh/rsaenh.c @@ -895,7 +895,7 @@ static void store_key_pair(HCRYPTKEY hCryptKey, HKEY hKey, LPCSTR szValueName, D { RegSetValueExA(hKey, szValueName, 0, REG_BINARY, blobOut.pbData, blobOut.cbData); - HeapFree(GetProcessHeap(), 0, blobOut.pbData); + LocalFree(blobOut.pbData); } } HeapFree(GetProcessHeap(), 0, pbKey); @@ -1135,7 +1135,7 @@ static BOOL read_key_value(HCRYPTPROV hKeyContainer, HKEY hKey, LPCSTR szValueNa { ret = RSAENH_CPImportKey(hKeyContainer, blobOut.pbData, blobOut.cbData, 0, 0, phCryptKey); - HeapFree(GetProcessHeap(), 0, blobOut.pbData); + LocalFree(blobOut.pbData); } } HeapFree(GetProcessHeap(), 0, pbKey);

1 0

Alistair Leslie-Hughes : msxml3: Fix memory leak.
by Alexandre Julliard 07 Nov '07

07 Nov '07

Module: wine Branch: master Commit: 2ad4f102f7b72046233c4de28c49dd1504af7890 URL: http://source.winehq.org/git/wine.git/?a=commit;h=2ad4f102f7b72046233c4de28… Author: Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> Date: Wed Nov 7 12:39:11 2007 +1100 msxml3: Fix memory leak. --- dlls/msxml3/xmlelem.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/dlls/msxml3/xmlelem.c b/dlls/msxml3/xmlelem.c index 787b13f..5af43a7 100644 --- a/dlls/msxml3/xmlelem.c +++ b/dlls/msxml3/xmlelem.c @@ -340,6 +340,8 @@ static HRESULT WINAPI xmlelem_get_text(IXMLElement *iface, BSTR *p) content = xmlNodeGetContent(This->node); *p = bstr_from_xmlChar(content); TRACE("returning %s\n", debugstr_w(*p)); + + xmlFree(content); return S_OK; }

1 0

Dan Kegel : msvcrt: Fix two buglets in heap test.
by Alexandre Julliard 07 Nov '07

07 Nov '07

Module: wine Branch: master Commit: a1e4758f076190dbef7d06f9e10241d0aabe3788 URL: http://source.winehq.org/git/wine.git/?a=commit;h=a1e4758f076190dbef7d06f9e… Author: Dan Kegel <dank(a)kegel.com> Date: Tue Nov 6 16:52:19 2007 -0800 msvcrt: Fix two buglets in heap test. --- dlls/msvcrt/tests/heap.c | 17 +++++++++++------ 1 files changed, 11 insertions(+), 6 deletions(-) diff --git a/dlls/msvcrt/tests/heap.c b/dlls/msvcrt/tests/heap.c index 6dac9b4..e2e9a4a 100644 --- a/dlls/msvcrt/tests/heap.c +++ b/dlls/msvcrt/tests/heap.c @@ -148,11 +148,12 @@ static void test_aligned_realloc(size_t size1, size_t size2, size_t alignment) } } } - } - else + p_aligned_free(mem2); + } else { ok(errno == EINVAL, "_aligned_realloc(%p, %d, %d) errno: %d != %d\n", mem, size2, alignment, errno, EINVAL); + p_aligned_free(mem); + } - p_aligned_free(mem); free(mem1); } else @@ -219,11 +220,12 @@ static void test_aligned_offset_realloc(size_t size1, size_t size2, size_t align } } } - } - else + p_aligned_free(mem2); + } else { ok(errno == EINVAL, "_aligned_offset_realloc(%p, %d, %d, %d) errno: %d != %d\n", mem, size2, alignment, offset, errno, EINVAL); + p_aligned_free(mem); + } - p_aligned_free(mem); free(mem1); } else @@ -353,5 +355,8 @@ START_TEST(heap) mem = realloc(NULL, 0); ok(mem != NULL, "memory not (re)allocated for size 0\n"); + if (mem) + free(mem); + test_aligned(); }

1 0

Alexandre Julliard : wine.inf: Add APPDATA and LOCALAPPDATA environment variables.
by Alexandre Julliard 07 Nov '07

07 Nov '07

Module: wine Branch: master Commit: 28b32972d95185ccafa3430a1db41c2671aa0f10 URL: http://source.winehq.org/git/wine.git/?a=commit;h=28b32972d95185ccafa3430a1… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Nov 7 13:32:57 2007 +0100 wine.inf: Add APPDATA and LOCALAPPDATA environment variables. --- tools/wine.inf | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/tools/wine.inf b/tools/wine.inf index fdb47b9..338dbd2 100644 --- a/tools/wine.inf +++ b/tools/wine.inf @@ -253,7 +253,9 @@ HKLM,Software\Microsoft\DirectPlay\Service Providers\Serial Connection For Direc HKLM,Software\Microsoft\DirectPlay\Service Providers\Serial Connection For DirectPlay,"Path",,"dpmodemx.dll" [Environment] +HKLM,System\CurrentControlSet\Control\Session Manager\Environment,"APPDATA",,"%16410%" HKLM,System\CurrentControlSet\Control\Session Manager\Environment,"ComSpec",,"%11%\cmd.exe" +HKLM,System\CurrentControlSet\Control\Session Manager\Environment,"LOCALAPPDATA",,"%16412%" HKLM,System\CurrentControlSet\Control\Session Manager\Environment,"PATH",2,"%11%;%10%" HKLM,System\CurrentControlSet\Control\Session Manager\Environment,"ProgramFiles",,"%16422%" HKLM,System\CurrentControlSet\Control\Session Manager\Environment,"SystemDrive",2,"c:"

1 0

Alexandre Julliard : include: Only use DECLSPEC_IMPORT for Windows compilers in unicode.h.
by Alexandre Julliard 07 Nov '07

07 Nov '07

Module: wine Branch: master Commit: 6678c07f5784365fee9dc3c43f71f9712897cfde URL: http://source.winehq.org/git/wine.git/?a=commit;h=6678c07f5784365fee9dc3c43… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Nov 7 12:30:51 2007 +0100 include: Only use DECLSPEC_IMPORT for Windows compilers in unicode.h. --- include/wine/unicode.h | 6 +++++- 1 files changed, 5 insertions(+), 1 deletions(-) diff --git a/include/wine/unicode.h b/include/wine/unicode.h index 27d7cee..49f9062 100644 --- a/include/wine/unicode.h +++ b/include/wine/unicode.h @@ -32,7 +32,11 @@ #endif #ifndef WINE_UNICODE_API -#define WINE_UNICODE_API DECLSPEC_IMPORT +# if defined(_MSC_VER) || defined(__MINGW32__) +# define WINE_UNICODE_API DECLSPEC_IMPORT +# else +# define WINE_UNICODE_API +# endif #endif #ifndef WINE_UNICODE_INLINE

1 0

Alexandre Julliard : include: Mark imported functions with hidden visibility.
by Alexandre Julliard 07 Nov '07

07 Nov '07

Module: wine Branch: master Commit: 95c728551d5ee9b71d2a6328d0220a8b678c5225 URL: http://source.winehq.org/git/wine.git/?a=commit;h=95c728551d5ee9b71d2a6328d… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Nov 7 12:31:16 2007 +0100 include: Mark imported functions with hidden visibility. --- include/winnt.h | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/include/winnt.h b/include/winnt.h index f068b53..78fa1f7 100644 --- a/include/winnt.h +++ b/include/winnt.h @@ -43,13 +43,13 @@ extern "C" { #define NTAPI __stdcall -#if (defined(_M_IX86) || defined(_M_IA64) || defined(_M_AMD64) || defined(__MINGW32__)) && !defined(MIDL_PASS) +#ifndef MIDL_PASS # if defined(_MSC_VER) # define DECLSPEC_IMPORT __declspec(dllimport) # elif defined(__MINGW32__) # define DECLSPEC_IMPORT __attribute__((dllimport)) # else -# define DECLSPEC_IMPORT +# define DECLSPEC_IMPORT DECLSPEC_HIDDEN # endif #else # define DECLSPEC_IMPORT

1 0

Alexandre Julliard : include: Add WINGDIAPI to the exported gdi32 functions .
by Alexandre Julliard 07 Nov '07

07 Nov '07

Module: wine Branch: master Commit: caf56765d5b8294dc5fd1e51e7492c0fcc2559e6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=caf56765d5b8294dc5fd1e51e… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Nov 7 12:26:49 2007 +0100 include: Add WINGDIAPI to the exported gdi32 functions. --- include/wingdi.h | 824 +++++++++++++++++++++++++++--------------------------- 1 files changed, 409 insertions(+), 415 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=caf56765d5b8294dc5fd1…

1 0

Alexandre Julliard : include: Add WINADVAPI to the exported advapi32 functions.
by Alexandre Julliard 07 Nov '07

07 Nov '07

Module: wine Branch: master Commit: 2d1da3430c2481e61f030475c24894a4df97b2bf URL: http://source.winehq.org/git/wine.git/?a=commit;h=2d1da3430c2481e61f030475c… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Nov 7 12:27:27 2007 +0100 include: Add WINADVAPI to the exported advapi32 functions. --- include/aclapi.h | 126 +++++++++++++--------------------- include/wincred.h | 38 ++++++---- include/wincrypt.h | 123 ++++++++++++++------------------- include/winreg.h | 196 ++++++++++++++++++++++++--------------------------- include/winsvc.h | 130 +++++++++++++++++----------------- 5 files changed, 279 insertions(+), 334 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=2d1da3430c2481e61f030…

1 0

Alexandre Julliard : include: Add WINBASEAPI to the exported kernel32 functions.
by Alexandre Julliard 07 Nov '07

07 Nov '07

Module: wine Branch: master Commit: 16d85e2d303e5612c575ab0fd1fd51542d670506 URL: http://source.winehq.org/git/wine.git/?a=commit;h=16d85e2d303e5612c575ab0fd… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Nov 7 12:27:51 2007 +0100 include: Add WINBASEAPI to the exported kernel32 functions. --- include/winbase.h | 1817 ++++++++++++++++++++++++++--------------------------- include/wincon.h | 149 ++--- include/winnls.h | 196 +++--- 3 files changed, 1073 insertions(+), 1089 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=16d85e2d303e5612c575a…

1 0