- Wine-Commits - list.winehq.org

Myah Caron : windows.ui: Support accent colors in uisettings3_GetColorValue.
by Alexandre Julliard Nov. 16, 2023

Nov. 16, 2023

Module: wine Branch: master Commit: 3448a84cac437f66bf7bdd833a6ce5026fddd4cc URL: https://gitlab.winehq.org/wine/wine/-/commit/3448a84cac437f66bf7bdd833a6ce5… Author: Myah Caron <qsniyg(a)protonmail.com> Date: Fri Sep 22 20:41:23 2023 +0000 windows.ui: Support accent colors in uisettings3_GetColorValue. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=55640 Signed-off-by: Myah Caron <qsniyg(a)protonmail.com> --- dlls/windows.ui/tests/uisettings.c | 95 ++++++++++++++++++++++++++++++++++++++ dlls/windows.ui/uisettings.c | 64 +++++++++++++++++++++++-- 2 files changed, 154 insertions(+), 5 deletions(-) diff --git a/dlls/windows.ui/tests/uisettings.c b/dlls/windows.ui/tests/uisettings.c index 8d739cb7bef..40588f0000f 100644 --- a/dlls/windows.ui/tests/uisettings.c +++ b/dlls/windows.ui/tests/uisettings.c @@ -75,6 +75,45 @@ static DWORD set_app_theme( DWORD mode ) return ret; } +static BOOL get_accent_palette( DWORD *colors, DWORD *len ) +{ + BOOL ret = TRUE; + DWORD type; + HKEY hkey; + + if (RegOpenKeyExW( HKEY_CURRENT_USER, L"Software\\Microsoft\\Windows\\CurrentVersion\\Explorer\\Accent", 0, KEY_QUERY_VALUE, &hkey )) return FALSE; + if (RegQueryValueExW( hkey, L"AccentPalette", NULL, &type, (BYTE *)colors, len ) || type != REG_BINARY) ret = FALSE; + RegCloseKey( hkey ); + return ret; +} + +static BOOL delete_accent_palette(void) +{ + BOOL ret = TRUE; + HRESULT res; + HKEY hkey; + + res = RegOpenKeyExW( HKEY_CURRENT_USER, L"Software\\Microsoft\\Windows\\CurrentVersion\\Explorer\\Accent", 0, KEY_SET_VALUE, &hkey ); + if (res != 0) return res == ERROR_FILE_NOT_FOUND; + + res = RegDeleteValueW( hkey, L"AccentPalette" ); + if (res != 0 && res != ERROR_FILE_NOT_FOUND) ret = FALSE; + + RegCloseKey( hkey ); + return ret; +} + +static DWORD set_accent_palette( DWORD *colors, DWORD len ) +{ + DWORD ret = 1; + HKEY hkey; + + if (RegOpenKeyExW( HKEY_CURRENT_USER, L"Software\\Microsoft\\Windows\\CurrentVersion\\Explorer\\Accent", 0, KEY_SET_VALUE, &hkey )) return 0; + if (RegSetValueExW( hkey, L"AccentPalette", 0, REG_BINARY, (const BYTE *)&colors, len )) ret = 0; + RegCloseKey( hkey ); + return ret; +} + static void reset_color( Color *value ) { value->A = 1; @@ -83,6 +122,60 @@ static void reset_color( Color *value ) value->B = 1; } +static BOOL compare_dword_color( Color color, DWORD dword ) +{ + return (color.R == (dword & 0xFF) && color.G == ((dword >> 8) & 0xFF) && color.B == ((dword >> 16) & 0xFF)); +} + +static void test_single_accent( IUISettings3 *uisettings3, UIColorType type, DWORD expected ) +{ + Color value; + HRESULT hr; + BOOL result; + + reset_color( &value ); + hr = IUISettings3_GetColorValue( uisettings3, type, &value ); + ok( hr == S_OK, "GetColorValue returned %#lx\n", hr ); + + result = compare_dword_color( value, expected ); + ok( result, "got unexpected value.A == %d value.R == %d value.G == %d value.B == %d (type = %d)\n", value.A, value.R, value.G, value.B, type ); +} + +static void test_AccentColor( IUISettings3 *uisettings3 ) +{ + DWORD default_palette[8]; + DWORD default_palette_len = sizeof(default_palette); + DWORD accent_palette[8]; + DWORD accent_palette_len = sizeof(accent_palette); + + if (!get_accent_palette( default_palette, &default_palette_len )) default_palette_len = 0; + + /* deleting AccentPalette fills a default value */ + ok( delete_accent_palette(), "failed to delete AccentPalette key.\n"); + ok( !get_accent_palette( accent_palette, &accent_palette_len ), "AccentPalette should not be available.\n" ); + + test_single_accent( uisettings3, UIColorType_Accent, 0x00d77800 ); + ok( get_accent_palette( accent_palette, &accent_palette_len ), "failed to retrieve AccentPalette key.\n" ); + + /* default values */ + test_single_accent( uisettings3, UIColorType_AccentDark1, 0x009e5a00 ); + test_single_accent( uisettings3, UIColorType_AccentDark2, 0x00754200 ); + test_single_accent( uisettings3, UIColorType_AccentDark3, 0x00422600 ); + test_single_accent( uisettings3, UIColorType_AccentLight1, 0x00e39c42 ); + test_single_accent( uisettings3, UIColorType_AccentLight2, 0x00edb976 ); + test_single_accent( uisettings3, UIColorType_AccentLight3, 0x00ffd8a6 ); + + test_single_accent( uisettings3, UIColorType_Accent, accent_palette[3] ); + test_single_accent( uisettings3, UIColorType_AccentDark1, accent_palette[4] ); + test_single_accent( uisettings3, UIColorType_AccentDark2, accent_palette[5] ); + test_single_accent( uisettings3, UIColorType_AccentDark3, accent_palette[6] ); + test_single_accent( uisettings3, UIColorType_AccentLight1, accent_palette[2] ); + test_single_accent( uisettings3, UIColorType_AccentLight2, accent_palette[1] ); + test_single_accent( uisettings3, UIColorType_AccentLight3, accent_palette[0] ); + + if (default_palette_len) set_accent_palette( default_palette, default_palette_len ); +} + static void test_UISettings(void) { static const WCHAR *uisettings_name = L"Windows.UI.ViewManagement.UISettings"; @@ -126,6 +219,8 @@ static void test_UISettings(void) check_interface( inspectable, &IID_IAgileObject, TRUE ); + test_AccentColor( uisettings3 ); + default_theme = get_app_theme(); /* Light Theme */ diff --git a/dlls/windows.ui/uisettings.c b/dlls/windows.ui/uisettings.c index 041388da33b..b36194a2ac3 100644 --- a/dlls/windows.ui/uisettings.c +++ b/dlls/windows.ui/uisettings.c @@ -266,6 +266,40 @@ static DWORD get_app_theme(void) return ret; } +static DWORD initialize_accent_palette( HKEY hkey, DWORD offset ) +{ + DWORD palette[7] = { 0x00ffd8a6, 0x00edb976, 0x00e39c42, 0x00d77800, 0x009e5a00, 0x00754200, 0x00422600 }; + DWORD len = sizeof(palette); + + RegSetValueExW( hkey, L"AccentPalette", 0, REG_BINARY, (const BYTE *)&palette, len ); + return palette[offset]; +} + +static void dword_to_color( DWORD color, Color *out ) +{ + out->R = color & 0xFF; + out->G = (color >> 8) & 0xFF; + out->B = (color >> 16) & 0xFF; + out->A = 0xFF; +} + +static void get_accent_palette_color( DWORD offset, Color *out ) +{ + DWORD palette[7]; + DWORD ret = 0, len = sizeof(palette), type; + HKEY hkey; + + if (!RegCreateKeyW( HKEY_CURRENT_USER, L"Software\\Microsoft\\Windows\\CurrentVersion\\Explorer\\Accent", &hkey )) + { + if (RegQueryValueExW( hkey, L"AccentPalette", NULL, &type, (BYTE *)&palette, &len ) || type != REG_BINARY) + ret = initialize_accent_palette( hkey, offset ); + else + ret = palette[offset]; + RegCloseKey( hkey ); + } + dword_to_color( ret, out ); +} + static void set_color_value( BYTE a, BYTE r, BYTE g, BYTE b, Color *out ) { out->A = a; @@ -283,19 +317,39 @@ static HRESULT WINAPI uisettings3_GetColorValue( IUISettings3 *iface, UIColorTyp switch (type) { case UIColorType_Foreground: + theme = get_app_theme(); + set_color_value( 255, theme ? 0 : 255, theme ? 0 : 255, theme ? 0 : 255, value ); + break; case UIColorType_Background: theme = get_app_theme(); + set_color_value( 255, theme ? 255 : 0, theme ? 255 : 0, theme ? 255 : 0, value ); + break; + case UIColorType_Accent: + get_accent_palette_color( 3, value ); + break; + case UIColorType_AccentDark1: + get_accent_palette_color( 4, value ); + break; + case UIColorType_AccentDark2: + get_accent_palette_color( 5, value ); + break; + case UIColorType_AccentDark3: + get_accent_palette_color( 6, value ); + break; + case UIColorType_AccentLight1: + get_accent_palette_color( 2, value ); + break; + case UIColorType_AccentLight2: + get_accent_palette_color( 1, value ); + break; + case UIColorType_AccentLight3: + get_accent_palette_color( 0, value ); break; default: FIXME( "type %d not implemented.\n", type ); return E_NOTIMPL; } - if (type == UIColorType_Foreground) - set_color_value( 255, theme ? 0 : 255, theme ? 0 : 255, theme ? 0 : 255, value ); - else - set_color_value( 255, theme ? 255 : 0, theme ? 255 : 0, theme ? 255 : 0, value ); - TRACE( "Returning value.A = %d, value.R = %d, value.G = %d, value.B = %d\n", value->A, value->R, value->G, value->B ); return S_OK; }

1 0

Eric Pouech : dbghelp: Pretend mach-o is present in case of failure.
by Alexandre Julliard Nov. 16, 2023

Nov. 16, 2023

Module: wine Branch: master Commit: 60108ea1bc284c08fd1d6f5ac9f480c7071c8937 URL: https://gitlab.winehq.org/wine/wine/-/commit/60108ea1bc284c08fd1d6f5ac9f480… Author: Eric Pouech <eric.pouech(a)gmail.com> Date: Thu Nov 16 18:38:02 2023 +0100 dbghelp: Pretend mach-o is present in case of failure. On MacOs, starting with Big Sur 11.0.1, the system dynamic libraries are no longer directly accessible on disk. They are still available through dlopen and friends. For getting access to the images (and their debug symbol), Apple provides, in the developper kit, the tools to extract the files. Note that this is handled as a database of all system libraries, where ASLR is in place such that segments of a given library are no longer contiguous in memory (dbghelp doesn't currently handle this). Apart from not having image information nor debug information, another side effect is that dbghelp tries every time it refreshes the mach-o module list to reload any library for which it didn't have an image file. This can be lengthy (esp when a typical process has more than 300 modules loaded). This patch forces the creation of the dbghelp module even if the image file isn't found. This patch cuts startup time of 'winedbg notepad' from 9.9 to 7.4s. YMMV. Signed-off-by: Eric Pouech <epouech(a)codeweavers.com> --- dlls/dbghelp/dbghelp_private.h | 3 ++- dlls/dbghelp/macho_module.c | 45 ++++++++++++++++++++++++++++++++---------- dlls/dbghelp/module.c | 3 +++ include/dbghelp.h | 3 ++- programs/winedbg/info.c | 9 +++++++-- 5 files changed, 49 insertions(+), 14 deletions(-) diff --git a/dlls/dbghelp/dbghelp_private.h b/dlls/dbghelp/dbghelp_private.h index 15f91e683f5..acba5faca37 100644 --- a/dlls/dbghelp/dbghelp_private.h +++ b/dlls/dbghelp/dbghelp_private.h @@ -441,7 +441,8 @@ struct module struct module* next; enum dhext_module_type type : 16; unsigned short is_virtual : 1, - is_wine_builtin : 1; + is_wine_builtin : 1, + has_file_image : 1; struct cpu* cpu; DWORD64 reloc_delta; WCHAR* real_path; diff --git a/dlls/dbghelp/macho_module.c b/dlls/dbghelp/macho_module.c index 2d831449d39..90c38b557bb 100644 --- a/dlls/dbghelp/macho_module.c +++ b/dlls/dbghelp/macho_module.c @@ -1405,6 +1405,9 @@ static BOOL macho_load_debug_info(struct process *pcs, struct module* module) return FALSE; } + if (!module->has_file_image) /* no much more we can do here */ + return FALSE; + ifm = &module->format_info[DFI_MACHO]->u.macho_info->file_map; fmap = &ifm->u.macho; @@ -1489,17 +1492,22 @@ static void macho_module_remove(struct process* pcs, struct module_format* modfm * TRUE on success */ static BOOL macho_load_file(struct process* pcs, const WCHAR* filename, - ULONG_PTR load_addr, struct macho_info* macho_info) + ULONG_PTR load_addr, struct macho_info* macho_info, BOOL with_image) { BOOL ret = TRUE; BOOL split_segs; struct image_file_map fmap; - TRACE("(%p/%p, %s, 0x%08Ix, %p/0x%08x)\n", pcs, pcs->handle, debugstr_w(filename), - load_addr, macho_info, macho_info->flags); + TRACE("(%p/%p, %s, 0x%08Ix, %p/0x%08x, %u)\n", pcs, pcs->handle, debugstr_w(filename), + load_addr, macho_info, macho_info->flags, with_image); split_segs = image_uses_split_segs(pcs, load_addr); - if (!macho_map_file(pcs, filename, split_segs, &fmap)) return FALSE; + if (with_image) + { + if (!macho_map_file(pcs, filename, split_segs, &fmap)) return FALSE; + } + else + reset_file_map(&fmap); if (macho_info->flags & MACHO_INFO_MODULE) { @@ -1507,18 +1515,21 @@ static BOOL macho_load_file(struct process* pcs, const WCHAR* filename, struct module_format* modfmt = HeapAlloc(GetProcessHeap(), 0, sizeof(struct module_format) + sizeof(struct macho_module_info)); if (!modfmt) goto leave; - if (!load_addr) + if (!load_addr && with_image) load_addr = fmap.u.macho.segs_start; macho_info->module = module_new(pcs, filename, DMT_MACHO, module_is_wine_host(filename, L".so"), FALSE, load_addr, - fmap.u.macho.segs_size, 0, calc_crc32(fmap.u.macho.handle), + with_image ? fmap.u.macho.segs_size : 1024, + 0, with_image ? calc_crc32(fmap.u.macho.handle) : 0, image_get_machine(pcs, load_addr)); if (!macho_info->module) { HeapFree(GetProcessHeap(), 0, modfmt); goto leave; } - macho_info->module->reloc_delta = macho_info->module->module.BaseOfImage - fmap.u.macho.segs_start; + macho_info->module->reloc_delta = macho_info->module->module.BaseOfImage; + if (with_image) macho_info->module->reloc_delta -= fmap.u.macho.segs_start; + macho_module_info = (void*)(modfmt + 1); macho_info->module->format_info[DFI_MACHO] = modfmt; @@ -1531,6 +1542,7 @@ static BOOL macho_load_file(struct process* pcs, const WCHAR* filename, macho_module_info->file_map = fmap; reset_file_map(&fmap); + if (!with_image) macho_info->module->has_file_image = 0; macho_info->module->format_info[DFI_MACHO]->u.macho_info->in_use = 1; macho_info->module->format_info[DFI_MACHO]->u.macho_info->is_loader = 0; @@ -1566,7 +1578,7 @@ struct macho_load_params static BOOL macho_load_file_cb(void *param, HANDLE handle, const WCHAR *filename) { struct macho_load_params *macho_load = param; - return macho_load_file(macho_load->process, filename, macho_load->load_addr, macho_load->macho_info); + return macho_load_file(macho_load->process, filename, macho_load->load_addr, macho_load->macho_info, TRUE); } /****************************************************************** @@ -1606,7 +1618,7 @@ static BOOL macho_search_and_load_file(struct process* pcs, const WCHAR* filenam /* Try the path as given. */ if (!ret) - ret = macho_load_file(pcs, filename, load_addr, macho_info); + ret = macho_load_file(pcs, filename, load_addr, macho_info, TRUE); /* Try DYLD_FALLBACK_LIBRARY_PATH, with just the filename (no directories). */ if (!ret) { @@ -1618,6 +1630,19 @@ static BOOL macho_search_and_load_file(struct process* pcs, const WCHAR* filenam if (!ret && p == filename) ret = search_dll_path(pcs, filename, IMAGE_FILE_MACHINE_UNKNOWN, macho_load_file_cb, &load_params); + if (!ret && load_addr) + { + /* Starting at macos 11.0, the system libraries are no longer present on the file system. + * So, if we cannot find an image by its filename, just declare the module without + * any debug information. + * This avoids, when walking the internal module list, to search each time + * for the module filename. + * Note: doesn't seem to be a simple way to get the size of the loaded Mach-O module + * without the corresponding file image. And it has also ASLR in place, + * where segments of the same module are not contiguous. + */ + ret = macho_load_file(pcs, filename, load_addr, macho_info, FALSE); + } return ret; } @@ -1908,7 +1933,7 @@ static BOOL macho_search_loader(struct process* pcs, struct macho_info* macho_in if (pathW) { MultiByteToWideChar(CP_UNIXCP, 0, path, -1, pathW, len); - ret = macho_load_file(pcs, pathW, 0, macho_info); + ret = macho_load_file(pcs, pathW, 0, macho_info, TRUE); HeapFree(GetProcessHeap(), 0, pathW); } } diff --git a/dlls/dbghelp/module.c b/dlls/dbghelp/module.c index b86234daf46..b12007d270e 100644 --- a/dlls/dbghelp/module.c +++ b/dlls/dbghelp/module.c @@ -237,6 +237,8 @@ struct module* module_new(struct process* pcs, const WCHAR* name, module->type = type; module->is_virtual = !!virtual; module->is_wine_builtin = !!builtin; + module->has_file_image = TRUE; + for (i = 0; i < DFI_LAST; i++) module->format_info[i] = NULL; module->sortlist_valid = FALSE; module->sorttab_size = 0; @@ -1687,6 +1689,7 @@ BOOL WINAPI wine_get_module_information(HANDLE proc, DWORD64 base, struct dhext_ dhmi.type = module->type; dhmi.is_virtual = module->is_virtual; dhmi.is_wine_builtin = module->is_wine_builtin; + dhmi.has_file_image = module->has_file_image; dhmi.debug_format_bitmask = module->debug_format_bitmask; if ((module = module_get_container(pcs, module))) { diff --git a/include/dbghelp.h b/include/dbghelp.h index b638447140c..919efb27be0 100644 --- a/include/dbghelp.h +++ b/include/dbghelp.h @@ -1267,7 +1267,8 @@ struct dhext_module_information { enum dhext_module_type type; unsigned is_wine_builtin : 1, - is_virtual : 1; + is_virtual : 1, + has_file_image : 1; unsigned debug_format_bitmask; }; diff --git a/programs/winedbg/info.c b/programs/winedbg/info.c index 7491c4bd5fe..d95b286454a 100644 --- a/programs/winedbg/info.c +++ b/programs/winedbg/info.c @@ -191,9 +191,10 @@ static const char* get_machine_str(DWORD machine) static void module_print_info(const struct info_module *module, BOOL is_embedded, BOOL multi_machine) { char buffer[9]; - snprintf(buffer, sizeof(buffer), "%s%s", + snprintf(buffer, sizeof(buffer), "%s%s%s", is_embedded ? " \\-" : "", - get_module_type(module, is_embedded)); + get_module_type(module, is_embedded), + module->ext_module_info.has_file_image ? "" : "^"); if (multi_machine) dbg_printf("%-8s%16I64x-%16I64x %-16s%-16s%s\n", @@ -266,6 +267,7 @@ void info_win32_module(DWORD64 base, BOOL multi_machine) UINT i, j, num_printed = 0; BOOL opt; DWORD machine; + BOOL has_missing_filename = FALSE; if (!dbg_curr_process) { @@ -308,6 +310,7 @@ void info_win32_module(DWORD64 base, BOOL multi_machine) (base < im.modules[i].mi.BaseOfImage || base >= im.modules[i].mi.BaseOfImage + im.modules[i].mi.ImageSize)) continue; if (!multi_machine && machine != im.modules[i].mi.MachineType) continue; + if (!im.modules[i].ext_module_info.has_file_image) has_missing_filename = TRUE; if (im.modules[i].ext_module_info.type == DMT_ELF || im.modules[i].ext_module_info.type == DMT_MACHO) { module_print_info(&im.modules[i], FALSE, multi_machine); @@ -335,6 +338,8 @@ void info_win32_module(DWORD64 base, BOOL multi_machine) if (base && !num_printed) dbg_printf("'0x%0*I64x' is not a valid module address\n", ADDRWIDTH, base); + if (has_missing_filename) + dbg_printf("^ denotes modules for which image file couldn't be found\n"); } struct class_walker

1 0

Eric Pouech : dbghelp: Expose PE native vs builtin information to winedbg.
by Alexandre Julliard Nov. 16, 2023

Nov. 16, 2023

Module: wine Branch: master Commit: 01c98c5eafd68d77f388aaba2c0a6d44a01ab41c URL: https://gitlab.winehq.org/wine/wine/-/commit/01c98c5eafd68d77f388aaba2c0a6d… Author: Eric Pouech <epouech(a)codeweavers.com> Date: Thu Nov 16 18:38:02 2023 +0100 dbghelp: Expose PE native vs builtin information to winedbg. Signed-off-by: Eric Pouech <epouech(a)codeweavers.com> --- dlls/dbghelp/dbghelp_private.h | 7 ++++--- dlls/dbghelp/elf_module.c | 5 +++-- dlls/dbghelp/macho_module.c | 3 ++- dlls/dbghelp/minidump.c | 2 +- dlls/dbghelp/module.c | 23 +++++++++++++---------- dlls/dbghelp/pe_module.c | 32 +++++++++++++++++++++++--------- include/dbghelp.h | 2 ++ programs/winedbg/info.c | 27 ++++++++++++++++----------- 8 files changed, 64 insertions(+), 37 deletions(-)

1 0

Eric Pouech : dbghelp: Expose some internal information about modules to winedbg.
by Alexandre Julliard Nov. 16, 2023

Nov. 16, 2023

Module: wine Branch: master Commit: c50e02ec9ce78361afca88f740c7b71bf7b23986 URL: https://gitlab.winehq.org/wine/wine/-/commit/c50e02ec9ce78361afca88f740c7b7… Author: Eric Pouech <epouech(a)codeweavers.com> Date: Thu Nov 16 18:38:02 2023 +0100 dbghelp: Expose some internal information about modules to winedbg. Signed-off-by: Eric Pouech <epouech(a)codeweavers.com> --- dlls/dbghelp/dbghelp.c | 7 +++++ dlls/dbghelp/dbghelp.spec | 3 ++ dlls/dbghelp/dbghelp_private.h | 13 +++----- dlls/dbghelp/dwarf.c | 25 ++++++++------- dlls/dbghelp/image_private.h | 2 +- dlls/dbghelp/module.c | 37 ++++++++++++++++++++-- dlls/dbghelp/stabs.c | 2 +- include/dbghelp.h | 33 ++++++++++++++++++++ programs/winedbg/info.c | 69 +++++++++++++++++++++--------------------- programs/winedbg/winedbg.c | 1 + 10 files changed, 131 insertions(+), 61 deletions(-)

1 0

Alexandre Julliard : wow64: Remove DECLSPEC_HIDDEN usage.
by Alexandre Julliard Nov. 16, 2023

Nov. 16, 2023

Module: wine Branch: master Commit: a7f7832107b74681f545e9146cedefdbebb049e4 URL: https://gitlab.winehq.org/wine/wine/-/commit/a7f7832107b74681f545e9146cedef… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Nov 16 16:07:14 2023 +0100 wow64: Remove DECLSPEC_HIDDEN usage. --- dlls/wow64/syscall.c | 2 +- dlls/wow64/wow64_private.h | 22 +++++++++++----------- 2 files changed, 12 insertions(+), 12 deletions(-) diff --git a/dlls/wow64/syscall.c b/dlls/wow64/syscall.c index 3da989acca6..11d2bebfae2 100644 --- a/dlls/wow64/syscall.c +++ b/dlls/wow64/syscall.c @@ -334,7 +334,7 @@ static void call_raise_user_exception_dispatcher( ULONG code ) /* based on RtlRaiseException: call NtRaiseException with context setup to return to caller */ -void WINAPI raise_exception( EXCEPTION_RECORD *rec, CONTEXT *context, BOOL first_chance ) DECLSPEC_HIDDEN; +void WINAPI raise_exception( EXCEPTION_RECORD *rec, CONTEXT *context, BOOL first_chance ); #ifdef __x86_64__ __ASM_GLOBAL_FUNC( raise_exception, "sub $0x28,%rsp\n\t" diff --git a/dlls/wow64/wow64_private.h b/dlls/wow64/wow64_private.h index bd955e3fdac..62eccd3474e 100644 --- a/dlls/wow64/wow64_private.h +++ b/dlls/wow64/wow64_private.h @@ -28,16 +28,16 @@ ALL_SYSCALLS32 #undef SYSCALL_ENTRY -extern void init_image_mapping( HMODULE module ) DECLSPEC_HIDDEN; -extern void init_file_redirects(void) DECLSPEC_HIDDEN; -extern BOOL get_file_redirect( OBJECT_ATTRIBUTES *attr ) DECLSPEC_HIDDEN; +extern void init_image_mapping( HMODULE module ); +extern void init_file_redirects(void); +extern BOOL get_file_redirect( OBJECT_ATTRIBUTES *attr ); -extern USHORT native_machine DECLSPEC_HIDDEN; -extern USHORT current_machine DECLSPEC_HIDDEN; -extern ULONG_PTR args_alignment DECLSPEC_HIDDEN; -extern ULONG_PTR highest_user_address DECLSPEC_HIDDEN; -extern ULONG_PTR default_zero_bits DECLSPEC_HIDDEN; -extern SYSTEM_DLL_INIT_BLOCK *pLdrSystemDllInitBlock DECLSPEC_HIDDEN; +extern USHORT native_machine; +extern USHORT current_machine; +extern ULONG_PTR args_alignment; +extern ULONG_PTR highest_user_address; +extern ULONG_PTR default_zero_bits; +extern SYSTEM_DLL_INIT_BLOCK *pLdrSystemDllInitBlock; extern void (WINAPI *pBTCpuNotifyFlushInstructionCache2)( const void *, SIZE_T ); extern void (WINAPI *pBTCpuNotifyMapViewOfSection)( void * ); @@ -245,8 +245,8 @@ static inline void put_iosb( IO_STATUS_BLOCK32 *io32, const IO_STATUS_BLOCK *io } extern void put_section_image_info( SECTION_IMAGE_INFORMATION32 *info32, - const SECTION_IMAGE_INFORMATION *info ) DECLSPEC_HIDDEN; + const SECTION_IMAGE_INFORMATION *info ); extern void put_vm_counters( VM_COUNTERS_EX32 *info32, const VM_COUNTERS_EX *info, - ULONG size ) DECLSPEC_HIDDEN; + ULONG size ); #endif /* __WOW64_PRIVATE_H */

1 0

Alexandre Julliard : wow64win: Use the generated syscall list.
by Alexandre Julliard Nov. 16, 2023

Nov. 16, 2023

Module: wine Branch: master Commit: c2868b331e625529cfc3cd79721e9ddd639e3bb6 URL: https://gitlab.winehq.org/wine/wine/-/commit/c2868b331e625529cfc3cd79721e9d… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Nov 16 14:09:55 2023 +0100 wow64win: Use the generated syscall list. --- dlls/wow64win/syscall.c | 8 +- dlls/wow64win/syscall.h | 427 --------------------------------------- dlls/wow64win/wow64win_private.h | 8 +- 3 files changed, 8 insertions(+), 435 deletions(-)

1 0

Alexandre Julliard : win32u: Use the generated syscall list.
by Alexandre Julliard Nov. 16, 2023

Nov. 16, 2023

Module: wine Branch: master Commit: ade158ea7df3719ada29aefec06e8f615a30446e URL: https://gitlab.winehq.org/wine/wine/-/commit/ade158ea7df3719ada29aefec06e8f… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Nov 16 14:08:24 2023 +0100 win32u: Use the generated syscall list. --- dlls/win32u/syscall.c | 409 +------------------------------------------------- 1 file changed, 8 insertions(+), 401 deletions(-)

1 0

Alexandre Julliard : wow64: Use the generated syscall list.
by Alexandre Julliard Nov. 16, 2023

Nov. 16, 2023

Module: wine Branch: master Commit: 4adf3bc0bb31fed012af12ab54875162d4273ed4 URL: https://gitlab.winehq.org/wine/wine/-/commit/4adf3bc0bb31fed012af12ab548751… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Nov 16 14:09:16 2023 +0100 wow64: Use the generated syscall list. --- dlls/wow64/syscall.c | 8 +- dlls/wow64/syscall.h | 265 --------------------------------------------- dlls/wow64/wow64_private.h | 6 +- 3 files changed, 7 insertions(+), 272 deletions(-)

1 0

Zebediah Figura : wined3d: Provide a null counter BO for slots with no corresponding XFB buffer bound.
by Alexandre Julliard Nov. 16, 2023

Nov. 16, 2023

Module: wine Branch: master Commit: fbd53c3a3f65792608281be25ad5dcb315a6f2eb URL: https://gitlab.winehq.org/wine/wine/-/commit/fbd53c3a3f65792608281be25ad5dc… Author: Zebediah Figura <zfigura(a)codeweavers.com> Date: Wed Nov 8 18:10:31 2023 -0600 wined3d: Provide a null counter BO for slots with no corresponding XFB buffer bound. The Vulkan specification does not currently mandate this. However, an (unfortunately internal) Khronos discussion determined that this is an oversight, and that many drivers likely expect that no counter buffer will be bound if no XFB buffer is bound. Accordingly the specification will probably mandate this in the future. lavapipe is such a driver. This fixes a crash with llvmpipe in test_index_buffer_offset(). --- dlls/wined3d/context_vk.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/dlls/wined3d/context_vk.c b/dlls/wined3d/context_vk.c index fcdaaa17966..29a4a4998d4 100644 --- a/dlls/wined3d/context_vk.c +++ b/dlls/wined3d/context_vk.c @@ -2820,13 +2820,12 @@ static void wined3d_context_vk_bind_stream_output_buffers(struct wined3d_context if (!wined3d_context_vk_create_bo(context_vk, ARRAY_SIZE(context_vk->vk_so_counters) * sizeof(uint32_t) * 2, VK_BUFFER_USAGE_TRANSFORM_FEEDBACK_COUNTER_BUFFER_BIT_EXT, VK_MEMORY_PROPERTY_HOST_VISIBLE_BIT, bo)) ERR("Failed to create counter BO.\n"); + for (i = 0; i < ARRAY_SIZE(context_vk->vk_so_counters); ++i) - { - context_vk->vk_so_counters[i] = bo->vk_buffer; context_vk->vk_so_offsets[i] = bo->b.buffer_offset + i * sizeof(uint32_t) * 2; - } } + memset(context_vk->vk_so_counters, 0, sizeof(context_vk->vk_so_counters)); first = 0; count = 0; for (i = 0; i < ARRAY_SIZE(state->stream_output); ++i) @@ -2835,6 +2834,8 @@ static void wined3d_context_vk_bind_stream_output_buffers(struct wined3d_context if ((buffer = stream->buffer)) { + context_vk->vk_so_counters[i] = context_vk->vk_so_counter_bo.vk_buffer; + buffer_vk = wined3d_buffer_vk(buffer); buffer_info = wined3d_buffer_vk_get_buffer_info(buffer_vk); wined3d_context_vk_reference_bo(context_vk, wined3d_bo_vk(buffer->buffer_object));

1 0

Alex Henrie : sapi: Use CRT allocation functions.
by Alexandre Julliard Nov. 16, 2023

Nov. 16, 2023

Module: wine Branch: master Commit: 8f26aaf2ba7fc7889c3ec6c5841741a21bc12973 URL: https://gitlab.winehq.org/wine/wine/-/commit/8f26aaf2ba7fc7889c3ec6c5841741… Author: Alex Henrie <alexhenrie24(a)gmail.com> Date: Thu Nov 16 00:23:51 2023 -0700 sapi: Use CRT allocation functions. --- dlls/sapi/async.c | 5 ++--- dlls/sapi/automation.c | 4 ++-- dlls/sapi/mmaudio.c | 22 +++++++++++----------- dlls/sapi/resource.c | 4 ++-- dlls/sapi/sapi_private.h | 1 - dlls/sapi/stream.c | 4 ++-- dlls/sapi/tts.c | 24 ++++++++++++------------ 7 files changed, 31 insertions(+), 33 deletions(-) diff --git a/dlls/sapi/async.c b/dlls/sapi/async.c index 61b99019abe..69f962d5502 100644 --- a/dlls/sapi/async.c +++ b/dlls/sapi/async.c @@ -24,7 +24,6 @@ #include "winbase.h" #include "objbase.h" -#include "wine/heap.h" #include "wine/list.h" #include "wine/debug.h" @@ -58,7 +57,7 @@ void async_empty_queue(struct async_queue *queue) LIST_FOR_EACH_ENTRY_SAFE(task, next, &queue->tasks, struct async_task, entry) { list_remove(&task->entry); - heap_free(task); + free(task); } LeaveCriticalSection(&queue->cs); @@ -87,7 +86,7 @@ static void CALLBACK async_worker(TP_CALLBACK_INSTANCE *instance, void *ctx) { ResetEvent(queue->empty); task->proc(task); - heap_free(task); + free(task); if (WaitForSingleObject(queue->cancel, 0) == WAIT_OBJECT_0) goto cancel; } diff --git a/dlls/sapi/automation.c b/dlls/sapi/automation.c index 8a0e0203198..6292a5119fa 100644 --- a/dlls/sapi/automation.c +++ b/dlls/sapi/automation.c @@ -95,7 +95,7 @@ static ULONG WINAPI file_stream_Release(ISpeechFileStream *iface) if (!ref) { - heap_free(This); + free(This); } return ref; @@ -375,7 +375,7 @@ const static ISpStreamVtbl spstream_vtbl = HRESULT file_stream_create(IUnknown *outer, REFIID iid, void **obj) { - struct file_stream *This = heap_alloc(sizeof(*This)); + struct file_stream *This = malloc(sizeof(*This)); HRESULT hr; if (!This) return E_OUTOFMEMORY; diff --git a/dlls/sapi/mmaudio.c b/dlls/sapi/mmaudio.c index 7452d9a7257..0369157acbc 100644 --- a/dlls/sapi/mmaudio.c +++ b/dlls/sapi/mmaudio.c @@ -381,12 +381,12 @@ static ULONG WINAPI mmsysaudio_Release(ISpMMSysAudio *iface) async_cancel_queue(&This->queue); if (This->token) ISpObjectToken_Release(This->token); - heap_free(This->wfx); + free(This->wfx); CloseHandle(This->event); DeleteCriticalSection(&This->pending_cs); DeleteCriticalSection(&This->cs); - heap_free(This); + free(This); } return ref; @@ -421,7 +421,7 @@ static HRESULT WINAPI mmsysaudio_Write(ISpMMSysAudio *iface, const void *pv, ULO return SP_AUDIO_STOPPED; } - if (!(buf = heap_alloc(sizeof(WAVEHDR) + cb))) + if (!(buf = malloc(sizeof(WAVEHDR) + cb))) { LeaveCriticalSection(&This->cs); return E_OUTOFMEMORY; @@ -434,7 +434,7 @@ static HRESULT WINAPI mmsysaudio_Write(ISpMMSysAudio *iface, const void *pv, ULO if (waveOutPrepareHeader(This->hwave.out, buf, sizeof(WAVEHDR)) != MMSYSERR_NOERROR) { LeaveCriticalSection(&This->cs); - heap_free(buf); + free(buf); return E_FAIL; } @@ -562,7 +562,7 @@ static void free_out_buf_proc(struct async_task *task) TRACE("(%p).\n", task); waveOutUnprepareHeader(fbt->audio->hwave.out, fbt->buf, sizeof(WAVEHDR)); - heap_free(fbt->buf); + free(fbt->buf); EnterCriticalSection(&fbt->audio->pending_cs); buf_count = --fbt->audio->pending_buf_count; @@ -582,7 +582,7 @@ static void CALLBACK wave_out_proc(HWAVEOUT hwo, UINT msg, DWORD_PTR instance, D switch (msg) { case WOM_DONE: - if (!(task = heap_alloc(sizeof(*task)))) + if (!(task = malloc(sizeof(*task)))) { ERR("failed to allocate free_buf_task.\n"); break; @@ -685,13 +685,13 @@ static HRESULT WINAPI mmsysaudio_SetFormat(ISpMMSysAudio *iface, const GUID *gui return res == WAVERR_BADFORMAT ? SPERR_UNSUPPORTED_FORMAT : SPERR_GENERIC_MMSYS_ERROR; } - if (!(new_wfx = heap_alloc(sizeof(*wfx) + wfx->cbSize))) + if (!(new_wfx = malloc(sizeof(*wfx) + wfx->cbSize))) { LeaveCriticalSection(&This->cs); return E_OUTOFMEMORY; } memcpy(new_wfx, wfx, sizeof(*wfx) + wfx->cbSize); - heap_free(This->wfx); + free(This->wfx); This->wfx = new_wfx; LeaveCriticalSection(&This->cs); @@ -874,7 +874,7 @@ static HRESULT mmaudio_create(IUnknown *outer, REFIID iid, void **obj, enum flow return E_NOTIMPL; } - if (!(This = heap_alloc_zero(sizeof(*This)))) + if (!(This = calloc(1, sizeof(*This)))) return E_OUTOFMEMORY; This->ISpEventSource_iface.lpVtbl = &event_source_vtbl; This->ISpEventSink_iface.lpVtbl = &event_sink_vtbl; @@ -887,9 +887,9 @@ static HRESULT mmaudio_create(IUnknown *outer, REFIID iid, void **obj, enum flow This->device_id = WAVE_MAPPER; This->state = SPAS_CLOSED; - if (!(This->wfx = heap_alloc(sizeof(*This->wfx)))) + if (!(This->wfx = malloc(sizeof(*This->wfx)))) { - heap_free(This); + free(This); return E_OUTOFMEMORY; } This->wfx->wFormatTag = WAVE_FORMAT_PCM; diff --git a/dlls/sapi/resource.c b/dlls/sapi/resource.c index 8db833b3b39..5fea496979f 100644 --- a/dlls/sapi/resource.c +++ b/dlls/sapi/resource.c @@ -84,7 +84,7 @@ static ULONG WINAPI resource_manager_Release(ISpResourceManager *iface) if (!ref) { - heap_free(This); + free(This); } return ref; @@ -126,7 +126,7 @@ const static ISpResourceManagerVtbl resource_manager_vtbl = HRESULT resource_manager_create(IUnknown *outer, REFIID iid, void **obj) { - struct resource_manager *This = heap_alloc(sizeof(*This)); + struct resource_manager *This = malloc(sizeof(*This)); HRESULT hr; if (!This) return E_OUTOFMEMORY; diff --git a/dlls/sapi/sapi_private.h b/dlls/sapi/sapi_private.h index 3cdb59b36a8..f6a5e966461 100644 --- a/dlls/sapi/sapi_private.h +++ b/dlls/sapi/sapi_private.h @@ -18,7 +18,6 @@ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA */ -#include "wine/heap.h" #include "wine/list.h" struct async_task diff --git a/dlls/sapi/stream.c b/dlls/sapi/stream.c index 6df2768b5cc..5cd64125ade 100644 --- a/dlls/sapi/stream.c +++ b/dlls/sapi/stream.c @@ -84,7 +84,7 @@ static ULONG WINAPI spstream_Release(ISpStream *iface) if (!ref) { - heap_free(This); + free(This); } return ref; @@ -235,7 +235,7 @@ const static ISpStreamVtbl spstream_vtbl = HRESULT speech_stream_create(IUnknown *outer, REFIID iid, void **obj) { - struct spstream *This = heap_alloc(sizeof(*This)); + struct spstream *This = malloc(sizeof(*This)); HRESULT hr; if (!This) return E_OUTOFMEMORY; diff --git a/dlls/sapi/tts.c b/dlls/sapi/tts.c index b1c90627d5e..410a57b4bfc 100644 --- a/dlls/sapi/tts.c +++ b/dlls/sapi/tts.c @@ -166,7 +166,7 @@ static ULONG WINAPI speech_voice_Release(ISpeechVoice *iface) if (This->engine) ISpTTSEngine_Release(This->engine); DeleteCriticalSection(&This->cs); - heap_free(This); + free(This); } return ref; @@ -822,7 +822,7 @@ done: } CoTaskMemFree(wfx); if (engine) ISpTTSEngine_Release(engine); - heap_free(speak_task->frag_list); + free(speak_task->frag_list); ISpTTSEngineSite_Release(speak_task->site); if (speak_task->result) @@ -898,7 +898,7 @@ static HRESULT WINAPI spvoice_Speak(ISpVoice *iface, const WCHAR *contents, DWOR return hr; } - if (!(frag = heap_alloc(sizeof(*frag) + contents_size))) + if (!(frag = malloc(sizeof(*frag) + contents_size))) return E_OUTOFMEMORY; memset(frag, 0, sizeof(*frag)); memcpy(frag + 1, contents, contents_size); @@ -915,7 +915,7 @@ static HRESULT WINAPI spvoice_Speak(ISpVoice *iface, const WCHAR *contents, DWOR goto fail; } - speak_task = heap_alloc(sizeof(*speak_task)); + speak_task = malloc(sizeof(*speak_task)); speak_task->task.proc = speak_proc; speak_task->result = NULL; @@ -926,7 +926,7 @@ static HRESULT WINAPI spvoice_Speak(ISpVoice *iface, const WCHAR *contents, DWOR if (!(flags & SPF_ASYNC)) { - if (!(result = heap_alloc(sizeof(*result)))) + if (!(result = malloc(sizeof(*result)))) { hr = E_OUTOFMEMORY; goto fail; @@ -952,18 +952,18 @@ static HRESULT WINAPI spvoice_Speak(ISpVoice *iface, const WCHAR *contents, DWOR WaitForSingleObject(result->done, INFINITE); hr = result->hr; CloseHandle(result->done); - heap_free(result); + free(result); return hr; } fail: if (site) ISpTTSEngineSite_Release(site); - heap_free(frag); - heap_free(speak_task); + free(frag); + free(speak_task); if (result) { CloseHandle(result->done); - heap_free(result); + free(result); } return hr; } @@ -1210,7 +1210,7 @@ static ULONG WINAPI ttsenginesite_Release(ISpTTSEngineSite *iface) { if (This->voice) ISpeechVoice_Release(&This->voice->ISpeechVoice_iface); - heap_free(This); + free(This); } return ref; @@ -1315,7 +1315,7 @@ const static ISpTTSEngineSiteVtbl ttsenginesite_vtbl = static HRESULT ttsenginesite_create(struct speech_voice *voice, ULONG stream_num, ISpTTSEngineSite **site) { - struct tts_engine_site *This = heap_alloc(sizeof(*This)); + struct tts_engine_site *This = malloc(sizeof(*This)); if (!This) return E_OUTOFMEMORY; @@ -1387,7 +1387,7 @@ const static IConnectionPointContainerVtbl container_vtbl = HRESULT speech_voice_create(IUnknown *outer, REFIID iid, void **obj) { - struct speech_voice *This = heap_alloc(sizeof(*This)); + struct speech_voice *This = malloc(sizeof(*This)); HRESULT hr; if (!This) return E_OUTOFMEMORY;

1 0