March 2022 - Wine-devel - list.winehq.org

[PATCH v3 2/6] win32u: Move reply_message_result implementation from user32.
by Jacek Caban 31 Mar '22

31 Mar '22

Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> --- dlls/user32/message.c | 181 +----------------------- dlls/win32u/message.c | 264 +++++++++++++++++++++++++++++++++++ dlls/win32u/ntuser_private.h | 9 ++ dlls/win32u/sysparams.c | 2 + dlls/win32u/win32u_private.h | 1 + include/ntuser.h | 153 ++++++++++++++++++++ 6 files changed, 431 insertions(+), 179 deletions(-)

3 2

[tools] testbot: Remove an unneeded Fcntl import in VM::Run().
by Francois Gouget 31 Mar '22

31 Mar '22

Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com> --- testbot/lib/WineTestBot/VMs.pm | 1 - 1 file changed, 1 deletion(-) diff --git a/testbot/lib/WineTestBot/VMs.pm b/testbot/lib/WineTestBot/VMs.pm index f850daaf3..65f633354 100644 --- a/testbot/lib/WineTestBot/VMs.pm +++ b/testbot/lib/WineTestBot/VMs.pm @@ -446,7 +446,6 @@ sub Run($$$$$$) # So set ChildPid in the parent and synchronize with the child so it only # execs once this is done. - use Fcntl; my ($fd_read, $fd_write); pipe($fd_read, $fd_write); # For synchronization -- 2.30.2

1 0

[tools] testbot/cgi: ItemPage does not need URI::Escape.
by Francois Gouget 31 Mar '22

31 Mar '22

Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com> --- testbot/lib/ObjectModel/CGI/ItemPage.pm | 1 - 1 file changed, 1 deletion(-) diff --git a/testbot/lib/ObjectModel/CGI/ItemPage.pm b/testbot/lib/ObjectModel/CGI/ItemPage.pm index 567a489fc..b88e27394 100644 --- a/testbot/lib/ObjectModel/CGI/ItemPage.pm +++ b/testbot/lib/ObjectModel/CGI/ItemPage.pm @@ -31,7 +31,6 @@ ObjectModel::CGI::ItemPage - Base class for bound web forms use ObjectModel::CGI::FormPage; our @ISA = qw(ObjectModel::CGI::FormPage); -use URI::Escape; sub _initialize($$$$) { -- 2.30.2

1 0

[tools] testbot/CheckForWinetestUpdate: HTTP::Response is not needed.
by Francois Gouget 31 Mar '22

31 Mar '22

Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com> --- testbot/bin/CheckForWinetestUpdate.pl | 1 - 1 file changed, 1 deletion(-) diff --git a/testbot/bin/CheckForWinetestUpdate.pl b/testbot/bin/CheckForWinetestUpdate.pl index 94d975071..c7aecce1e 100755 --- a/testbot/bin/CheckForWinetestUpdate.pl +++ b/testbot/bin/CheckForWinetestUpdate.pl @@ -46,7 +46,6 @@ use File::Compare; use File::Copy; use LWP::UserAgent; use HTTP::Request; -use HTTP::Response; use HTTP::Status; use WineTestBot::Config; -- 2.30.2

1 0

[tools] testbot/web: Remove unneeded imports from the Submit page.
by Francois Gouget 31 Mar '22

31 Mar '22

Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com> --- testbot/web/Submit.pl | 2 -- 1 file changed, 2 deletions(-) diff --git a/testbot/web/Submit.pl b/testbot/web/Submit.pl index 3d49aaeaa..aba2b6dc5 100644 --- a/testbot/web/Submit.pl +++ b/testbot/web/Submit.pl @@ -26,9 +26,7 @@ use ObjectModel::CGI::FreeFormPage; our @ISA = qw(ObjectModel::CGI::FreeFormPage); use Fcntl qw(O_RDONLY); -use IO::Handle; use POSIX qw(:fcntl_h); # For SEEK_XXX -use File::Basename; use ObjectModel::BasicPropertyDescriptor; use ObjectModel::EnumPropertyDescriptor; -- 2.30.2

1 0

[PATCH v3 1/6] user32: Introduce reply_message_result helper.
by Jacek Caban 31 Mar '22

31 Mar '22

Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> --- dlls/user32/message.c | 42 ++++++++++++++++++++++++++++-------------- 1 file changed, 28 insertions(+), 14 deletions(-)

3 2

[PATCH v2 4/6] win32u: Move process_sent_messages implementation from user32.
by Jacek Caban 31 Mar '22

31 Mar '22

3 3

[PATCH] explorerframe: Use CRT memory allocation functions.
by Nikolay Sivov 31 Mar '22

31 Mar '22

Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> --- dlls/explorerframe/nstc.c | 15 +++++++-------- dlls/explorerframe/taskbarlist.c | 5 ++--- 2 files changed, 9 insertions(+), 11 deletions(-) diff --git a/dlls/explorerframe/nstc.c b/dlls/explorerframe/nstc.c index 315316f7e7a..3b56d33367a 100644 --- a/dlls/explorerframe/nstc.c +++ b/dlls/explorerframe/nstc.c @@ -33,7 +33,6 @@ #include "wine/list.h" #include "wine/debug.h" -#include "wine/heap.h" #include "explorerframe_main.h" @@ -852,7 +851,7 @@ static ULONG WINAPI NSTC2_fnRelease(INameSpaceTreeControl2* iface) if(!ref) { TRACE("Freeing.\n"); - heap_free(This); + free(This); EFRAME_UnlockModule(); } @@ -989,7 +988,7 @@ static HRESULT WINAPI NSTC2_fnInsertRoot(INameSpaceTreeControl2* iface, TRACE("%p, %d, %p, %lx, %lx, %p\n", This, iIndex, psiRoot, grfEnumFlags, grfRootStyle, pif); - new_root = heap_alloc(sizeof(*new_root)); + new_root = malloc(sizeof(*new_root)); if(!new_root) return E_OUTOFMEMORY; @@ -1013,7 +1012,7 @@ static HRESULT WINAPI NSTC2_fnInsertRoot(INameSpaceTreeControl2* iface, if(!new_root->htreeitem) { WARN("Failed to add the root.\n"); - heap_free(new_root); + free(new_root); return E_FAIL; } @@ -1081,7 +1080,7 @@ static HRESULT WINAPI NSTC2_fnRemoveRoot(INameSpaceTreeControl2* iface, events_OnItemDeleted(This, root->psi, TRUE); SendMessageW(This->hwnd_tv, TVM_DELETEITEM, 0, (LPARAM)root->htreeitem); list_remove(&root->entry); - heap_free(root); + free(root); return S_OK; } else @@ -1123,7 +1122,7 @@ static HRESULT WINAPI NSTC2_fnGetRootItems(INameSpaceTreeControl2* iface, if(!count) return E_INVALIDARG; - array = heap_alloc(sizeof(LPITEMIDLIST)*count); + array = malloc(sizeof(LPITEMIDLIST)*count); i = 0; LIST_FOR_EACH_ENTRY(root, &This->roots, nstc_root, entry) @@ -1137,7 +1136,7 @@ static HRESULT WINAPI NSTC2_fnGetRootItems(INameSpaceTreeControl2* iface, for(i = 0; i < count; i++) ILFree(array[i]); - heap_free(array); + free(array); return hr; } @@ -1598,7 +1597,7 @@ HRESULT NamespaceTreeControl_Constructor(IUnknown *pUnkOuter, REFIID riid, void EFRAME_LockModule(); - nstc = heap_alloc_zero(sizeof(*nstc)); + nstc = calloc(1, sizeof(*nstc)); if (!nstc) return E_OUTOFMEMORY; diff --git a/dlls/explorerframe/taskbarlist.c b/dlls/explorerframe/taskbarlist.c index b4bc104cd6b..dd61cc62ae4 100644 --- a/dlls/explorerframe/taskbarlist.c +++ b/dlls/explorerframe/taskbarlist.c @@ -21,7 +21,6 @@ #include "explorerframe_main.h" #include "wine/debug.h" -#include "wine/heap.h" WINE_DEFAULT_DEBUG_CHANNEL(explorerframe); @@ -81,7 +80,7 @@ static ULONG STDMETHODCALLTYPE taskbar_list_Release(ITaskbarList4 *iface) if (!refcount) { - heap_free(This); + free(This); EFRAME_UnlockModule(); } @@ -309,7 +308,7 @@ HRESULT TaskbarList_Constructor(IUnknown *outer, REFIID riid, void **taskbar_lis return CLASS_E_NOAGGREGATION; } - object = heap_alloc(sizeof(*object)); + object = malloc(sizeof(*object)); if (!object) { ERR("Failed to allocate taskbar list object memory\n"); -- 2.35.1

1 0

[PATCH] sxs: Use CRT memory allocation functions.
by Nikolay Sivov 31 Mar '22

31 Mar '22

Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> --- dlls/appwiz.cpl/appwiz.c | 44 ++++++++++++++++++++-------------------- 1 file changed, 22 insertions(+), 22 deletions(-) diff --git a/dlls/appwiz.cpl/appwiz.c b/dlls/appwiz.cpl/appwiz.c index 593c8644b7e..012bc054129 100644 --- a/dlls/appwiz.cpl/appwiz.c +++ b/dlls/appwiz.cpl/appwiz.c @@ -106,19 +106,19 @@ BOOL WINAPI DllMain(HINSTANCE hinstDLL, DWORD fdwReason, */ static void FreeAppInfo(APPINFO *info) { - HeapFree(GetProcessHeap(), 0, info->title); - HeapFree(GetProcessHeap(), 0, info->path); - HeapFree(GetProcessHeap(), 0, info->path_modify); - HeapFree(GetProcessHeap(), 0, info->icon); - HeapFree(GetProcessHeap(), 0, info->publisher); - HeapFree(GetProcessHeap(), 0, info->version); - HeapFree(GetProcessHeap(), 0, info->contact); - HeapFree(GetProcessHeap(), 0, info->helplink); - HeapFree(GetProcessHeap(), 0, info->helptelephone); - HeapFree(GetProcessHeap(), 0, info->readme); - HeapFree(GetProcessHeap(), 0, info->urlupdateinfo); - HeapFree(GetProcessHeap(), 0, info->comments); - HeapFree(GetProcessHeap(), 0, info); + free(info->title); + free(info->path); + free(info->path_modify); + free(info->icon); + free(info->publisher); + free(info->version); + free(info->contact); + free(info->helplink); + free(info->helptelephone); + free(info->readme); + free(info->urlupdateinfo); + free(info->comments); + free(info); } static WCHAR *get_reg_str(HKEY hkey, const WCHAR *value) @@ -127,7 +127,7 @@ static WCHAR *get_reg_str(HKEY hkey, const WCHAR *value) WCHAR *ret = NULL; if (!RegQueryValueExW(hkey, value, NULL, &type, NULL, &len) && type == REG_SZ) { - if (!(ret = HeapAlloc(GetProcessHeap(), 0, len))) return NULL; + if (!(ret = malloc(len))) return NULL; RegQueryValueExW(hkey, value, 0, 0, (BYTE *)ret, &len); } return ret; @@ -176,12 +176,12 @@ static BOOL ReadApplicationsFromRegistry(HKEY root) { int len = lstrlenW(L"msiexec /x%s") + lstrlenW(subKeyName); - if (!(command = HeapAlloc(GetProcessHeap(), 0, len * sizeof(WCHAR)))) goto err; + if (!(command = malloc(len * sizeof(WCHAR)))) goto err; wsprintfW(command, L"msiexec /x%s", subKeyName); } else if (!RegQueryValueExW(hkeyApp, L"UninstallString", 0, 0, NULL, &uninstlen)) { - if (!(command = HeapAlloc(GetProcessHeap(), 0, uninstlen))) goto err; + if (!(command = malloc(uninstlen))) goto err; RegQueryValueExW(hkeyApp, L"UninstallString", 0, 0, (BYTE *)command, &uninstlen); } else @@ -191,10 +191,10 @@ static BOOL ReadApplicationsFromRegistry(HKEY root) continue; } - info = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(struct APPINFO)); + info = calloc(1, sizeof(*info)); if (!info) goto err; - info->title = HeapAlloc(GetProcessHeap(), 0, displen); + info->title = malloc(displen); if (!info->title) goto err; @@ -209,7 +209,7 @@ static BOOL ReadApplicationsFromRegistry(HKEY root) info->icon = 0; else { - info->icon = HeapAlloc(GetProcessHeap(), 0, displen); + info->icon = malloc(displen); if (!info->icon) goto err; @@ -259,12 +259,12 @@ static BOOL ReadApplicationsFromRegistry(HKEY root) { int len = lstrlenW(L"msiexec /i%s") + lstrlenW(subKeyName); - if (!(info->path_modify = HeapAlloc(GetProcessHeap(), 0, len * sizeof(WCHAR)))) goto err; + if (!(info->path_modify = malloc(len * sizeof(WCHAR)))) goto err; wsprintfW(info->path_modify, L"msiexec /i%s", subKeyName); } else if (!RegQueryValueExW(hkeyApp, L"ModifyPath", 0, 0, NULL, &displen)) { - if (!(info->path_modify = HeapAlloc(GetProcessHeap(), 0, displen))) goto err; + if (!(info->path_modify = malloc(displen))) goto err; RegQueryValueExW(hkeyApp, L"ModifyPath", 0, 0, (BYTE *)info->path_modify, &displen); } } @@ -286,7 +286,7 @@ static BOOL ReadApplicationsFromRegistry(HKEY root) err: RegCloseKey(hkeyApp); if (info) FreeAppInfo(info); - HeapFree(GetProcessHeap(), 0, command); + free(command); return FALSE; } -- 2.35.1

1 0

[PATCH] atl: Use CRT allocation functions.
by Nikolay Sivov 31 Mar '22

31 Mar '22

Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> --- dlls/atl/atl.c | 11 +++++------ dlls/atl/atl30.c | 2 +- dlls/atl/atl_ax.c | 24 +++++++++++------------ dlls/atl/registrar.c | 45 ++++++++++++++++++++++---------------------- 4 files changed, 40 insertions(+), 42 deletions(-) diff --git a/dlls/atl/atl.c b/dlls/atl/atl.c index ed6afe4ea2e..87a24e33ed7 100644 --- a/dlls/atl/atl.c +++ b/dlls/atl/atl.c @@ -23,7 +23,6 @@ #include "atlcom.h" #include "wine/debug.h" -#include "wine/heap.h" WINE_DEFAULT_DEBUG_CHANNEL(atl); @@ -294,7 +293,7 @@ HRESULT WINAPI AtlModuleAddTermFunc(_ATL_MODULE *pM, _ATL_TERMFUNC *pFunc, DWORD TRACE("version %04x (%p %p %Id)\n", _ATL_VER, pM, pFunc, dw); if (_ATL_VER > _ATL_VER_30 || pM->cbSize > ATLVer1Size) { - termfunc_elem = HeapAlloc(GetProcessHeap(), 0, sizeof(_ATL_TERMFUNC_ELEM)); + termfunc_elem = malloc(sizeof(*termfunc_elem)); termfunc_elem->pFunc = pFunc; termfunc_elem->dw = dw; termfunc_elem->pNext = pM->m_pTermFuncs; @@ -320,7 +319,7 @@ void WINAPI AtlCallTermFunc(_ATL_MODULE *pM) iter->pFunc(iter->dw); tmp = iter; iter = iter->pNext; - HeapFree(GetProcessHeap(), 0, tmp); + free(tmp); } pM->m_pTermFuncs = NULL; @@ -342,13 +341,13 @@ HRESULT WINAPI AtlLoadTypeLib(HINSTANCE inst, LPCOLESTR lpszIndex, TRACE("(%p %s %p %p)\n", inst, debugstr_w(lpszIndex), pbstrPath, ppTypeLib); index_len = lpszIndex ? lstrlenW(lpszIndex) : 0; - path = heap_alloc((MAX_PATH+index_len)*sizeof(WCHAR) + sizeof(L".tlb")); + path = malloc((MAX_PATH+index_len)*sizeof(WCHAR) + sizeof(L".tlb")); if(!path) return E_OUTOFMEMORY; path_len = GetModuleFileNameW(inst, path, MAX_PATH); if(!path_len) { - heap_free(path); + free(path); return HRESULT_FROM_WIN32(GetLastError()); } @@ -374,7 +373,7 @@ HRESULT WINAPI AtlLoadTypeLib(HINSTANCE inst, LPCOLESTR lpszIndex, } } - heap_free(path); + free(path); if(FAILED(hres)) return hres; diff --git a/dlls/atl/atl30.c b/dlls/atl/atl30.c index e84656998b8..d7d130c2a03 100644 --- a/dlls/atl/atl30.c +++ b/dlls/atl/atl30.c @@ -121,7 +121,7 @@ HRESULT WINAPI AtlModuleTerm(_ATL_MODULE *pM) iter->pFunc(iter->dw); tmp = iter; iter = iter->pNext; - HeapFree(GetProcessHeap(), 0, tmp); + free(tmp); } } diff --git a/dlls/atl/atl_ax.c b/dlls/atl/atl_ax.c index 04b0afdb759..4ff0a796d3b 100644 --- a/dlls/atl/atl_ax.c +++ b/dlls/atl/atl_ax.c @@ -63,12 +63,12 @@ static LRESULT CALLBACK AtlAxWin_wndproc( HWND hWnd, UINT wMsg, WPARAM wParam, L if ( wMsg == WM_CREATE ) { DWORD len = GetWindowTextLengthW( hWnd ) + 1; - WCHAR *ptr = HeapAlloc( GetProcessHeap(), 0, len*sizeof(WCHAR) ); + WCHAR *ptr = malloc( len*sizeof(WCHAR) ); if (!ptr) return 1; GetWindowTextW( hWnd, ptr, len ); AtlAxCreateControlEx( ptr, hWnd, NULL, NULL, NULL, NULL, NULL ); - HeapFree( GetProcessHeap(), 0, ptr ); + free( ptr ); return 0; } return DefWindowProcW( hWnd, wMsg, wParam, lParam ); @@ -225,7 +225,7 @@ static ULONG WINAPI OleClientSite_Release(IOleClientSite *iface) if (!ref) { IOCS_Detach( This ); - HeapFree( GetProcessHeap(), 0, This ); + free( This ); } return ref; @@ -945,7 +945,7 @@ static HRESULT IOCS_Create( HWND hWnd, IUnknown *pUnkControl, IUnknown **contain return S_OK; *container = NULL; - This = HeapAlloc(GetProcessHeap(), 0, sizeof(IOCS)); + This = malloc(sizeof(*This)); if (!This) return E_OUTOFMEMORY; @@ -969,7 +969,7 @@ static HRESULT IOCS_Create( HWND hWnd, IUnknown *pUnkControl, IUnknown **contain else { IOCS_Detach( This ); - HeapFree(GetProcessHeap(), 0, This); + free(This); } return hr; @@ -1160,7 +1160,7 @@ static inline BOOL advance_array(WORD **pptr, DWORD *palloc, DWORD *pfilled, con if ( (*pfilled + size) > *palloc ) { *palloc = ((*pfilled+size) + 0xFF) & ~0xFF; - *pptr = HeapReAlloc( GetProcessHeap(), 0, *pptr, *palloc * sizeof(WORD) ); + *pptr = realloc( *pptr, *palloc * sizeof(WORD) ); if (!*pptr) return FALSE; } @@ -1186,7 +1186,7 @@ static LPDLGTEMPLATEW AX_ConvertDialogTemplate(LPCDLGTEMPLATEW src_tmpl) DWORD style; filled = 0; allocated = 256; - output = HeapAlloc( GetProcessHeap(), 0, allocated * sizeof(WORD) ); + output = malloc( allocated * sizeof(WORD) ); if (!output) return NULL; @@ -1307,12 +1307,12 @@ HWND WINAPI AtlAxCreateDialogA(HINSTANCE hInst, LPCSTR name, HWND owner, DLGPROC return AtlAxCreateDialogW( hInst, (LPCWSTR) name, owner, dlgProc, param ); length = MultiByteToWideChar( CP_ACP, 0, name, -1, NULL, 0 ); - nameW = HeapAlloc( GetProcessHeap(), 0, length * sizeof(WCHAR) ); + nameW = malloc( length * sizeof(WCHAR) ); if (nameW) { MultiByteToWideChar( CP_ACP, 0, name, -1, nameW, length ); res = AtlAxCreateDialogW( hInst, nameW, owner, dlgProc, param ); - HeapFree( GetProcessHeap(), 0, nameW ); + free( nameW ); } return res; } @@ -1348,8 +1348,8 @@ HWND WINAPI AtlAxCreateDialogW(HINSTANCE hInst, LPCWSTR name, HWND owner, DLGPRO newptr = AX_ConvertDialogTemplate( ptr ); if ( newptr ) { - res = CreateDialogIndirectParamW( hInst, newptr, owner, dlgProc, param ); - HeapFree( GetProcessHeap(), 0, newptr ); + res = CreateDialogIndirectParamW( hInst, newptr, owner, dlgProc, param ); + free( newptr ); } else res = NULL; FreeResource ( hrsrc ); @@ -1425,7 +1425,7 @@ INT_PTR WINAPI AtlAxDialogBoxW(HINSTANCE instance, const WCHAR *name, return 0; ret = DialogBoxIndirectParamW(instance, template, owner, proc, param); - HeapFree(GetProcessHeap(), 0, template); + free(template); return ret; } diff --git a/dlls/atl/registrar.c b/dlls/atl/registrar.c index 061483f6513..5a3eef7dd4f 100644 --- a/dlls/atl/registrar.c +++ b/dlls/atl/registrar.c @@ -74,7 +74,7 @@ static inline Registrar *impl_from_IRegistrar(IRegistrar *iface) static void strbuf_init(strbuf *buf) { - buf->str = HeapAlloc(GetProcessHeap(), 0, 128*sizeof(WCHAR)); + buf->str = malloc(128*sizeof(WCHAR)); buf->alloc = 128; buf->len = 0; } @@ -85,7 +85,7 @@ static void strbuf_write(LPCOLESTR str, strbuf *buf, int len) len = lstrlenW(str); if(buf->len+len+1 >= buf->alloc) { buf->alloc = (buf->len+len)<<1; - buf->str = HeapReAlloc(GetProcessHeap(), 0, buf->str, buf->alloc*sizeof(WCHAR)); + buf->str = realloc(buf->str, buf->alloc*sizeof(WCHAR)); } memcpy(buf->str+buf->len, str, len*sizeof(OLECHAR)); buf->len += len; @@ -292,7 +292,7 @@ static HRESULT do_process_key(LPCOLESTR *pstr, HKEY parent_key, strbuf *buf, BOO if(FAILED(hres)) break; count = (lstrlenW(buf->str) + 1) / 2; - bytes = HeapAlloc(GetProcessHeap(), 0, count); + bytes = malloc(count); if(bytes == NULL) { hres = E_OUTOFMEMORY; break; @@ -313,7 +313,7 @@ static HRESULT do_process_key(LPCOLESTR *pstr, HKEY parent_key, strbuf *buf, BOO hres = HRESULT_FROM_WIN32(lres); } } - HeapFree(GetProcessHeap(), 0, bytes); + free(bytes); break; } default: @@ -360,7 +360,7 @@ static HRESULT do_process_key(LPCOLESTR *pstr, HKEY parent_key, strbuf *buf, BOO break; } - HeapFree(GetProcessHeap(), 0, name.str); + free(name.str); if(hkey && key_type != IS_VAL) RegCloseKey(hkey); *pstr = iter; @@ -411,7 +411,7 @@ static HRESULT do_process_root_key(LPCOLESTR data, BOOL do_register) if(FAILED(hres)) break; } - HeapFree(GetProcessHeap(), 0, buf.str); + free(buf.str); return hres; } @@ -426,7 +426,7 @@ static HRESULT string_register(Registrar *This, LPCOLESTR data, BOOL do_register hres = do_preprocess(This, data, &buf); if(FAILED(hres)) { WARN("preprocessing failed!\n"); - HeapFree(GetProcessHeap(), 0, buf.str); + free(buf.str); return hres; } @@ -434,7 +434,7 @@ static HRESULT string_register(Registrar *This, LPCOLESTR data, BOOL do_register if(FAILED(hres) && do_register) do_process_root_key(buf.str, FALSE); - HeapFree(GetProcessHeap(), 0, buf.str); + free(buf.str); return hres; } @@ -456,13 +456,13 @@ static HRESULT resource_register(Registrar *This, LPCOLESTR resFileName, reslen = SizeofResource(hins, src); if(regstra) { len = MultiByteToWideChar(CP_ACP, 0, regstra, reslen, NULL, 0)+1; - regstrw = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, len*sizeof(WCHAR)); + regstrw = calloc(len, sizeof(WCHAR)); MultiByteToWideChar(CP_ACP, 0, regstra, reslen, regstrw, len); regstrw[len-1] = '\0'; hres = string_register(This, regstrw, do_register); - HeapFree(GetProcessHeap(), 0, regstrw); + free(regstrw); }else { WARN("could not load resource\n"); hres = HRESULT_FROM_WIN32(GetLastError()); @@ -491,21 +491,21 @@ static HRESULT file_register(Registrar *This, LPCOLESTR fileName, BOOL do_regist file = CreateFileW(fileName, GENERIC_READ, FILE_SHARE_READ, NULL, OPEN_EXISTING, 0, NULL); if(file != INVALID_HANDLE_VALUE) { filelen = GetFileSize(file, NULL); - regstra = HeapAlloc(GetProcessHeap(), 0, filelen); + regstra = malloc(filelen); if(ReadFile(file, regstra, filelen, NULL, NULL)) { len = MultiByteToWideChar(CP_ACP, 0, regstra, filelen, NULL, 0)+1; - regstrw = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, len*sizeof(WCHAR)); + regstrw = calloc(len, sizeof(WCHAR)); MultiByteToWideChar(CP_ACP, 0, regstra, filelen, regstrw, len); regstrw[len-1] = '\0'; hres = string_register(This, regstrw, do_register); - HeapFree(GetProcessHeap(), 0, regstrw); + free(regstrw); }else { WARN("Failed to read file %s\n", debugstr_w(fileName)); hres = HRESULT_FROM_WIN32(GetLastError()); } - HeapFree(GetProcessHeap(), 0, regstra); + free(regstra); CloseHandle(file); }else { WARN("Could not open file %s\n", debugstr_w(fileName)); @@ -545,7 +545,7 @@ static ULONG WINAPI Registrar_Release(IRegistrar *iface) TRACE("(%p) ->%ld\n", This, ref); if(!ref) { IRegistrar_ClearReplacements(iface); - HeapFree(GetProcessHeap(), 0, This); + free(This); } return ref; } @@ -558,14 +558,14 @@ static HRESULT WINAPI Registrar_AddReplacement(IRegistrar *iface, LPCOLESTR Key, TRACE("(%p)->(%s %s)\n", This, debugstr_w(Key), debugstr_w(item)); - new_rep = HeapAlloc(GetProcessHeap(), 0, sizeof(rep_list)); + new_rep = malloc(sizeof(*new_rep)); new_rep->key_len = lstrlenW(Key); - new_rep->key = HeapAlloc(GetProcessHeap(), 0, (new_rep->key_len + 1) * sizeof(OLECHAR)); + new_rep->key = malloc((new_rep->key_len + 1) * sizeof(OLECHAR)); memcpy(new_rep->key, Key, (new_rep->key_len+1)*sizeof(OLECHAR)); len = lstrlenW(item)+1; - new_rep->item = HeapAlloc(GetProcessHeap(), 0, len*sizeof(OLECHAR)); + new_rep->item = malloc(len*sizeof(OLECHAR)); memcpy(new_rep->item, item, len*sizeof(OLECHAR)); new_rep->next = This->rep; @@ -587,9 +587,9 @@ static HRESULT WINAPI Registrar_ClearReplacements(IRegistrar *iface) iter = This->rep; while(iter) { iter2 = iter->next; - HeapFree(GetProcessHeap(), 0, iter->key); - HeapFree(GetProcessHeap(), 0, iter->item); - HeapFree(GetProcessHeap(), 0, iter); + free(iter->key); + free(iter->item); + free(iter); iter = iter2; } @@ -680,13 +680,12 @@ HRESULT WINAPI AtlCreateRegistrar(IRegistrar **ret) { Registrar *registrar; - registrar = HeapAlloc(GetProcessHeap(), 0, sizeof(*registrar)); + registrar = calloc(1, sizeof(*registrar)); if(!registrar) return E_OUTOFMEMORY; registrar->IRegistrar_iface.lpVtbl = &RegistrarVtbl; registrar->ref = 1; - registrar->rep = NULL; *ret = &registrar->IRegistrar_iface; return S_OK; -- 2.35.1

1 0