November 2022 - Wine-Commits

Rémi Bernon : hidclass.sys: Add more trace messages.
by Alexandre Julliard Nov. 22, 2022

Nov. 22, 2022

Module: wine Branch: master Commit: 1d2ea5fb890a3b1cb6d7aceb44f2ea3305eb948e URL: https://gitlab.winehq.org/wine/wine/-/commit/1d2ea5fb890a3b1cb6d7aceb44f2ea… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Mon Jan 3 17:23:38 2022 +0100 hidclass.sys: Add more trace messages. --- dlls/hidclass.sys/device.c | 2 ++ dlls/hidclass.sys/pnp.c | 4 ++++ 2 files changed, 6 insertions(+) diff --git a/dlls/hidclass.sys/device.c b/dlls/hidclass.sys/device.c index 7c0dbdcbc10..ac201afeddf 100644 --- a/dlls/hidclass.sys/device.c +++ b/dlls/hidclass.sys/device.c @@ -229,6 +229,8 @@ static void hid_device_queue_input( DEVICE_OBJECT *device, HID_XFER_PACKET *pack KIRQL irql; IRP *irp; + TRACE("device %p, packet %p\n", device, packet); + if (IsEqualGUID( ext->class_guid, &GUID_DEVINTERFACE_HID )) { size = offsetof( RAWINPUT, data.hid.bRawData[report_len] ); diff --git a/dlls/hidclass.sys/pnp.c b/dlls/hidclass.sys/pnp.c index 91624b40eff..46b6e803465 100644 --- a/dlls/hidclass.sys/pnp.c +++ b/dlls/hidclass.sys/pnp.c @@ -117,6 +117,8 @@ static void send_wm_input_device_change(BASE_DEVICE_EXTENSION *ext, LPARAM param RAWINPUT rawinput; INPUT input; + TRACE("ext %p, lparam %p\n", ext, (void *)param); + if (!IsEqualGUID( ext->class_guid, &GUID_DEVINTERFACE_HID )) return; rawinput.header.dwType = RIM_TYPEHID; @@ -298,6 +300,8 @@ static void create_child(minidriver *minidriver, DEVICE_OBJECT *fdo) HID_StartDeviceThread(child_pdo); send_wm_input_device_change(pdo_ext, GIDC_ARRIVAL); + + TRACE( "created device %p, rawinput handle %#x\n", pdo_ext, pdo_ext->u.pdo.rawinput_handle ); } static NTSTATUS fdo_pnp(DEVICE_OBJECT *device, IRP *irp)

1 0

Rémi Bernon : winebus.sys: Add more trace messages.
by Alexandre Julliard Nov. 22, 2022

Nov. 22, 2022

Module: wine Branch: master Commit: 1429d211ac0c1df9fb65691b0443f08820f05dd4 URL: https://gitlab.winehq.org/wine/wine/-/commit/1429d211ac0c1df9fb65691b0443f0… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Mon Jan 3 17:23:38 2022 +0100 winebus.sys: Add more trace messages. --- dlls/winebus.sys/main.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/dlls/winebus.sys/main.c b/dlls/winebus.sys/main.c index 1037295c955..35f84bd9e29 100644 --- a/dlls/winebus.sys/main.c +++ b/dlls/winebus.sys/main.c @@ -322,6 +322,8 @@ static DEVICE_OBJECT *bus_create_hid_device(struct device_desc *desc, UINT64 uni list_add_tail(&device_list, &ext->entry); RtlLeaveCriticalSection(&device_list_cs); + + TRACE("created device %p/%#I64x\n", device, unix_device); return device; } @@ -415,7 +417,7 @@ static BOOL deliver_next_report(struct device_extension *ext, IRP *irp) if (TRACE_ON(hid)) { - TRACE("read input report length %lu:\n", report->length); + TRACE("device %p/%#I64x input report length %lu:\n", ext->device, ext->unix_device, report->length); for (i = 0; i < report->length;) { char buffer[256], *buf = buffer;

1 0

Nikolay Sivov : comctl32/hotkey: Use CRT allocation functions.
by Alexandre Julliard Nov. 22, 2022

Nov. 22, 2022

Module: wine Branch: master Commit: 644143d55366d044c206266b3908e109ac2ad031 URL: https://gitlab.winehq.org/wine/wine/-/commit/644143d55366d044c206266b3908e1… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Tue Nov 22 12:25:29 2022 +0300 comctl32/hotkey: Use CRT allocation functions. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> --- dlls/comctl32/hotkey.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/dlls/comctl32/hotkey.c b/dlls/comctl32/hotkey.c index f27b71e3124..ed2be658cc9 100644 --- a/dlls/comctl32/hotkey.c +++ b/dlls/comctl32/hotkey.c @@ -29,6 +29,7 @@ */ #include <stdarg.h> +#include <stdlib.h> #include <string.h> #include "windef.h" #include "winbase.h" @@ -39,7 +40,6 @@ #include "comctl32.h" #include "uxtheme.h" #include "wine/debug.h" -#include "wine/heap.h" WINE_DEFAULT_DEBUG_CHANNEL(hotkey); @@ -243,7 +243,7 @@ HOTKEY_Destroy (HOTKEY_INFO *infoPtr) { /* free hotkey info data */ SetWindowLongPtrW (infoPtr->hwndSelf, 0, 0); - heap_free (infoPtr); + free (infoPtr); return 0; } @@ -412,7 +412,7 @@ HOTKEY_NCCreate (HWND hwnd, const CREATESTRUCTW *lpcs) dwExStyle | WS_EX_CLIENTEDGE); /* allocate memory for info structure */ - infoPtr = heap_alloc_zero (sizeof(*infoPtr)); + infoPtr = calloc(1, sizeof(*infoPtr)); SetWindowLongPtrW(hwnd, 0, (DWORD_PTR)infoPtr); /* initialize info structure */

1 0

Nikolay Sivov : comctl32/ipaddress: Use CRT allocation functions.
by Alexandre Julliard Nov. 22, 2022

Nov. 22, 2022

Module: wine Branch: master Commit: 389f961bc58bd64a85cc2dc28c18ae941f5291ca URL: https://gitlab.winehq.org/wine/wine/-/commit/389f961bc58bd64a85cc2dc28c18ae… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Tue Nov 22 12:23:32 2022 +0300 comctl32/ipaddress: Use CRT allocation functions. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> --- dlls/comctl32/ipaddress.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/dlls/comctl32/ipaddress.c b/dlls/comctl32/ipaddress.c index 146a5a5ae1f..2bdf4c0503b 100644 --- a/dlls/comctl32/ipaddress.c +++ b/dlls/comctl32/ipaddress.c @@ -39,7 +39,6 @@ #include "vsstyle.h" #include "vssym32.h" #include "wine/debug.h" -#include "wine/heap.h" WINE_DEFAULT_DEBUG_CHANNEL(ipaddress); @@ -210,7 +209,7 @@ static LRESULT IPADDRESS_Create (HWND hwnd, const CREATESTRUCTA *lpCreate) SetWindowLongW (hwnd, GWL_STYLE, GetWindowLongW(hwnd, GWL_STYLE) & ~WS_BORDER); - infoPtr = heap_alloc_zero (sizeof(*infoPtr)); + infoPtr = calloc (1, sizeof(*infoPtr)); if (!infoPtr) return -1; SetWindowLongPtrW (hwnd, 0, (DWORD_PTR)infoPtr); @@ -273,7 +272,7 @@ static LRESULT IPADDRESS_Destroy (IPADDRESS_INFO *infoPtr) SetWindowLongPtrW (infoPtr->Self, 0, 0); theme = GetWindowTheme (infoPtr->Self); CloseThemeData (theme); - heap_free (infoPtr); + free (infoPtr); return 0; }

1 0

Nikolay Sivov : comctl32/updown: Use CRT allocation functions.
by Alexandre Julliard Nov. 22, 2022

Nov. 22, 2022

Module: wine Branch: master Commit: fca468beb45ef313bd7c8184379f098bde705f52 URL: https://gitlab.winehq.org/wine/wine/-/commit/fca468beb45ef313bd7c8184379f09… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Tue Nov 22 12:16:46 2022 +0300 comctl32/updown: Use CRT allocation functions. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> --- dlls/comctl32/updown.c | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/dlls/comctl32/updown.c b/dlls/comctl32/updown.c index 1d00db275f2..f2befd9d448 100644 --- a/dlls/comctl32/updown.c +++ b/dlls/comctl32/updown.c @@ -33,7 +33,6 @@ #include "comctl32.h" #include "uxtheme.h" #include "vssym32.h" -#include "wine/heap.h" #include "wine/debug.h" WINE_DEFAULT_DEBUG_CHANNEL(updown); @@ -902,7 +901,7 @@ static LRESULT WINAPI UpDownWindowProc(HWND hwnd, UINT message, WPARAM wParam, L { CREATESTRUCTW *pcs = (CREATESTRUCTW*)lParam; - infoPtr = heap_alloc_zero(sizeof(*infoPtr)); + infoPtr = calloc(1, sizeof(*infoPtr)); SetWindowLongPtrW (hwnd, 0, (DWORD_PTR)infoPtr); /* initialize the info struct */ @@ -935,9 +934,9 @@ static LRESULT WINAPI UpDownWindowProc(HWND hwnd, UINT message, WPARAM wParam, L break; case WM_DESTROY: - heap_free (infoPtr->AccelVect); + free (infoPtr->AccelVect); UPDOWN_ResetSubclass (infoPtr); - heap_free (infoPtr); + free (infoPtr); SetWindowLongPtrW (hwnd, 0, 0); theme = GetWindowTheme (hwnd); CloseThemeData (theme); @@ -1060,12 +1059,12 @@ static LRESULT WINAPI UpDownWindowProc(HWND hwnd, UINT message, WPARAM wParam, L TRACE("UDM_SETACCEL\n"); if(infoPtr->AccelVect) { - heap_free (infoPtr->AccelVect); + free (infoPtr->AccelVect); infoPtr->AccelCount = 0; infoPtr->AccelVect = 0; } if(wParam==0) return TRUE; - infoPtr->AccelVect = heap_alloc(wParam*sizeof(UDACCEL)); + infoPtr->AccelVect = malloc(wParam*sizeof(UDACCEL)); if(!infoPtr->AccelVect) return FALSE; memcpy(infoPtr->AccelVect, (void*)lParam, wParam*sizeof(UDACCEL)); infoPtr->AccelCount = wParam;

1 0

Nikolay Sivov : comctl32/progress: Use CRT allocation functions.
by Alexandre Julliard Nov. 22, 2022

Nov. 22, 2022

Module: wine Branch: master Commit: 40c9966efefce273708bfc62dd0416e7c0e47fa7 URL: https://gitlab.winehq.org/wine/wine/-/commit/40c9966efefce273708bfc62dd0416… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Tue Nov 22 12:22:08 2022 +0300 comctl32/progress: Use CRT allocation functions. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> --- dlls/comctl32/progress.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/dlls/comctl32/progress.c b/dlls/comctl32/progress.c index f8fc2a14eb0..d8e4e343156 100644 --- a/dlls/comctl32/progress.c +++ b/dlls/comctl32/progress.c @@ -26,6 +26,7 @@ */ #include <stdarg.h> +#include <stdlib.h> #include <string.h> #include "windef.h" #include "winbase.h" @@ -37,7 +38,6 @@ #include "uxtheme.h" #include "vssym32.h" #include "wine/debug.h" -#include "wine/heap.h" WINE_DEFAULT_DEBUG_CHANNEL(progress); @@ -554,7 +554,7 @@ static LRESULT WINAPI ProgressWindowProc(HWND hwnd, UINT message, SWP_FRAMECHANGED | SWP_NOSIZE | SWP_NOMOVE | SWP_NOZORDER | SWP_NOACTIVATE); /* allocate memory for info struct */ - infoPtr = heap_alloc_zero (sizeof(*infoPtr)); + infoPtr = calloc(1, sizeof(*infoPtr)); if (!infoPtr) return -1; SetWindowLongPtrW (hwnd, 0, (DWORD_PTR)infoPtr); @@ -576,7 +576,7 @@ static LRESULT WINAPI ProgressWindowProc(HWND hwnd, UINT message, case WM_DESTROY: TRACE("Progress Ctrl destruction, hwnd=%p\n", hwnd); - heap_free (infoPtr); + free (infoPtr); SetWindowLongPtrW(hwnd, 0, 0); theme = GetWindowTheme (hwnd); CloseThemeData (theme);

1 0

Nikolay Sivov : comctl32/treeview: Use CRT allocation functions.
by Alexandre Julliard Nov. 22, 2022

Nov. 22, 2022

Module: wine Branch: master Commit: 25ad6d759480036adc0f4a607d5e34733450dbc4 URL: https://gitlab.winehq.org/wine/wine/-/commit/25ad6d759480036adc0f4a607d5e34… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Tue Nov 22 12:19:18 2022 +0300 comctl32/treeview: Use CRT allocation functions. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> --- dlls/comctl32/treeview.c | 45 +++++++++++++++++++++------------------------ 1 file changed, 21 insertions(+), 24 deletions(-) diff --git a/dlls/comctl32/treeview.c b/dlls/comctl32/treeview.c index d3a69bb438e..d9ef075489b 100644 --- a/dlls/comctl32/treeview.c +++ b/dlls/comctl32/treeview.c @@ -60,7 +60,6 @@ #include "vssym32.h" #include "wine/debug.h" #include "wine/exception.h" -#include "wine/heap.h" WINE_DEFAULT_DEBUG_CHANNEL(treeview); @@ -549,7 +548,7 @@ TREEVIEW_TVItemFromItem(const TREEVIEW_INFO *infoPtr, UINT mask, TVITEMW *tvItem if (!infoPtr->bNtfUnicode) { tvItem->cchTextMax = WideCharToMultiByte( CP_ACP, 0, item->pszText, -1, NULL, 0, NULL, NULL ); - tvItem->pszText = heap_alloc (tvItem->cchTextMax); + tvItem->pszText = malloc (tvItem->cchTextMax); WideCharToMultiByte( CP_ACP, 0, item->pszText, -1, (LPSTR)tvItem->pszText, tvItem->cchTextMax, 0, 0 ); } else @@ -590,8 +589,8 @@ TREEVIEW_SendTreeviewNotify(const TREEVIEW_INFO *infoPtr, UINT code, UINT action ret = TREEVIEW_SendRealNotify(infoPtr, code, &nmhdr.hdr); if (!infoPtr->bNtfUnicode) { - heap_free(nmhdr.itemOld.pszText); - heap_free(nmhdr.itemNew.pszText); + free(nmhdr.itemOld.pszText); + free(nmhdr.itemNew.pszText); } return ret; } @@ -690,7 +689,7 @@ TREEVIEW_BeginLabelEditNotify(const TREEVIEW_INFO *infoPtr, TREEVIEW_ITEM *editI ret = TREEVIEW_SendRealNotify(infoPtr, TVN_BEGINLABELEDITW, &tvdi.hdr); if (!infoPtr->bNtfUnicode) - heap_free(tvdi.item.pszText); + free(tvdi.item.pszText); return ret; } @@ -736,7 +735,7 @@ TREEVIEW_UpdateDispInfo(const TREEVIEW_INFO *infoPtr, TREEVIEW_ITEM *item, (LPSTR)callback.item.pszText, -1, NULL, 0); buflen = max((len)*sizeof(WCHAR), TEXT_CALLBACK_SIZE); - newText = heap_realloc(item->pszText, buflen); + newText = realloc(item->pszText, buflen); TRACE("returned str %s, len=%d, buflen=%d\n", debugstr_a((LPSTR)callback.item.pszText), len, buflen); @@ -754,7 +753,7 @@ TREEVIEW_UpdateDispInfo(const TREEVIEW_INFO *infoPtr, TREEVIEW_ITEM *item, else { int len = max(lstrlenW(callback.item.pszText) + 1, TEXT_CALLBACK_SIZE); - LPWSTR newText = heap_realloc(item->pszText, len*sizeof(WCHAR)); + LPWSTR newText = realloc(item->pszText, len*sizeof(WCHAR)); TRACE("returned wstr %s, len=%d\n", debugstr_w(callback.item.pszText), len); @@ -777,7 +776,7 @@ TREEVIEW_UpdateDispInfo(const TREEVIEW_INFO *infoPtr, TREEVIEW_ITEM *item, (LPSTR)callback.item.pszText, -1, NULL, 0); buflen = max((len)*sizeof(WCHAR), TEXT_CALLBACK_SIZE); - newText = heap_alloc(buflen); + newText = malloc(buflen); TRACE("same buffer str %s, len=%d, buflen=%d\n", debugstr_a((LPSTR)callback.item.pszText), len, buflen); @@ -790,7 +789,7 @@ TREEVIEW_UpdateDispInfo(const TREEVIEW_INFO *infoPtr, TREEVIEW_ITEM *item, (LPSTR)callback.item.pszText, -1, item->pszText, buflen/sizeof(WCHAR)); item->cchTextMax = buflen/sizeof(WCHAR); - heap_free(oldText); + free(oldText); } } } @@ -998,7 +997,7 @@ TREEVIEW_UpdateSubTree(const TREEVIEW_INFO *infoPtr, TREEVIEW_ITEM *root) static TREEVIEW_ITEM * TREEVIEW_AllocateItem(const TREEVIEW_INFO *infoPtr) { - TREEVIEW_ITEM *newItem = heap_alloc_zero(sizeof(*newItem)); + TREEVIEW_ITEM *newItem = calloc(1, sizeof(*newItem)); if (!newItem) return NULL; @@ -1014,7 +1013,7 @@ TREEVIEW_AllocateItem(const TREEVIEW_INFO *infoPtr) if (DPA_InsertPtr(infoPtr->items, INT_MAX, newItem) == -1) { - heap_free(newItem); + free(newItem); return NULL; } @@ -1039,7 +1038,7 @@ TREEVIEW_FreeItem(TREEVIEW_INFO *infoPtr, TREEVIEW_ITEM *item) infoPtr->dropItem = NULL; if (infoPtr->insertMarkItem == item) infoPtr->insertMarkItem = NULL; - heap_free(item); + free(item); } @@ -1133,13 +1132,13 @@ TREEVIEW_DoSetItemT(const TREEVIEW_INFO *infoPtr, TREEVIEW_ITEM *item, len = MultiByteToWideChar(CP_ACP, 0, (LPSTR)tvItem->pszText, -1, NULL, 0); /* Allocate new block to make pointer comparison in item_changed() work. */ - newText = heap_alloc(len * sizeof(WCHAR)); + newText = malloc(len * sizeof(WCHAR)); if (newText == NULL) return FALSE; callbackClear |= TVIF_TEXT; - heap_free(item->pszText); + free(item->pszText); item->pszText = newText; item->cchTextMax = len; if (isW) @@ -1153,7 +1152,7 @@ TREEVIEW_DoSetItemT(const TREEVIEW_INFO *infoPtr, TREEVIEW_ITEM *item, else { callbackSet |= TVIF_TEXT; - item->pszText = heap_realloc(item->pszText, TEXT_CALLBACK_SIZE * sizeof(WCHAR)); + item->pszText = realloc(item->pszText, TEXT_CALLBACK_SIZE * sizeof(WCHAR)); item->cchTextMax = TEXT_CALLBACK_SIZE; TRACE("setting callback, item %p\n", item); } @@ -1498,7 +1497,7 @@ TREEVIEW_RemoveItem(TREEVIEW_INFO *infoPtr, TREEVIEW_ITEM *item) infoPtr->uNumItems--; if (item->pszText != LPSTR_TEXTCALLBACKW) - heap_free(item->pszText); + free(item->pszText); TREEVIEW_FreeItem(infoPtr, item); } @@ -4086,7 +4085,7 @@ TREEVIEW_EndEditLabelNow(TREEVIEW_INFO *infoPtr, BOOL bCancel) if (!infoPtr->bNtfUnicode) { DWORD len = MultiByteToWideChar( CP_ACP, 0, (LPSTR)tvdi.item.pszText, -1, NULL, 0 ); - newText = heap_alloc(len * sizeof(WCHAR)); + newText = malloc(len * sizeof(WCHAR)); MultiByteToWideChar( CP_ACP, 0, (LPSTR)tvdi.item.pszText, -1, newText, len ); iLength = len - 1; } @@ -4095,11 +4094,11 @@ TREEVIEW_EndEditLabelNow(TREEVIEW_INFO *infoPtr, BOOL bCancel) if (lstrcmpW(newText, editedItem->pszText) != 0) { - WCHAR *ptr = heap_realloc(editedItem->pszText, sizeof(WCHAR)*(iLength + 1)); + WCHAR *ptr = realloc(editedItem->pszText, sizeof(WCHAR)*(iLength + 1)); if (ptr == NULL) { ERR("OutOfMemory, cannot allocate space for label\n"); - if (newText != tmpText) heap_free(newText); + if (newText != tmpText) free(newText); DestroyWindow(infoPtr->hwndEdit); infoPtr->hwndEdit = 0; infoPtr->editItem = NULL; @@ -4113,7 +4112,7 @@ TREEVIEW_EndEditLabelNow(TREEVIEW_INFO *infoPtr, BOOL bCancel) TREEVIEW_ComputeTextWidth(infoPtr, editedItem, 0); } } - if (newText != tmpText) heap_free(newText); + if (newText != tmpText) free(newText); } ShowWindow(infoPtr->hwndEdit, SW_HIDE); @@ -5135,9 +5134,7 @@ TREEVIEW_Create(HWND hwnd, const CREATESTRUCTW *lpcs) TRACE("wnd %p, style %#lx\n", hwnd, GetWindowLongW(hwnd, GWL_STYLE)); - infoPtr = heap_alloc_zero(sizeof(TREEVIEW_INFO)); - - if (infoPtr == NULL) + if (!(infoPtr = calloc(1, sizeof(*infoPtr)))) { ERR("could not allocate info memory!\n"); return 0; @@ -5260,7 +5257,7 @@ TREEVIEW_Destroy(TREEVIEW_INFO *infoPtr) DeleteObject(infoPtr->hUnderlineFont); DeleteObject(infoPtr->hBoldUnderlineFont); DestroyWindow(infoPtr->hwndToolTip); - heap_free(infoPtr); + free(infoPtr); return 0; }

1 0

Nikolay Sivov : comctl32/pager: Use CRT allocation functions.
by Alexandre Julliard Nov. 22, 2022

Nov. 22, 2022

Module: wine Branch: master Commit: a79b761cdd14c2af69e03df9a5db9e7ac8db5047 URL: https://gitlab.winehq.org/wine/wine/-/commit/a79b761cdd14c2af69e03df9a5db9e… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Tue Nov 22 12:15:04 2022 +0300 comctl32/pager: Use CRT allocation functions. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> --- dlls/comctl32/pager.c | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/dlls/comctl32/pager.c b/dlls/comctl32/pager.c index 7d807c8b2c4..42516a34607 100644 --- a/dlls/comctl32/pager.c +++ b/dlls/comctl32/pager.c @@ -53,6 +53,7 @@ */ #include <stdarg.h> +#include <stdlib.h> #include <string.h> #include "windef.h" #include "winbase.h" @@ -62,7 +63,6 @@ #include "commctrl.h" #include "comctl32.h" #include "wine/debug.h" -#include "wine/heap.h" WINE_DEFAULT_DEBUG_CHANNEL(pager); @@ -576,7 +576,7 @@ PAGER_Create (HWND hwnd, const CREATESTRUCTW *lpcs) INT ret; /* allocate memory for info structure */ - infoPtr = heap_alloc_zero (sizeof(*infoPtr)); + infoPtr = calloc(1, sizeof(*infoPtr)); if (!infoPtr) return -1; SetWindowLongPtrW (hwnd, 0, (DWORD_PTR)infoPtr); @@ -611,8 +611,8 @@ static LRESULT PAGER_Destroy (PAGER_INFO *infoPtr) { SetWindowLongPtrW (infoPtr->hwndSelf, 0, 0); - heap_free (infoPtr->pwszBuffer); - heap_free (infoPtr); + free (infoPtr->pwszBuffer); + free (infoPtr); return 0; } @@ -1099,9 +1099,9 @@ static UINT PAGER_GetAnsiNtfCode(UINT code) static BOOL PAGER_AdjustBuffer(PAGER_INFO *infoPtr, INT size) { if (!infoPtr->pwszBuffer) - infoPtr->pwszBuffer = heap_alloc(size); + infoPtr->pwszBuffer = malloc(size); else if (infoPtr->nBufferSize < size) - infoPtr->pwszBuffer = heap_realloc(infoPtr->pwszBuffer, size); + infoPtr->pwszBuffer = realloc(infoPtr->pwszBuffer, size); if (!infoPtr->pwszBuffer) return FALSE; if (infoPtr->nBufferSize < size) infoPtr->nBufferSize = size; @@ -1153,7 +1153,7 @@ static LRESULT PAGER_SendConvertedNotify(PAGER_INFO *infoPtr, NMHDR *hdr, UINT * if ((*text && flags & (CONVERT_SEND | ZERO_SEND)) || (!*text && flags & SEND_EMPTY_IF_NULL)) { bufferSize = textMax ? *textMax : lstrlenW(*text) + 1; - sendBuffer = heap_alloc_zero(bufferSize); + sendBuffer = calloc(1, bufferSize); if (!sendBuffer) goto done; if (!(flags & ZERO_SEND)) WideCharToMultiByte(CP_ACP, 0, *text, -1, sendBuffer, bufferSize, NULL, FALSE); *text = (WCHAR *)sendBuffer; @@ -1167,18 +1167,18 @@ static LRESULT PAGER_SendConvertedNotify(PAGER_INFO *infoPtr, NMHDR *hdr, UINT * if (*text == oldText) { bufferSize = lstrlenA((CHAR *)*text) + 1; - receiveBuffer = heap_alloc(bufferSize); + receiveBuffer = malloc(bufferSize); if (!receiveBuffer) goto done; memcpy(receiveBuffer, *text, bufferSize); MultiByteToWideChar(CP_ACP, 0, receiveBuffer, bufferSize, oldText, oldTextMax); - heap_free(receiveBuffer); + free(receiveBuffer); } else MultiByteToWideChar(CP_ACP, 0, (CHAR *)*text, -1, oldText, oldTextMax); } done: - heap_free(sendBuffer); + free(sendBuffer); *text = oldText; return ret; }

1 0

Nikolay Sivov : comctl32/animate: Use CRT allocation functions.
by Alexandre Julliard Nov. 22, 2022

Nov. 22, 2022

Module: wine Branch: master Commit: 80de4101393cdba01209cb383a74f691f19efcc0 URL: https://gitlab.winehq.org/wine/wine/-/commit/80de4101393cdba01209cb383a74f6… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Tue Nov 22 12:12:26 2022 +0300 comctl32/animate: Use CRT allocation functions. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> --- dlls/comctl32/animate.c | 35 ++++++++++++++++------------------- 1 file changed, 16 insertions(+), 19 deletions(-) diff --git a/dlls/comctl32/animate.c b/dlls/comctl32/animate.c index adac333b663..87687fce7b1 100644 --- a/dlls/comctl32/animate.c +++ b/dlls/comctl32/animate.c @@ -36,7 +36,6 @@ #include "mmsystem.h" #include "comctl32.h" #include "wine/debug.h" -#include "wine/heap.h" WINE_DEFAULT_DEBUG_CHANNEL(animate); @@ -188,20 +187,20 @@ static void ANIMATE_Free(ANIMATE_INFO *infoPtr) FreeResource(infoPtr->hRes); infoPtr->hRes = 0; } - heap_free (infoPtr->lpIndex); + free (infoPtr->lpIndex); infoPtr->lpIndex = NULL; if (infoPtr->hic) { fnIC.fnICClose(infoPtr->hic); infoPtr->hic = 0; } - heap_free (infoPtr->inbih); + free (infoPtr->inbih); infoPtr->inbih = NULL; - heap_free (infoPtr->outbih); + free (infoPtr->outbih); infoPtr->outbih = NULL; - heap_free (infoPtr->indata); + free (infoPtr->indata); infoPtr->indata = NULL; - heap_free (infoPtr->outdata); + free (infoPtr->outdata); infoPtr->outdata = NULL; if (infoPtr->hbmPrevFrame) { @@ -565,7 +564,7 @@ static BOOL ANIMATE_GetAviInfo(ANIMATE_INFO *infoPtr) return FALSE; } - infoPtr->inbih = heap_alloc_zero(mmckInfo.cksize); + infoPtr->inbih = calloc(1, mmckInfo.cksize); if (!infoPtr->inbih) { WARN("Can't alloc input BIH\n"); return FALSE; @@ -612,7 +611,7 @@ static BOOL ANIMATE_GetAviInfo(ANIMATE_INFO *infoPtr) /* FIXME: should handle the 'rec ' LIST when present */ - infoPtr->lpIndex = heap_alloc_zero(infoPtr->mah.dwTotalFrames * sizeof(DWORD)); + infoPtr->lpIndex = calloc(infoPtr->mah.dwTotalFrames, sizeof(DWORD)); if (!infoPtr->lpIndex) return FALSE; @@ -636,7 +635,7 @@ static BOOL ANIMATE_GetAviInfo(ANIMATE_INFO *infoPtr) infoPtr->ash.dwSuggestedBufferSize = insize; } - infoPtr->indata = heap_alloc_zero(infoPtr->ash.dwSuggestedBufferSize); + infoPtr->indata = calloc(1, infoPtr->ash.dwSuggestedBufferSize); if (!infoPtr->indata) return FALSE; @@ -667,9 +666,8 @@ static BOOL ANIMATE_GetAviCodec(ANIMATE_INFO *infoPtr) outSize = fnIC.fnICSendMessage(infoPtr->hic, ICM_DECOMPRESS_GET_FORMAT, (DWORD_PTR)infoPtr->inbih, 0L); - infoPtr->outbih = heap_alloc_zero(outSize); - if (!infoPtr->outbih) - return FALSE; + if (!(infoPtr->outbih = calloc(1, outSize))) + return FALSE; if (fnIC.fnICSendMessage(infoPtr->hic, ICM_DECOMPRESS_GET_FORMAT, (DWORD_PTR)infoPtr->inbih, (DWORD_PTR)infoPtr->outbih) != ICERR_OK) @@ -678,9 +676,8 @@ static BOOL ANIMATE_GetAviCodec(ANIMATE_INFO *infoPtr) return FALSE; } - infoPtr->outdata = heap_alloc_zero(infoPtr->outbih->biSizeImage); - if (!infoPtr->outdata) - return FALSE; + if (!(infoPtr->outdata = calloc(1, infoPtr->outbih->biSizeImage))) + return FALSE; if (fnIC.fnICSendMessage(infoPtr->hic, ICM_DECOMPRESS_BEGIN, (DWORD_PTR)infoPtr->inbih, (DWORD_PTR)infoPtr->outbih) != ICERR_OK) { @@ -772,12 +769,12 @@ static BOOL ANIMATE_OpenA(ANIMATE_INFO *infoPtr, HINSTANCE hInstance, LPSTR lpsz return ANIMATE_OpenW(infoPtr, hInstance, (LPWSTR)lpszName); len = MultiByteToWideChar(CP_ACP, 0, lpszName, -1, NULL, 0); - lpwszName = heap_alloc(len * sizeof(WCHAR)); + lpwszName = malloc(len * sizeof(WCHAR)); if (!lpwszName) return FALSE; MultiByteToWideChar(CP_ACP, 0, lpszName, -1, lpwszName, len); result = ANIMATE_OpenW(infoPtr, hInstance, lpwszName); - heap_free (lpwszName); + free (lpwszName); return result; } @@ -809,7 +806,7 @@ static BOOL ANIMATE_Create(HWND hWnd, const CREATESTRUCTW *lpcs) } /* allocate memory for info structure */ - infoPtr = heap_alloc_zero(sizeof(*infoPtr)); + infoPtr = calloc(1, sizeof(*infoPtr)); if (!infoPtr) return FALSE; /* store crossref hWnd <-> info structure */ @@ -839,7 +836,7 @@ static LRESULT ANIMATE_Destroy(ANIMATE_INFO *infoPtr) infoPtr->cs.DebugInfo->Spare[0] = 0; DeleteCriticalSection(&infoPtr->cs); - heap_free(infoPtr); + free(infoPtr); return 0; }

1 0

Ivo Ivanov : dinput: Remove redundant physical range values.
by Alexandre Julliard Nov. 22, 2022

Nov. 22, 2022

Module: wine Branch: master Commit: 6b1b5211934aa9e5302de2bde19d018c8fb59a6b URL: https://gitlab.winehq.org/wine/wine/-/commit/6b1b5211934aa9e5302de2bde19d01… Author: Ivo Ivanov <logos128(a)gmail.com> Date: Thu Aug 18 05:27:32 2022 +0300 dinput: Remove redundant physical range values. The physical range is not correct for the given usage. Signed-off-by: Ivo Ivanov <logos128(a)gmail.com> --- dlls/dinput/joystick_hid.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/dlls/dinput/joystick_hid.c b/dlls/dinput/joystick_hid.c index 3ea15009d8f..2065fe429eb 100644 --- a/dlls/dinput/joystick_hid.c +++ b/dlls/dinput/joystick_hid.c @@ -1845,8 +1845,6 @@ static BOOL init_pid_caps( struct hid_joystick *impl, struct hid_value_caps *cap if (instance->wCollectionNumber == effect_update->axes_coll) { SET_REPORT_ID( effect_update ); - caps->physical_min = 0; - caps->physical_max = 36000; if (effect_update->axis_count >= 6) FIXME( "more than 6 PID axes detected\n" ); else effect_update->axis_caps[effect_update->axis_count] = caps; effect_update->axis_count++;

1 0