- Wine-Commits - list.winehq.org

Alexandre Julliard : msvcrt: Use RVAs in rtti and exception data on all platforms except i386.
by Alexandre Julliard May 22, 2024

May 22, 2024

Module: wine Branch: master Commit: a794b0224e27f91bf0855cd7376d6af3de44523d URL: https://gitlab.winehq.org/wine/wine/-/commit/a794b0224e27f91bf0855cd7376d6a… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed May 22 13:50:39 2024 +0200 msvcrt: Use RVAs in rtti and exception data on all platforms except i386. --- dlls/concrt140/concrt140.c | 2 +- dlls/concrt140/details.c | 2 +- dlls/msvcr90/tests/msvcr90.c | 10 +++++----- dlls/msvcrt/concurrency.c | 2 +- dlls/msvcrt/cpp.c | 2 +- dlls/msvcrt/cxx.h | 2 +- dlls/msvcrt/msvcrt.h | 3 +-- dlls/msvcrt/tests/cpp.c | 4 ++-- 8 files changed, 13 insertions(+), 14 deletions(-) diff --git a/dlls/concrt140/concrt140.c b/dlls/concrt140/concrt140.c index 72d8da0dd62..75b4ae24581 100644 --- a/dlls/concrt140/concrt140.c +++ b/dlls/concrt140/concrt140.c @@ -140,7 +140,7 @@ BOOL WINAPI DllMain(HINSTANCE inst, DWORD reason, LPVOID reserved) { case DLL_PROCESS_ATTACH: if (!init_cxx_funcs()) return FALSE; -#ifdef __x86_64__ +#ifdef RTTI_USE_RVA init_exception_rtti((char*)inst); init_range_error_rtti((char*)inst); init_runtime_error_rtti((char*)inst); diff --git a/dlls/concrt140/details.c b/dlls/concrt140/details.c index 978a7a8f96e..a8e7478b9a6 100644 --- a/dlls/concrt140/details.c +++ b/dlls/concrt140/details.c @@ -1100,7 +1100,7 @@ __ASM_BLOCK_END void init_concurrency_details(void *base) { -#ifdef __x86_64__ +#ifdef RTTI_USE_RVA init__Concurrent_queue_base_v4_rtti(base); init__Runtime_object_rtti(base); #endif diff --git a/dlls/msvcr90/tests/msvcr90.c b/dlls/msvcr90/tests/msvcr90.c index 0b274fd694f..6a584d81b04 100644 --- a/dlls/msvcr90/tests/msvcr90.c +++ b/dlls/msvcr90/tests/msvcr90.c @@ -234,7 +234,7 @@ typedef struct typedef void (*cxx_copy_ctor)(void); /* complete information about a C++ type */ -#ifndef __x86_64__ +#ifdef __i386__ typedef struct __cxx_type_info { UINT flags; /* flags (see CLASS_* flags below) */ @@ -255,7 +255,7 @@ typedef struct __cxx_type_info #endif /* table of C++ types that apply for a given object */ -#ifndef __x86_64__ +#ifdef __i386__ typedef struct __cxx_type_info_table { UINT count; /* number of types */ @@ -270,7 +270,7 @@ typedef struct __cxx_type_info_table #endif /* type information for an exception object */ -#ifndef __x86_64__ +#ifdef __i386__ typedef struct __cxx_exception_type { UINT flags; /* TYPE_FLAG flags */ @@ -1489,7 +1489,7 @@ static void test_access_s(void) ok(errno == ENOENT, "got %x\n", res); } -#ifndef __x86_64__ +#ifdef __i386__ #define EXCEPTION_REF(instance, name) &instance.name #else #define EXCEPTION_REF(instance, name) FIELD_OFFSET(struct _exception_data, name) @@ -1521,7 +1521,7 @@ static void test_is_exception_typeof(void) int ret; -#ifdef __x86_64__ +#ifndef __i386__ rec.NumberParameters = 4; rec.ExceptionInformation[3] = (ULONG_PTR)&exception_data; #endif diff --git a/dlls/msvcrt/concurrency.c b/dlls/msvcrt/concurrency.c index f075c685326..39059d66b38 100644 --- a/dlls/msvcrt/concurrency.c +++ b/dlls/msvcrt/concurrency.c @@ -3775,7 +3775,7 @@ __ASM_BLOCK_END void msvcrt_init_concurrency(void *base) { -#ifdef __x86_64__ +#ifdef RTTI_USE_RVA init_cexception_rtti(base); init_improper_lock_rtti(base); init_improper_scheduler_attach_rtti(base); diff --git a/dlls/msvcrt/cpp.c b/dlls/msvcrt/cpp.c index 7e0d3f69396..2db4ef32e99 100644 --- a/dlls/msvcrt/cpp.c +++ b/dlls/msvcrt/cpp.c @@ -573,7 +573,7 @@ DEFINE_CXX_DATA1( bad_alloc, &exception_cxx_type_info, bad_alloc_dtor ) void msvcrt_init_exception(void *base) { -#ifdef __x86_64__ +#ifdef RTTI_USE_RVA init_type_info_rtti(base); init_exception_rtti(base); #if _MSVCR_VER >= 80 diff --git a/dlls/msvcrt/cxx.h b/dlls/msvcrt/cxx.h index c0b43f78256..8037d7def56 100644 --- a/dlls/msvcrt/cxx.h +++ b/dlls/msvcrt/cxx.h @@ -21,7 +21,7 @@ #include "rtlsupportapi.h" #include "wine/asm.h" -#ifndef __x86_64__ +#ifdef __i386__ #undef RTTI_USE_RVA #else #define RTTI_USE_RVA 1 diff --git a/dlls/msvcrt/msvcrt.h b/dlls/msvcrt/msvcrt.h index fe626e2e79f..dd7fb5895a0 100644 --- a/dlls/msvcrt/msvcrt.h +++ b/dlls/msvcrt/msvcrt.h @@ -117,11 +117,10 @@ BOOL __cdecl __CxxRegisterExceptionObject(EXCEPTION_POINTERS*, cxx_frame_info*); void __cdecl __CxxUnregisterExceptionObject(cxx_frame_info*, BOOL); void CDECL __DestructExceptionObject(EXCEPTION_RECORD*); -#if defined(__x86_64__) && _MSVCR_VER>=140 void** __cdecl __current_exception(void); int* __cdecl __processing_throw(void); -void __cdecl terminate(void); +#if defined(__x86_64__) && _MSVCR_VER>=140 BOOL msvcrt_init_handler4(void); void msvcrt_attach_handler4(void); void msvcrt_free_handler4(void); diff --git a/dlls/msvcrt/tests/cpp.c b/dlls/msvcrt/tests/cpp.c index 29bb126bcda..bfc092e0591 100644 --- a/dlls/msvcrt/tests/cpp.c +++ b/dlls/msvcrt/tests/cpp.c @@ -869,7 +869,7 @@ static inline void/*rtti_object_locator*/ *get_obj_locator( void *cppobj ) return (void *)vtable[-1]; } -#ifndef __x86_64__ +#ifdef __i386__ #define DEFINE_RTTI_REF(type, name) type *name #define RTTI_REF(instance, name) &instance.name #define RTTI_REF_SIG0(instance, name, base) RTTI_REF(instance, name) @@ -970,7 +970,7 @@ static void test_rtti(void) exception e,b; void *casted; BOOL old_signature; -#ifdef __x86_64__ +#ifndef __i386__ char *base = (char*)GetModuleHandleW(NULL); #endif

1 0

Esme Povirk : gdiplus: Don't call GetDeviceCaps for NULL dc.
by Alexandre Julliard May 21, 2024

May 21, 2024

Module: wine Branch: master Commit: d41a1f0183f5686fa3ed1b42c4de07e393833925 URL: https://gitlab.winehq.org/wine/wine/-/commit/d41a1f0183f5686fa3ed1b42c4de07… Author: Esme Povirk <esme(a)codeweavers.com> Date: Mon May 6 11:52:56 2024 -0500 gdiplus: Don't call GetDeviceCaps for NULL dc. --- dlls/gdiplus/graphics.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/dlls/gdiplus/graphics.c b/dlls/gdiplus/graphics.c index 58fa129b504..14e09e7cc47 100644 --- a/dlls/gdiplus/graphics.c +++ b/dlls/gdiplus/graphics.c @@ -519,7 +519,8 @@ static GpStatus alpha_blend_hdc_pixels(GpGraphics *graphics, INT dst_x, INT dst_ if(!hbitmap || !temp_bits) goto done; - if ((GetDeviceCaps(graphics->hdc, TECHNOLOGY) == DT_RASPRINTER && + if ((graphics->hdc && + GetDeviceCaps(graphics->hdc, TECHNOLOGY) == DT_RASPRINTER && GetDeviceCaps(graphics->hdc, SHADEBLENDCAPS) == SB_NONE) || fmt & PixelFormatPAlpha) memcpy(temp_bits, src, src_width * src_height * 4);

1 0

Esme Povirk : gdiplus: Do not store HDC on HWND Graphics objects.
by Alexandre Julliard May 21, 2024

May 21, 2024

Module: wine Branch: master Commit: f20a1bb94fd567b39b3cce1a52c88edc7efca811 URL: https://gitlab.winehq.org/wine/wine/-/commit/f20a1bb94fd567b39b3cce1a52c88e… Author: Esme Povirk <esme(a)codeweavers.com> Date: Mon May 6 11:47:09 2024 -0500 gdiplus: Do not store HDC on HWND Graphics objects. --- dlls/gdiplus/gdiplus_private.h | 2 +- dlls/gdiplus/graphics.c | 38 ++++++++++++++++++++++++++++++++++---- 2 files changed, 35 insertions(+), 5 deletions(-) diff --git a/dlls/gdiplus/gdiplus_private.h b/dlls/gdiplus/gdiplus_private.h index e25f60de78e..d247b973b3b 100644 --- a/dlls/gdiplus/gdiplus_private.h +++ b/dlls/gdiplus/gdiplus_private.h @@ -665,7 +665,7 @@ static inline void image_unlock(GpImage *image) static inline BOOL has_gdi_dc(GpGraphics *graphics) { - return graphics->hdc != NULL; + return graphics->hdc != NULL || graphics->owndc; } static inline void set_rect(GpRectF *rect, REAL x, REAL y, REAL width, REAL height) diff --git a/dlls/gdiplus/graphics.c b/dlls/gdiplus/graphics.c index d50379e5943..58fa129b504 100644 --- a/dlls/gdiplus/graphics.c +++ b/dlls/gdiplus/graphics.c @@ -55,6 +55,14 @@ GpStatus gdi_dc_acquire(GpGraphics *graphics, HDC *hdc) graphics->hdc_refs++; return Ok; } + else if (graphics->owndc) + { + *hdc = graphics->hdc = GetDC(graphics->hwnd); + if (!graphics->hdc) + return OutOfMemory; + graphics->hdc_refs++; + return Ok; + } *hdc = NULL; return InvalidParameter; @@ -64,6 +72,13 @@ void gdi_dc_release(GpGraphics *graphics, HDC hdc) { assert(graphics->hdc_refs > 0); graphics->hdc_refs--; + + if (graphics->owndc && !graphics->hdc_refs) + { + assert(graphics->hdc == hdc); + graphics->hdc = NULL; + ReleaseDC(graphics->hwnd, hdc); + } } static GpStatus draw_driver_string(GpGraphics *graphics, GDIPCONST UINT16 *text, INT length, @@ -2556,6 +2571,9 @@ GpStatus WINGDIPAPI GdipCreateFromHWND(HWND hwnd, GpGraphics **graphics) (*graphics)->hwnd = hwnd; (*graphics)->owndc = TRUE; + ReleaseDC(hwnd, hdc); + (*graphics)->hdc = NULL; + return Ok; } @@ -2610,13 +2628,13 @@ GpStatus WINGDIPAPI GdipDeleteGraphics(GpGraphics *graphics) if (graphics->temp_hdc) { - DeleteDC(graphics->temp_hdc); + if (graphics->owndc) + ReleaseDC(graphics->hwnd, graphics->temp_hdc); + else + DeleteDC(graphics->temp_hdc); graphics->temp_hdc = NULL; } - if(graphics->owndc) - ReleaseDC(graphics->hwnd, graphics->hdc); - LIST_FOR_EACH_ENTRY_SAFE(cont, next, &graphics->containers, GraphicsContainerItem, entry){ list_remove(&cont->entry); delete_container(cont); @@ -7002,6 +7020,13 @@ GpStatus WINGDIPAPI GdipGetDC(GpGraphics *graphics, HDC *hdc) { stat = METAFILE_GetDC((GpMetafile*)graphics->image, hdc); } + else if (graphics->owndc) + { + graphics->temp_hdc = GetDC(graphics->hwnd); + if (!graphics->temp_hdc) + return OutOfMemory; + *hdc = graphics->temp_hdc; + } else if (!graphics->hdc || (graphics->image && graphics->image->type == ImageTypeBitmap)) { @@ -7083,6 +7108,11 @@ GpStatus WINGDIPAPI GdipReleaseDC(GpGraphics *graphics, HDC hdc) { stat = METAFILE_ReleaseDC((GpMetafile*)graphics->image, hdc); } + else if (graphics->owndc) + { + ReleaseDC(graphics->hwnd, graphics->temp_hdc); + graphics->temp_hdc = NULL; + } else if (graphics->temp_hdc == hdc) { DWORD* pos;

1 0

Esme Povirk : gdiplus: Bracket HDC use in gdi_transform_acquire/release.
by Alexandre Julliard May 21, 2024

May 21, 2024

Module: wine Branch: master Commit: 8c04e7c0cceb34241b7276aee358987620d4502f URL: https://gitlab.winehq.org/wine/wine/-/commit/8c04e7c0cceb34241b7276aee35898… Author: Esme Povirk <esme(a)codeweavers.com> Date: Sat May 4 21:07:29 2024 +0000 gdiplus: Bracket HDC use in gdi_transform_acquire/release. --- dlls/gdiplus/graphics.c | 22 +++++++++++++++------- 1 file changed, 15 insertions(+), 7 deletions(-) diff --git a/dlls/gdiplus/graphics.c b/dlls/gdiplus/graphics.c index 4dd33e5616b..d50379e5943 100644 --- a/dlls/gdiplus/graphics.c +++ b/dlls/gdiplus/graphics.c @@ -7154,14 +7154,21 @@ GpStatus WINGDIPAPI GdipGetClip(GpGraphics *graphics, GpRegion *region) GpStatus gdi_transform_acquire(GpGraphics *graphics) { - if (graphics->gdi_transform_acquire_count == 0 && graphics->hdc) + if (graphics->gdi_transform_acquire_count == 0 && has_gdi_dc(graphics)) { - graphics->gdi_transform_save = SaveDC(graphics->hdc); - ModifyWorldTransform(graphics->hdc, NULL, MWT_IDENTITY); - SetGraphicsMode(graphics->hdc, GM_COMPATIBLE); - SetMapMode(graphics->hdc, MM_TEXT); - SetWindowOrgEx(graphics->hdc, 0, 0, NULL); - SetViewportOrgEx(graphics->hdc, 0, 0, NULL); + HDC hdc; + GpStatus stat; + + stat = gdi_dc_acquire(graphics, &hdc); + if (stat != Ok) + return stat; + + graphics->gdi_transform_save = SaveDC(hdc); + ModifyWorldTransform(hdc, NULL, MWT_IDENTITY); + SetGraphicsMode(hdc, GM_COMPATIBLE); + SetMapMode(hdc, MM_TEXT); + SetWindowOrgEx(hdc, 0, 0, NULL); + SetViewportOrgEx(hdc, 0, 0, NULL); } graphics->gdi_transform_acquire_count++; return Ok; @@ -7177,6 +7184,7 @@ GpStatus gdi_transform_release(GpGraphics *graphics) if (graphics->gdi_transform_acquire_count == 1 && graphics->hdc) { RestoreDC(graphics->hdc, graphics->gdi_transform_save); + gdi_dc_release(graphics, graphics->hdc); } graphics->gdi_transform_acquire_count--; return Ok;

1 0

Rémi Bernon : dinput: Dynamically allocate the internal device / event arrays.
by Alexandre Julliard May 21, 2024

May 21, 2024

Module: wine Branch: master Commit: 98e6b744e5fd523ee1f47802ea6e6dbf4232a4cf URL: https://gitlab.winehq.org/wine/wine/-/commit/98e6b744e5fd523ee1f47802ea6e6d… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Tue May 21 19:27:10 2024 +0200 dinput: Dynamically allocate the internal device / event arrays. --- dlls/dinput/dinput_main.c | 31 ++++++++++++++++++++++++------- 1 file changed, 24 insertions(+), 7 deletions(-) diff --git a/dlls/dinput/dinput_main.c b/dlls/dinput/dinput_main.c index a3ff8283561..b27339901e5 100644 --- a/dlls/dinput/dinput_main.c +++ b/dlls/dinput/dinput_main.c @@ -45,8 +45,6 @@ WINE_DEFAULT_DEBUG_CHANNEL(dinput); -#define INPUT_THREAD_MAX_DEVICES 128 - #define INPUT_THREAD_NOTIFY (WM_USER + 0x10) #define NOTIFY_THREAD_STOP 0 #define NOTIFY_REFRESH_DEVICES 1 @@ -57,11 +55,12 @@ struct input_thread_state BOOL running; UINT events_count; UINT devices_count; + UINT devices_capacity; HHOOK mouse_ll_hook; HHOOK keyboard_ll_hook; RAWINPUTDEVICE rawinput_devices[2]; - struct dinput_device *devices[INPUT_THREAD_MAX_DEVICES]; - HANDLE events[INPUT_THREAD_MAX_DEVICES]; + struct dinput_device **devices; + HANDLE *events; }; static inline struct dinput_device *impl_from_IDirectInputDevice8W( IDirectInputDevice8W *iface ) @@ -193,10 +192,24 @@ static void input_thread_update_device_list( struct input_thread_state *state ) { RAWINPUTDEVICE rawinput_keyboard = {.usUsagePage = HID_USAGE_PAGE_GENERIC, .usUsage = HID_USAGE_GENERIC_KEYBOARD, .dwFlags = RIDEV_REMOVE}; RAWINPUTDEVICE rawinput_mouse = {.usUsagePage = HID_USAGE_PAGE_GENERIC, .usUsage = HID_USAGE_GENERIC_MOUSE, .dwFlags = RIDEV_REMOVE}; - UINT count = 0, keyboard_ll_count = 0, mouse_ll_count = 0; + UINT count = 0, keyboard_ll_count = 0, mouse_ll_count = 0, capacity; struct dinput_device *device; EnterCriticalSection( &dinput_hook_crit ); + + if ((capacity = list_count( &acquired_device_list )) && capacity > state->devices_capacity) + { + struct dinput_device **devices; + HANDLE *events; + + if (!state->devices_capacity) capacity = max( capacity, 128 ); + else capacity = max( capacity, state->devices_capacity * 3 / 2 ); + + if ((events = realloc( state->events, capacity * sizeof(*events) ))) state->events = events; + if ((devices = realloc( state->devices, capacity * sizeof(*devices) ))) state->devices = devices; + if (events && devices) state->devices_capacity = capacity; + } + LIST_FOR_EACH_ENTRY( device, &acquired_device_list, struct dinput_device, entry ) { unhook_device_window_foreground_changes( device ); @@ -205,7 +218,7 @@ static void input_thread_update_device_list( struct input_thread_state *state ) if (!device->read_event || !device->vtbl->read) continue; state->events[count] = device->read_event; dinput_device_internal_addref( (state->devices[count] = device) ); - if (++count >= INPUT_THREAD_MAX_DEVICES) break; + if (++count >= state->devices_capacity) break; } state->events_count = count; @@ -238,7 +251,8 @@ static void input_thread_update_device_list( struct input_thread_state *state ) rawinput_device->hwndTarget = di_em_win; } - if (count < INPUT_THREAD_MAX_DEVICES) dinput_device_internal_addref( (state->devices[count++] = device) ); + dinput_device_internal_addref( (state->devices[count] = device) ); + if (++count >= state->devices_capacity) break; } state->devices_count = count; LeaveCriticalSection( &dinput_hook_crit ); @@ -402,6 +416,9 @@ static DWORD WINAPI dinput_thread_proc( void *params ) while (state.devices_count--) dinput_device_internal_release( state.devices[state.devices_count] ); if (state.keyboard_ll_hook) UnhookWindowsHookEx( state.keyboard_ll_hook ); if (state.mouse_ll_hook) UnhookWindowsHookEx( state.mouse_ll_hook ); + free( state.devices ); + free( state.events ); + DestroyWindow( di_em_win ); di_em_win = NULL; return 0;

1 0

Esme Povirk : gdiplus: Replace HDC use in draw_driver_string.
by Alexandre Julliard May 21, 2024

May 21, 2024

Module: wine Branch: master Commit: 3c14122fa5bf9bd2ef085ff5c7c731e2f73e9ef5 URL: https://gitlab.winehq.org/wine/wine/-/commit/3c14122fa5bf9bd2ef085ff5c7c731… Author: Esme Povirk <esme(a)codeweavers.com> Date: Sat May 4 20:40:42 2024 +0000 gdiplus: Replace HDC use in draw_driver_string. --- dlls/gdiplus/graphics.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/gdiplus/graphics.c b/dlls/gdiplus/graphics.c index 83fca7a9bfa..4dd33e5616b 100644 --- a/dlls/gdiplus/graphics.c +++ b/dlls/gdiplus/graphics.c @@ -7806,7 +7806,7 @@ static GpStatus draw_driver_string(GpGraphics *graphics, GDIPCONST UINT16 *text, return METAFILE_DrawDriverString((GpMetafile*)graphics->image, text, length, font, format, brush, positions, flags, matrix); - if (graphics->hdc && !graphics->alpha_hdc && + if (has_gdi_dc(graphics) && !graphics->alpha_hdc && brush->bt == BrushTypeSolidColor && (((GpSolidFill*)brush)->color & 0xff000000) == 0xff000000) stat = GDI32_GdipDrawDriverString(graphics, text, length, font, format,

1 0

Esme Povirk : gdiplus: Bracket HDC use in get_path_hrgn.
by Alexandre Julliard May 21, 2024

May 21, 2024

Module: wine Branch: master Commit: bc78d54d35738e9de7451fd24435396e65138c17 URL: https://gitlab.winehq.org/wine/wine/-/commit/bc78d54d35738e9de7451fd2443539… Author: Esme Povirk <esme(a)codeweavers.com> Date: Sat May 4 20:57:02 2024 +0000 gdiplus: Bracket HDC use in get_path_hrgn. --- dlls/gdiplus/region.c | 27 +++++++++++++++++---------- 1 file changed, 17 insertions(+), 10 deletions(-) diff --git a/dlls/gdiplus/region.c b/dlls/gdiplus/region.c index 063ccc96d90..ebab5c0a8a8 100644 --- a/dlls/gdiplus/region.c +++ b/dlls/gdiplus/region.c @@ -990,7 +990,7 @@ GpStatus WINGDIPAPI GdipGetRegionDataSize(GpRegion *region, UINT *needed) static GpStatus get_path_hrgn(GpPath *path, GpGraphics *graphics, HRGN *hrgn) { - HDC new_hdc=NULL; + HDC new_hdc=NULL, hdc; GpGraphics *new_graphics=NULL; GpStatus stat; INT save_state; @@ -1003,7 +1003,7 @@ static GpStatus get_path_hrgn(GpPath *path, GpGraphics *graphics, HRGN *hrgn) if (!graphics) { - new_hdc = CreateCompatibleDC(0); + hdc = new_hdc = CreateCompatibleDC(0); if (!new_hdc) return OutOfMemory; @@ -1015,31 +1015,36 @@ static GpStatus get_path_hrgn(GpPath *path, GpGraphics *graphics, HRGN *hrgn) return stat; } } - else if (!graphics->hdc) + else if (has_gdi_dc(graphics)) { - graphics->hdc = new_hdc = CreateCompatibleDC(0); + stat = gdi_dc_acquire(graphics, &hdc); + if (stat != Ok) + return stat; + } + else + { + graphics->hdc = hdc = new_hdc = CreateCompatibleDC(0); if (!new_hdc) return OutOfMemory; } - save_state = SaveDC(graphics->hdc); - EndPath(graphics->hdc); + save_state = SaveDC(hdc); + EndPath(hdc); - SetPolyFillMode(graphics->hdc, (path->fill == FillModeAlternate ? ALTERNATE - : WINDING)); + SetPolyFillMode(hdc, (path->fill == FillModeAlternate ? ALTERNATE : WINDING)); gdi_transform_acquire(graphics); stat = trace_path(graphics, path); if (stat == Ok) { - *hrgn = PathToRegion(graphics->hdc); + *hrgn = PathToRegion(hdc); stat = *hrgn ? Ok : OutOfMemory; } gdi_transform_release(graphics); - RestoreDC(graphics->hdc, save_state); + RestoreDC(hdc, save_state); if (new_hdc) { DeleteDC(new_hdc); @@ -1048,6 +1053,8 @@ static GpStatus get_path_hrgn(GpPath *path, GpGraphics *graphics, HRGN *hrgn) else graphics->hdc = NULL; } + else + gdi_dc_release(graphics, hdc); return stat; }

1 0

Danyil Blyschak : shcore: Check optional pointer in filestream_CopyTo() before writing to it.
by Alexandre Julliard May 21, 2024

May 21, 2024

Module: wine Branch: master Commit: d28d7bd52cfb8f98835d4792901c12785b3bf0b3 URL: https://gitlab.winehq.org/wine/wine/-/commit/d28d7bd52cfb8f98835d4792901c12… Author: Danyil Blyschak <dblyschak(a)codeweavers.com> Date: Tue May 21 11:15:45 2024 -0500 shcore: Check optional pointer in filestream_CopyTo() before writing to it. Check that the 'written' parameter in this function is non-null before incrementing it by 'written_chunk'. --- dlls/shcore/main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/shcore/main.c b/dlls/shcore/main.c index a64b6661d23..58ddda3fb2a 100644 --- a/dlls/shcore/main.c +++ b/dlls/shcore/main.c @@ -1017,7 +1017,7 @@ static HRESULT WINAPI filestream_CopyTo(IStream *iface, IStream *dest, ULARGE_IN /* Write */ hr = IStream_Write(dest, buff, read_chunk, &written_chunk); - if (written_chunk) + if (written) written->QuadPart += written_chunk; if (FAILED(hr) || written_chunk != left) break;

1 0

Jacek Caban : mshtml: Use DISPEX_IDISPATCH_IMPL macro in htmlstyleelem.c.
by Alexandre Julliard May 21, 2024

May 21, 2024

Module: wine Branch: master Commit: 3ee6f3b9183c3dab8f4a89331a345d88f561e0e0 URL: https://gitlab.winehq.org/wine/wine/-/commit/3ee6f3b9183c3dab8f4a89331a345d… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue May 21 14:36:17 2024 +0200 mshtml: Use DISPEX_IDISPATCH_IMPL macro in htmlstyleelem.c. --- dlls/mshtml/htmlstyleelem.c | 108 ++------------------------------------------ 1 file changed, 4 insertions(+), 104 deletions(-) diff --git a/dlls/mshtml/htmlstyleelem.c b/dlls/mshtml/htmlstyleelem.c index 5b3d9165964..af5831dd85a 100644 --- a/dlls/mshtml/htmlstyleelem.c +++ b/dlls/mshtml/htmlstyleelem.c @@ -49,58 +49,8 @@ static inline HTMLStyleElement *impl_from_IHTMLStyleElement(IHTMLStyleElement *i return CONTAINING_RECORD(iface, HTMLStyleElement, IHTMLStyleElement_iface); } -static HRESULT WINAPI HTMLStyleElement_QueryInterface(IHTMLStyleElement *iface, - REFIID riid, void **ppv) -{ - HTMLStyleElement *This = impl_from_IHTMLStyleElement(iface); - - return IHTMLDOMNode_QueryInterface(&This->element.node.IHTMLDOMNode_iface, riid, ppv); -} - -static ULONG WINAPI HTMLStyleElement_AddRef(IHTMLStyleElement *iface) -{ - HTMLStyleElement *This = impl_from_IHTMLStyleElement(iface); - - return IHTMLDOMNode_AddRef(&This->element.node.IHTMLDOMNode_iface); -} - -static ULONG WINAPI HTMLStyleElement_Release(IHTMLStyleElement *iface) -{ - HTMLStyleElement *This = impl_from_IHTMLStyleElement(iface); - - return IHTMLDOMNode_Release(&This->element.node.IHTMLDOMNode_iface); -} - -static HRESULT WINAPI HTMLStyleElement_GetTypeInfoCount(IHTMLStyleElement *iface, UINT *pctinfo) -{ - HTMLStyleElement *This = impl_from_IHTMLStyleElement(iface); - return IDispatchEx_GetTypeInfoCount(&This->element.node.event_target.dispex.IDispatchEx_iface, pctinfo); -} - -static HRESULT WINAPI HTMLStyleElement_GetTypeInfo(IHTMLStyleElement *iface, UINT iTInfo, - LCID lcid, ITypeInfo **ppTInfo) -{ - HTMLStyleElement *This = impl_from_IHTMLStyleElement(iface); - return IDispatchEx_GetTypeInfo(&This->element.node.event_target.dispex.IDispatchEx_iface, iTInfo, lcid, - ppTInfo); -} - -static HRESULT WINAPI HTMLStyleElement_GetIDsOfNames(IHTMLStyleElement *iface, REFIID riid, - LPOLESTR *rgszNames, UINT cNames, LCID lcid, DISPID *rgDispId) -{ - HTMLStyleElement *This = impl_from_IHTMLStyleElement(iface); - return IDispatchEx_GetIDsOfNames(&This->element.node.event_target.dispex.IDispatchEx_iface, riid, rgszNames, - cNames, lcid, rgDispId); -} - -static HRESULT WINAPI HTMLStyleElement_Invoke(IHTMLStyleElement *iface, DISPID dispIdMember, - REFIID riid, LCID lcid, WORD wFlags, DISPPARAMS *pDispParams, - VARIANT *pVarResult, EXCEPINFO *pExcepInfo, UINT *puArgErr) -{ - HTMLStyleElement *This = impl_from_IHTMLStyleElement(iface); - return IDispatchEx_Invoke(&This->element.node.event_target.dispex.IDispatchEx_iface, dispIdMember, riid, - lcid, wFlags, pDispParams, pVarResult, pExcepInfo, puArgErr); -} +DISPEX_IDISPATCH_IMPL(HTMLStyleElement, IHTMLStyleElement, + impl_from_IHTMLStyleElement(iface)->element.node.event_target.dispex) static HRESULT WINAPI HTMLStyleElement_put_type(IHTMLStyleElement *iface, BSTR v) { @@ -289,58 +239,8 @@ static inline HTMLStyleElement *impl_from_IHTMLStyleElement2(IHTMLStyleElement2 return CONTAINING_RECORD(iface, HTMLStyleElement, IHTMLStyleElement2_iface); } -static HRESULT WINAPI HTMLStyleElement2_QueryInterface(IHTMLStyleElement2 *iface, - REFIID riid, void **ppv) -{ - HTMLStyleElement *This = impl_from_IHTMLStyleElement2(iface); - - return IHTMLDOMNode_QueryInterface(&This->element.node.IHTMLDOMNode_iface, riid, ppv); -} - -static ULONG WINAPI HTMLStyleElement2_AddRef(IHTMLStyleElement2 *iface) -{ - HTMLStyleElement *This = impl_from_IHTMLStyleElement2(iface); - - return IHTMLDOMNode_AddRef(&This->element.node.IHTMLDOMNode_iface); -} - -static ULONG WINAPI HTMLStyleElement2_Release(IHTMLStyleElement2 *iface) -{ - HTMLStyleElement *This = impl_from_IHTMLStyleElement2(iface); - - return IHTMLDOMNode_Release(&This->element.node.IHTMLDOMNode_iface); -} - -static HRESULT WINAPI HTMLStyleElement2_GetTypeInfoCount(IHTMLStyleElement2 *iface, UINT *pctinfo) -{ - HTMLStyleElement *This = impl_from_IHTMLStyleElement2(iface); - return IDispatchEx_GetTypeInfoCount(&This->element.node.event_target.dispex.IDispatchEx_iface, pctinfo); -} - -static HRESULT WINAPI HTMLStyleElement2_GetTypeInfo(IHTMLStyleElement2 *iface, UINT iTInfo, - LCID lcid, ITypeInfo **ppTInfo) -{ - HTMLStyleElement *This = impl_from_IHTMLStyleElement2(iface); - return IDispatchEx_GetTypeInfo(&This->element.node.event_target.dispex.IDispatchEx_iface, iTInfo, lcid, - ppTInfo); -} - -static HRESULT WINAPI HTMLStyleElement2_GetIDsOfNames(IHTMLStyleElement2 *iface, REFIID riid, - LPOLESTR *rgszNames, UINT cNames, LCID lcid, DISPID *rgDispId) -{ - HTMLStyleElement *This = impl_from_IHTMLStyleElement2(iface); - return IDispatchEx_GetIDsOfNames(&This->element.node.event_target.dispex.IDispatchEx_iface, riid, rgszNames, - cNames, lcid, rgDispId); -} - -static HRESULT WINAPI HTMLStyleElement2_Invoke(IHTMLStyleElement2 *iface, DISPID dispIdMember, - REFIID riid, LCID lcid, WORD wFlags, DISPPARAMS *pDispParams, - VARIANT *pVarResult, EXCEPINFO *pExcepInfo, UINT *puArgErr) -{ - HTMLStyleElement *This = impl_from_IHTMLStyleElement2(iface); - return IDispatchEx_Invoke(&This->element.node.event_target.dispex.IDispatchEx_iface, dispIdMember, riid, - lcid, wFlags, pDispParams, pVarResult, pExcepInfo, puArgErr); -} +DISPEX_IDISPATCH_IMPL(HTMLStyleElement2, IHTMLStyleElement2, + impl_from_IHTMLStyleElement2(iface)->element.node.event_target.dispex) static HRESULT WINAPI HTMLStyleElement2_get_sheet(IHTMLStyleElement2 *iface, IHTMLStyleSheet **p) {

1 0

Jacek Caban : mshtml: Use DISPEX_IDISPATCH_IMPL macro in htmlstyle.c.
by Alexandre Julliard May 21, 2024

May 21, 2024

Module: wine Branch: master Commit: 72dccb8f639be24e5cc8979f772a0701826f5aa1 URL: https://gitlab.winehq.org/wine/wine/-/commit/72dccb8f639be24e5cc8979f772a07… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue May 21 14:33:57 2024 +0200 mshtml: Use DISPEX_IDISPATCH_IMPL macro in htmlstyle.c. --- dlls/mshtml/htmlstyle.c | 419 ++---------------------------------------------- 1 file changed, 10 insertions(+), 409 deletions(-)

1 0