January 2010 - Wine-commits

James Hawkins : wintrust: Fix a memory leak until our implementation is fixed.
by Alexandre Julliard 07 Jan '10

07 Jan '10

Module: wine Branch: master Commit: b2d1c3eeecac64d77243450ce9446e2115e4f30e URL: http://source.winehq.org/git/wine.git/?a=commit;h=b2d1c3eeecac64d77243450ce… Author: James Hawkins <truiken(a)gmail.com> Date: Wed Jan 6 18:26:57 2010 -0800 wintrust: Fix a memory leak until our implementation is fixed. --- dlls/wintrust/tests/crypt.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/dlls/wintrust/tests/crypt.c b/dlls/wintrust/tests/crypt.c index 3e77814..a2e096a 100644 --- a/dlls/wintrust/tests/crypt.c +++ b/dlls/wintrust/tests/crypt.c @@ -515,6 +515,8 @@ static void test_CryptCATAdminAddRemoveCatalog(void) ok(hcatinfo == NULL, "CryptCATAdminAddCatalog succeeded\n"); ok(error == ERROR_BAD_FORMAT, "got %u expected ERROR_BAD_FORMAT\n", GetLastError()); } + if (hcatinfo != NULL) + pCryptCATAdminReleaseCatalogContext(hcatadmin, hcatinfo, 0); SetLastError(0xdeadbeef); hcatinfo = pCryptCATAdminAddCatalog(hcatadmin, tmpfileW, basenameW, 1);

1 0

James Hawkins : browseui: Fix two memory leaks in the autocomplete tests.
by Alexandre Julliard 07 Jan '10

07 Jan '10

Module: wine Branch: master Commit: 5324cb6135cfb43a5fb227ed39837b3375e87171 URL: http://source.winehq.org/git/wine.git/?a=commit;h=5324cb6135cfb43a5fb227ed3… Author: James Hawkins <truiken(a)gmail.com> Date: Wed Jan 6 18:27:01 2010 -0800 browseui: Fix two memory leaks in the autocomplete tests. --- dlls/browseui/tests/autocomplete.c | 3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/dlls/browseui/tests/autocomplete.c b/dlls/browseui/tests/autocomplete.c index 84be84d..89f864d 100644 --- a/dlls/browseui/tests/autocomplete.c +++ b/dlls/browseui/tests/autocomplete.c @@ -321,6 +321,9 @@ static void test_ACLMulti(void) ok(mgr->lpVtbl->Release(mgr) == 0, "Unexpected references\n"); ok(acl1->ref == 1, "acl1 not released\n"); ok(acl2->ref == 1, "acl2 not released\n"); + + CoTaskMemFree(acl1); + CoTaskMemFree(acl2); } START_TEST(autocomplete)

1 0

James Hawkins : setupapi: Refactor freeing the INF file into free_inf_file. Use this new function to free any remnants of the parsing on error.
by Alexandre Julliard 07 Jan '10

07 Jan '10

Module: wine Branch: master Commit: b116175d4a5f4e818643c0b66dc8c4943f71d8cc URL: http://source.winehq.org/git/wine.git/?a=commit;h=b116175d4a5f4e818643c0b66… Author: James Hawkins <truiken(a)gmail.com> Date: Wed Jan 6 18:26:52 2010 -0800 setupapi: Refactor freeing the INF file into free_inf_file. Use this new function to free any remnants of the parsing on error. --- dlls/setupapi/parser.c | 23 +++++++++++++++-------- 1 files changed, 15 insertions(+), 8 deletions(-) diff --git a/dlls/setupapi/parser.c b/dlls/setupapi/parser.c index 37b900f..7f581bd 100644 --- a/dlls/setupapi/parser.c +++ b/dlls/setupapi/parser.c @@ -856,6 +856,19 @@ static const WCHAR *comment_state( struct parser *parser, const WCHAR *pos ) } +static void free_inf_file( struct inf_file *file ) +{ + unsigned int i; + + for (i = 0; i < file->nb_sections; i++) HeapFree( GetProcessHeap(), 0, file->sections[i] ); + HeapFree( GetProcessHeap(), 0, file->filename ); + HeapFree( GetProcessHeap(), 0, file->sections ); + HeapFree( GetProcessHeap(), 0, file->fields ); + HeapFree( GetProcessHeap(), 0, file->strings ); + HeapFree( GetProcessHeap(), 0, file ); +} + + /* parse a complete buffer */ static DWORD parse_buffer( struct inf_file *file, const WCHAR *buffer, const WCHAR *end, UINT *error_line ) @@ -1011,7 +1024,7 @@ static struct inf_file *parse_file( HANDLE handle, const WCHAR *class, DWORD sty UnmapViewOfFile( buffer ); if (err) { - HeapFree( GetProcessHeap(), 0, file ); + free_inf_file( file ); SetLastError( err ); file = NULL; } @@ -1231,16 +1244,10 @@ HINF WINAPI SetupOpenMasterInf( VOID ) void WINAPI SetupCloseInfFile( HINF hinf ) { struct inf_file *file = hinf; - unsigned int i; if (!hinf || (hinf == INVALID_HANDLE_VALUE)) return; - for (i = 0; i < file->nb_sections; i++) HeapFree( GetProcessHeap(), 0, file->sections[i] ); - HeapFree( GetProcessHeap(), 0, file->filename ); - HeapFree( GetProcessHeap(), 0, file->sections ); - HeapFree( GetProcessHeap(), 0, file->fields ); - HeapFree( GetProcessHeap(), 0, file->strings ); - HeapFree( GetProcessHeap(), 0, file ); + free_inf_file( file ); }

1 0

Nikolay Sivov : oleaut32/tests: Fix some leaks in safearray tests.
by Alexandre Julliard 07 Jan '10

07 Jan '10

Module: wine Branch: master Commit: 88472df473b0cb8a81fb03956c8fb48386054704 URL: http://source.winehq.org/git/wine.git/?a=commit;h=88472df473b0cb8a81fb03956… Author: Nikolay Sivov <bunglehead(a)gmail.com> Date: Thu Jan 7 13:48:58 2010 +0300 oleaut32/tests: Fix some leaks in safearray tests. --- dlls/oleaut32/tests/safearray.c | 3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/dlls/oleaut32/tests/safearray.c b/dlls/oleaut32/tests/safearray.c index 439325d..29e8761 100644 --- a/dlls/oleaut32/tests/safearray.c +++ b/dlls/oleaut32/tests/safearray.c @@ -1321,6 +1321,7 @@ static void test_SafeArrayCopyData(void) ok(SafeArrayGetElemsize(sa) == SafeArrayGetElemsize(sacopy),"elemsize wrong\n"); ok(SafeArrayGetDim(sa) == SafeArrayGetDim(sacopy),"dimensions wrong\n"); ok(!memcmp(sa->pvData, sacopy->pvData, size * sizeof(int)), "compared different\n"); + SafeArrayDestroy(sacopy); } SafeArrayDestroy(sa); @@ -1548,6 +1549,7 @@ static void test_SafeArrayCopy(void) hres = SafeArrayCopy(sa, &sa2); ok(hres == S_OK, "SafeArrayCopy failed with error 0x%08x\n", hres); + SafeArrayDestroy(sa2); SafeArrayDestroy(sa); } @@ -1652,6 +1654,7 @@ static void test_SafeArrayChangeTypeEx(void) hres = VariantChangeTypeEx(&v2, &v, 0, 0, VT_ARRAY|VT_UI1); ok(hres == S_OK, "CTE VT_ARRAY|VT_UI1->VT_ARRAY|VT_UI1 returned %x\n", hres); SafeArrayDestroy(sa); + VariantClear(&v2); } /* NULL/EMPTY */

1 0

Jacek Caban : urlmon: Fixed accept_mimes leak in tests.
by Alexandre Julliard 07 Jan '10

07 Jan '10

Module: wine Branch: master Commit: 04cfe6acf75b598474c59c09272d272473157583 URL: http://source.winehq.org/git/wine.git/?a=commit;h=04cfe6acf75b598474c59c092… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu Jan 7 13:18:33 2010 +0100 urlmon: Fixed accept_mimes leak in tests. --- dlls/urlmon/tests/protocol.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/dlls/urlmon/tests/protocol.c b/dlls/urlmon/tests/protocol.c index afaafcb..d54d00d 100644 --- a/dlls/urlmon/tests/protocol.c +++ b/dlls/urlmon/tests/protocol.c @@ -1355,6 +1355,7 @@ static HRESULT WINAPI ProtocolEmul_Start(IInternetProtocol *iface, LPCWSTR szUrl "GetBindString(BINDSTRING_ACCEPT_MIMES) failed: %08x\n", hres); ok(fetched == 1, "fetched = %d, expected 1\n", fetched); ok(!strcmp_ww(acc_mimeW, accept_mimes[0]), "unexpected mimes %s\n", wine_dbgstr_w(accept_mimes[0])); + CoTaskMemFree(accept_mimes[0]); hres = IInternetBindInfo_QueryInterface(pOIBindInfo, &IID_IServiceProvider, (void**)&service_provider);

1 0

Nikolay Sivov : shell32/shelllink: Fix some string buffers leaks.
by Alexandre Julliard 07 Jan '10

07 Jan '10

Module: wine Branch: master Commit: 9a53cb52637f9856041eb70f17c1369004a5d8e1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9a53cb52637f9856041eb70f1… Author: Nikolay Sivov <bunglehead(a)gmail.com> Date: Thu Jan 7 16:00:50 2010 +0300 shell32/shelllink: Fix some string buffers leaks. --- dlls/shell32/shelllink.c | 5 ++++- 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/dlls/shell32/shelllink.c b/dlls/shell32/shelllink.c index 6dd62a7..e304058 100644 --- a/dlls/shell32/shelllink.c +++ b/dlls/shell32/shelllink.c @@ -303,7 +303,10 @@ static ULONG ShellLink_Release( IShellLinkImpl *This ) HeapFree(GetProcessHeap(), 0, This->sArgs); HeapFree(GetProcessHeap(), 0, This->sWorkDir); HeapFree(GetProcessHeap(), 0, This->sDescription); - HeapFree(GetProcessHeap(),0,This->sPath); + HeapFree(GetProcessHeap(), 0, This->sPath); + HeapFree(GetProcessHeap(), 0, This->sPathRel); + HeapFree(GetProcessHeap(), 0, This->sProduct); + HeapFree(GetProcessHeap(), 0, This->sComponent); if (This->site) IUnknown_Release( This->site );

1 0

Marcin Baczyński : winedump: Remove superfluous void* cast.
by Alexandre Julliard 07 Jan '10

07 Jan '10

Module: wine Branch: master Commit: 6a02e744b92e87b70611c40f42900c49898fef9f URL: http://source.winehq.org/git/wine.git/?a=commit;h=6a02e744b92e87b70611c40f4… Author: Marcin Baczyński <marbacz(a)gmail.com> Date: Wed Jan 6 23:38:30 2010 +0100 winedump: Remove superfluous void* cast. --- tools/winedump/pe.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/tools/winedump/pe.c b/tools/winedump/pe.c index d00d2ea..2732662 100644 --- a/tools/winedump/pe.c +++ b/tools/winedump/pe.c @@ -1425,7 +1425,7 @@ static void dump_symbol_table(void) numsym = PE_nt_headers->FileHeader.NumberOfSymbols; if (!PE_nt_headers->FileHeader.PointerToSymbolTable || !numsym) return; - sym = (const IMAGE_SYMBOL*)PRD(PE_nt_headers->FileHeader.PointerToSymbolTable, + sym = PRD(PE_nt_headers->FileHeader.PointerToSymbolTable, sizeof(*sym) * numsym); if (!sym) return; /* FIXME: no way to get strtable size */

1 0

Marcin Baczyński : libport: Remove superfluous void* cast.
by Alexandre Julliard 07 Jan '10

07 Jan '10

Module: wine Branch: master Commit: bb9c9264c85fcb4a0309fde72d9ba3b99704cb76 URL: http://source.winehq.org/git/wine.git/?a=commit;h=bb9c9264c85fcb4a0309fde72… Author: Marcin Baczyński <marbacz(a)gmail.com> Date: Wed Jan 6 23:38:33 2010 +0100 libport: Remove superfluous void* cast. --- libs/port/getopt.c | 3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/libs/port/getopt.c b/libs/port/getopt.c index 041bef0..eeb64a6 100644 --- a/libs/port/getopt.c +++ b/libs/port/getopt.c @@ -441,8 +441,7 @@ _getopt_initialize (argc, argv, optstring) int len = nonoption_flags_max_len = strlen (orig_str); if (nonoption_flags_max_len < argc) nonoption_flags_max_len = argc; - __getopt_nonoption_flags = - (char *) malloc (nonoption_flags_max_len); + __getopt_nonoption_flags = malloc (nonoption_flags_max_len); if (__getopt_nonoption_flags == NULL) nonoption_flags_max_len = -1; else

1 0

Marcin Baczyński : comctl32: Remove superfluous void* cast.
by Alexandre Julliard 07 Jan '10

07 Jan '10

Module: wine Branch: master Commit: 9e951e8d14df8d1415eb8c64e5310668deb4e84d URL: http://source.winehq.org/git/wine.git/?a=commit;h=9e951e8d14df8d1415eb8c64e… Author: Marcin Baczyński <marbacz(a)gmail.com> Date: Wed Jan 6 23:38:28 2010 +0100 comctl32: Remove superfluous void* cast. --- dlls/comctl32/listview.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/comctl32/listview.c b/dlls/comctl32/listview.c index d98b108..d21935f 100644 --- a/dlls/comctl32/listview.c +++ b/dlls/comctl32/listview.c @@ -7464,7 +7464,7 @@ static INT LISTVIEW_InsertItemT(LISTVIEW_INFO *infoPtr, const LVITEMW *lpLVItem, while (i < infoPtr->nItemCount) { hItem = DPA_GetPtr( infoPtr->hdpaItems, i); - item_s = (ITEM_INFO*)DPA_GetPtr(hItem, 0); + item_s = DPA_GetPtr(hItem, 0); cmpv = textcmpWT(item_s->hdr.pszText, lpLVItem->pszText, isW); if (infoPtr->dwStyle & LVS_SORTDESCENDING) cmpv *= -1;

1 0

Marcin Baczyński : winedbg: Remove superfluous void* cast.
by Alexandre Julliard 07 Jan '10

07 Jan '10

Module: wine Branch: master Commit: 477117a03fe7a4b945e294d93d56178a43ea1623 URL: http://source.winehq.org/git/wine.git/?a=commit;h=477117a03fe7a4b945e294d93… Author: Marcin Baczyński <marbacz(a)gmail.com> Date: Wed Jan 6 23:38:34 2010 +0100 winedbg: Remove superfluous void* cast. --- programs/winedbg/break.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/programs/winedbg/break.c b/programs/winedbg/break.c index e9f7148..ed382bc 100644 --- a/programs/winedbg/break.c +++ b/programs/winedbg/break.c @@ -60,7 +60,7 @@ void break_set_xpoints(BOOL set) size = 0; else size = bp[i].w.len + 1; - addr = (void*)memory_to_linear_addr(&bp[i].addr); + addr = memory_to_linear_addr(&bp[i].addr); if (set) ret = be_cpu->insert_Xpoint(dbg_curr_process->handle,

1 0