wine-commits November 2023

wine-commits@winehq.org

1 participants
989 discussions

Alistair Leslie-Hughes : msado15: Updating recordset EditMode when field value changes.
by Alexandre Julliard 17 Nov '23

17 Nov '23

Module: wine Branch: master Commit: a85055ee6c5d59ed4258a5501fa072ecae8816e7 URL: https://gitlab.winehq.org/wine/wine/-/commit/a85055ee6c5d59ed4258a5501fa072… Author: Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> Date: Sun Nov 12 12:23:21 2023 +1100 msado15: Updating recordset EditMode when field value changes. We need to know what type of Edit we are Performing, eg AddNew or Updated existing. --- dlls/msado15/recordset.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/dlls/msado15/recordset.c b/dlls/msado15/recordset.c index 2b936f5c096..d64ccdeee3d 100644 --- a/dlls/msado15/recordset.c +++ b/dlls/msado15/recordset.c @@ -298,6 +298,10 @@ static HRESULT WINAPI field_put_Value( Field *iface, VARIANT val ) if ((hr = VariantCopy( &copy, &val )) != S_OK) return hr; field->recordset->data[row * col_count + col] = copy; + + if (field->recordset->editmode != adEditNone) + field->recordset->editmode = adEditInProgress; + return S_OK; }

1 0

Alexandre Julliard : win32u: Remove DECLSPEC_HIDDEN usage.
by Alexandre Julliard 17 Nov '23

17 Nov '23

Module: wine Branch: master Commit: 909e2b952a82b6d0248bf5df1dd4b9dbd7e4130d URL: https://gitlab.winehq.org/wine/wine/-/commit/909e2b952a82b6d0248bf5df1dd4b9… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Nov 17 10:20:54 2023 +0100 win32u: Remove DECLSPEC_HIDDEN usage. --- dlls/win32u/dibdrv/dibdrv.h | 114 ++++++------- dlls/win32u/font.c | 2 +- dlls/win32u/imm.c | 4 +- dlls/win32u/ntgdi_private.h | 230 +++++++++++++------------- dlls/win32u/ntuser_private.h | 66 ++++---- dlls/win32u/win32u_private.h | 376 +++++++++++++++++++++---------------------- 6 files changed, 396 insertions(+), 396 deletions(-)

1 0

Alexandre Julliard : winebuild: Stop generating the syscall argument array and related data.
by Alexandre Julliard 17 Nov '23

17 Nov '23

Module: wine Branch: master Commit: e9afc9bf8f9643f27b81345123da592f352e1318 URL: https://gitlab.winehq.org/wine/wine/-/commit/e9afc9bf8f9643f27b81345123da59… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Nov 17 10:16:21 2023 +0100 winebuild: Stop generating the syscall argument array and related data. --- dlls/ntdll/loader.c | 1 + dlls/win32u/main.c | 2 +- tools/winebuild/import.c | 9 --------- 3 files changed, 2 insertions(+), 10 deletions(-) diff --git a/dlls/ntdll/loader.c b/dlls/ntdll/loader.c index 89756c57433..4bc7e66d76b 100644 --- a/dlls/ntdll/loader.c +++ b/dlls/ntdll/loader.c @@ -80,6 +80,7 @@ static DWORD (WINAPI *pCtrlRoutine)(void *); SYSTEM_DLL_INIT_BLOCK LdrSystemDllInitBlock = { 0xf0 }; +void *__wine_syscall_dispatcher = NULL; unixlib_handle_t __wine_unixlib_handle = 0; /* windows directory */ diff --git a/dlls/win32u/main.c b/dlls/win32u/main.c index e074186cd90..d650483fb99 100644 --- a/dlls/win32u/main.c +++ b/dlls/win32u/main.c @@ -27,7 +27,7 @@ #include "ntgdi.h" #include "wine/unixlib.h" -extern void *__wine_syscall_dispatcher DECLSPEC_HIDDEN; +void *__wine_syscall_dispatcher = NULL; static unixlib_handle_t win32u_handle; diff --git a/tools/winebuild/import.c b/tools/winebuild/import.c index c3a1dad4452..e3bb8b7f3db 100644 --- a/tools/winebuild/import.c +++ b/tools/winebuild/import.c @@ -1519,15 +1519,6 @@ void output_syscalls( DLLSPEC *spec ) default: break; } - output( "\t.data\n" ); - output( "\t.balign %u\n", get_ptr_size() ); - output( "%s\n", asm_globl("__wine_syscall_dispatcher") ); - output( "\t%s 0\n", get_asm_ptr_keyword() ); /* dispatcher */ - output( "\t.long 0xca110001\n" ); /* version */ - output( "\t.short %u\n", spec->syscall_table ); /* id */ - output( "\t.short %u\n", count ); /* limit */ - for (i = 0; i < count; i++) output( "\t.short %u\n", syscalls[i]->hint ); - for (i = 0; i < count; i++) output( "\t.byte %u\n", get_args_size( syscalls[i] )); }

1 0

Alexandre Julliard : wow64: Build the syscall argument array from the generated list.
by Alexandre Julliard 17 Nov '23

17 Nov '23

Module: wine Branch: master Commit: 85dbc0466f2c40f25e6734083cbca7511200bf38 URL: https://gitlab.winehq.org/wine/wine/-/commit/85dbc0466f2c40f25e6734083cbca7… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Nov 17 10:00:48 2023 +0100 wow64: Build the syscall argument array from the generated list. --- dlls/wow64/syscall.c | 102 ++++-------------------------------------------- dlls/wow64win/syscall.c | 10 ++--- 2 files changed, 12 insertions(+), 100 deletions(-) diff --git a/dlls/wow64/syscall.c b/dlls/wow64/syscall.c index 11d2bebfae2..764f5fb2c05 100644 --- a/dlls/wow64/syscall.c +++ b/dlls/wow64/syscall.c @@ -49,22 +49,18 @@ static const syscall_thunk syscall_thunks[] = #undef SYSCALL_ENTRY }; -static const char *syscall_names[] = +static BYTE syscall_args[ARRAY_SIZE(syscall_thunks)] = { -#define SYSCALL_ENTRY(id,name,args) #name, +#define SYSCALL_ENTRY(id,name,args) args, ALL_SYSCALLS32 #undef SYSCALL_ENTRY }; -static const SYSTEM_SERVICE_TABLE ntdll_syscall_table = +static SYSTEM_SERVICE_TABLE syscall_tables[4] = { - (ULONG_PTR *)syscall_thunks, - (ULONG_PTR *)syscall_names, - ARRAY_SIZE(syscall_thunks) + { (ULONG_PTR *)syscall_thunks, NULL, ARRAY_SIZE(syscall_thunks), syscall_args } }; -static SYSTEM_SERVICE_TABLE syscall_tables[4]; - /* header for Wow64AllocTemp blocks; probably not the right layout */ struct mem_header { @@ -94,9 +90,6 @@ struct user_apc_frame SYSTEM_DLL_INIT_BLOCK *pLdrSystemDllInitBlock = NULL; -/* wow64win syscall table */ -static const SYSTEM_SERVICE_TABLE *psdwhwin32; -static HMODULE win32u_module; static WOW64INFO *wow64info; /* cpu backend dll functions */ @@ -643,81 +636,6 @@ NTSTATUS WINAPI wow64_NtWow64IsProcessorFeaturePresent( UINT *args ) } -/********************************************************************** - * get_rva - */ -static void *get_rva( HMODULE module, DWORD rva ) -{ - return (void *)((char *)module + rva); -} - - -/********************************************************************** - * init_syscall_table - */ -static void init_syscall_table( HMODULE module, void *dispatcher, const SYSTEM_SERVICE_TABLE *orig_table ) -{ - static syscall_thunk thunks[2048]; - static ULONG start_pos; - - struct syscall_info32 - { - UINT dispatcher; - UINT version; - USHORT id; - USHORT limit; - /* USHORT names[limit]; */ - /* BYTE args[limit]; */ - } *info = dispatcher; - - const USHORT *name_ptrs = (const USHORT *)(info + 1); - const IMAGE_EXPORT_DIRECTORY *exports; - const ULONG *names; - ULONG id, exp_size, wrap_pos; - const char **syscall_names = (const char **)orig_table->CounterTable; - - if (info->version != 0xca110001) - { - WARN( "invalid syscall table version %x\n", info->version ); - return; - } - if (syscall_tables[info->id].ServiceTable) return; /* already initialized */ - - exports = RtlImageDirectoryEntryToData( module, TRUE, IMAGE_DIRECTORY_ENTRY_EXPORT, &exp_size ); - names = get_rva( module, exports->AddressOfNames ); - - for (id = wrap_pos = 0; id < info->limit; id++) - { - char *name = get_rva( module, names[name_ptrs[id]] ); - int res = -1; - - if (wrap_pos < orig_table->ServiceLimit) res = strcmp( name, syscall_names[wrap_pos] ); - - if (!res) /* got a match */ - { - if (start_pos + id < ARRAY_SIZE(thunks)) - thunks[start_pos + id] = (syscall_thunk)orig_table->ServiceTable[wrap_pos++]; - else - ERR( "invalid syscall id %04lx for %s\n", id, name ); - } - else if (res > 0) - { - FIXME( "no export for syscall %s\n", syscall_names[wrap_pos] ); - wrap_pos++; - id--; /* try again */ - } - else FIXME( "missing wrapper for syscall %04lx %s\n", id, name ); - } - - for ( ; wrap_pos < orig_table->ServiceLimit; wrap_pos++) - FIXME( "no export for syscall %s\n", syscall_names[wrap_pos] ); - - syscall_tables[info->id].ServiceTable = (ULONG_PTR *)(thunks + start_pos); - syscall_tables[info->id].ServiceLimit = info->limit; - start_pos += info->limit; -} - - /********************************************************************** * init_image_mapping */ @@ -725,13 +643,7 @@ void init_image_mapping( HMODULE module ) { ULONG *ptr = RtlFindExportedRoutineByName( module, "Wow64Transition" ); - if (!ptr) return; - *ptr = PtrToUlong( pBTCpuGetBopCode() ); - if (!win32u_module && RtlFindExportedRoutineByName( module, "NtUserInitializeClientPfnArrays" )) - { - win32u_module = module; - init_syscall_table( win32u_module, ptr, psdwhwin32 ); - } + if (ptr) *ptr = PtrToUlong( pBTCpuGetBopCode() ); } @@ -842,6 +754,7 @@ static DWORD WINAPI process_init( RTL_RUN_ONCE *once, void *param, void **contex UNICODE_STRING str = RTL_CONSTANT_STRING( L"ntdll.dll" ); SYSTEM_BASIC_INFORMATION info; ULONG *p__wine_syscall_dispatcher, *p__wine_unix_call_dispatcher; + const SYSTEM_SERVICE_TABLE *psdwhwin32; RtlWow64GetProcessMachines( GetCurrentProcess(), &current_machine, &native_machine ); if (!current_machine) current_machine = native_machine; @@ -883,6 +796,7 @@ static DWORD WINAPI process_init( RTL_RUN_ONCE *once, void *param, void **contex module = load_64bit_module( L"wow64win.dll" ); GET_PTR( sdwhwin32 ); + syscall_tables[1] = *psdwhwin32; pBTCpuProcessInit(); @@ -895,8 +809,6 @@ static DWORD WINAPI process_init( RTL_RUN_ONCE *once, void *param, void **contex *p__wine_syscall_dispatcher = PtrToUlong( pBTCpuGetBopCode() ); *p__wine_unix_call_dispatcher = PtrToUlong( p__wine_get_unix_opcode() ); - init_syscall_table( module, p__wine_syscall_dispatcher, &ntdll_syscall_table ); - if (wow64info->CpuFlags & WOW64_CPUFLAGS_SOFTWARE) create_cross_process_work_list( wow64info ); init_file_redirects(); diff --git a/dlls/wow64win/syscall.c b/dlls/wow64win/syscall.c index 5d08e1fd905..55b9dec9722 100644 --- a/dlls/wow64win/syscall.c +++ b/dlls/wow64win/syscall.c @@ -35,19 +35,19 @@ static void * const win32_syscalls[] = #undef SYSCALL_ENTRY }; -static const char * const win32_syscall_names[] = +static BYTE arguments[ARRAY_SIZE(win32_syscalls)] = { -#define SYSCALL_ENTRY(id,name,args) #name, +#define SYSCALL_ENTRY(id,name,args) args, ALL_SYSCALLS32 #undef SYSCALL_ENTRY }; -SYSTEM_SERVICE_TABLE sdwhwin32 = +const SYSTEM_SERVICE_TABLE sdwhwin32 = { (ULONG_PTR *)win32_syscalls, - (ULONG_PTR *)win32_syscall_names, + NULL, ARRAY_SIZE(win32_syscalls), - NULL + arguments };

1 0

Alexandre Julliard : win32u: Store the service table entry directly into the ntdll variable.
by Alexandre Julliard 17 Nov '23

17 Nov '23

Module: wine Branch: master Commit: 92b6ab2d6e2af1f70966fa4631cae2789d052730 URL: https://gitlab.winehq.org/wine/wine/-/commit/92b6ab2d6e2af1f70966fa4631cae2… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Nov 17 09:57:56 2023 +0100 win32u: Store the service table entry directly into the ntdll variable. --- dlls/ntdll/unix/loader.c | 31 ------------------------------- dlls/win32u/main.c | 2 +- dlls/win32u/syscall.c | 7 ++++--- include/wine/unixlib.h | 2 +- 4 files changed, 6 insertions(+), 36 deletions(-) diff --git a/dlls/ntdll/unix/loader.c b/dlls/ntdll/unix/loader.c index fa38cea0615..aa120ae38ec 100644 --- a/dlls/ntdll/unix/loader.c +++ b/dlls/ntdll/unix/loader.c @@ -919,37 +919,6 @@ static NTSTATUS dlopen_dll( const char *so_name, UNICODE_STRING *nt_name, void * } -/*********************************************************************** - * ntdll_init_syscalls - */ -NTSTATUS ntdll_init_syscalls( SYSTEM_SERVICE_TABLE *table, void **dispatcher ) -{ - struct syscall_info - { - void *dispatcher; - UINT version; - USHORT id; - USHORT limit; - /* USHORT names[limit]; */ - /* BYTE args[limit]; */ - } *info = (struct syscall_info *)dispatcher; - - if (info->version != 0xca110001) - { - ERR( "invalid syscall table version %x\n", info->version ); - NtTerminateProcess( GetCurrentProcess(), STATUS_INVALID_PARAMETER ); - } - if (info->limit != table->ServiceLimit) - { - ERR( "syscall count mismatch %u / %lu\n", info->limit, table->ServiceLimit ); - NtTerminateProcess( GetCurrentProcess(), STATUS_INVALID_PARAMETER ); - } - info->dispatcher = __wine_syscall_dispatcher; - KeServiceDescriptorTable[info->id] = *table; - return STATUS_SUCCESS; -} - - /*********************************************************************** * load_so_dll */ diff --git a/dlls/win32u/main.c b/dlls/win32u/main.c index fa75ad4f5c5..e074186cd90 100644 --- a/dlls/win32u/main.c +++ b/dlls/win32u/main.c @@ -61,7 +61,7 @@ BOOL WINAPI DllMain( HINSTANCE inst, DWORD reason, void *reserved ) __wine_syscall_dispatcher = *dispatcher_ptr; if (!NtQueryVirtualMemory( GetCurrentProcess(), inst, MemoryWineUnixFuncs, &win32u_handle, sizeof(win32u_handle), NULL )) - __wine_unix_call( win32u_handle, 0, &__wine_syscall_dispatcher ); + __wine_unix_call( win32u_handle, 0, NULL ); break; } return TRUE; diff --git a/dlls/win32u/syscall.c b/dlls/win32u/syscall.c index b741df91c66..2320ac18744 100644 --- a/dlls/win32u/syscall.c +++ b/dlls/win32u/syscall.c @@ -58,7 +58,7 @@ static BYTE arguments[ARRAY_SIZE(syscalls)] = #undef SYSCALL_ENTRY }; -static SYSTEM_SERVICE_TABLE syscall_table = +static const SYSTEM_SERVICE_TABLE syscall_table = { (ULONG_PTR *)syscalls, 0, @@ -66,7 +66,7 @@ static SYSTEM_SERVICE_TABLE syscall_table = arguments }; -static NTSTATUS init( void *dispatcher ) +static NTSTATUS init( void *args ) { #ifdef _WIN64 if (NtCurrentTeb()->WowTebOffset) @@ -78,7 +78,8 @@ static NTSTATUS init( void *dispatcher ) } #endif - return ntdll_init_syscalls( &syscall_table, dispatcher ); + KeServiceDescriptorTable[1] = syscall_table; + return STATUS_SUCCESS; } const unixlib_entry_t __wine_unix_call_funcs[] = diff --git a/include/wine/unixlib.h b/include/wine/unixlib.h index 81c59e732ba..2a615179e64 100644 --- a/include/wine/unixlib.h +++ b/include/wine/unixlib.h @@ -31,6 +31,7 @@ typedef NTSTATUS (*unixlib_entry_t)( void *args ); extern DECLSPEC_EXPORT const unixlib_entry_t __wine_unix_call_funcs[]; extern DECLSPEC_EXPORT const unixlib_entry_t __wine_unix_call_wow64_funcs[]; +extern DECLSPEC_EXPORT SYSTEM_SERVICE_TABLE KeServiceDescriptorTable[4]; /* some useful helpers from ntdll */ NTSYSAPI const char *ntdll_get_build_dir(void); @@ -39,7 +40,6 @@ NTSYSAPI DWORD ntdll_umbstowcs( const char *src, DWORD srclen, WCHAR *dst, DWORD NTSYSAPI int ntdll_wcstoumbs( const WCHAR *src, DWORD srclen, char *dst, DWORD dstlen, BOOL strict ); NTSYSAPI int ntdll_wcsicmp( const WCHAR *str1, const WCHAR *str2 ); NTSYSAPI int ntdll_wcsnicmp( const WCHAR *str1, const WCHAR *str2, int n ); -NTSYSAPI NTSTATUS ntdll_init_syscalls( SYSTEM_SERVICE_TABLE *table, void **dispatcher ); /* exception handling */

1 0

Alexandre Julliard : win32u: Build the syscall argument array from the generated syscall list.
by Alexandre Julliard 17 Nov '23

17 Nov '23

Module: wine Branch: master Commit: c8ecc35966f7a640050d12ccf7676a40cd1ae9d0 URL: https://gitlab.winehq.org/wine/wine/-/commit/c8ecc35966f7a640050d12ccf7676a… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Nov 17 09:57:14 2023 +0100 win32u: Build the syscall argument array from the generated syscall list. --- dlls/ntdll/unix/loader.c | 1 - dlls/win32u/syscall.c | 11 ++++++++++- 2 files changed, 10 insertions(+), 2 deletions(-) diff --git a/dlls/ntdll/unix/loader.c b/dlls/ntdll/unix/loader.c index c5ffbf1eb0a..fa38cea0615 100644 --- a/dlls/ntdll/unix/loader.c +++ b/dlls/ntdll/unix/loader.c @@ -945,7 +945,6 @@ NTSTATUS ntdll_init_syscalls( SYSTEM_SERVICE_TABLE *table, void **dispatcher ) NtTerminateProcess( GetCurrentProcess(), STATUS_INVALID_PARAMETER ); } info->dispatcher = __wine_syscall_dispatcher; - memcpy( table->ArgumentTable, (USHORT *)(info + 1) + info->limit, table->ServiceLimit ); KeServiceDescriptorTable[info->id] = *table; return STATUS_SUCCESS; } diff --git a/dlls/win32u/syscall.c b/dlls/win32u/syscall.c index 64bba9fbf99..b741df91c66 100644 --- a/dlls/win32u/syscall.c +++ b/dlls/win32u/syscall.c @@ -47,7 +47,16 @@ static void * const syscalls[] = #undef SYSCALL_ENTRY }; -static BYTE arguments[ARRAY_SIZE(syscalls)]; +static BYTE arguments[ARRAY_SIZE(syscalls)] = +{ +#define SYSCALL_ENTRY(id,name,args) args, +#ifdef _WIN64 + ALL_SYSCALLS64 +#else + ALL_SYSCALLS32 +#endif +#undef SYSCALL_ENTRY +}; static SYSTEM_SERVICE_TABLE syscall_table = {

1 0

Alexandre Julliard : win32u: Set the syscall dispatcher on the PE side.
by Alexandre Julliard 17 Nov '23

17 Nov '23

Module: wine Branch: master Commit: f799f6f47172e5feed9e1541979e6515a0e5d0b9 URL: https://gitlab.winehq.org/wine/wine/-/commit/f799f6f47172e5feed9e1541979e65… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Nov 17 09:56:37 2023 +0100 win32u: Set the syscall dispatcher on the PE side. --- dlls/win32u/main.c | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/dlls/win32u/main.c b/dlls/win32u/main.c index d2cd039b701..fa75ad4f5c5 100644 --- a/dlls/win32u/main.c +++ b/dlls/win32u/main.c @@ -47,11 +47,18 @@ void __cdecl __wine_spec_unimplemented_stub( const char *module, const char *fun BOOL WINAPI DllMain( HINSTANCE inst, DWORD reason, void *reserved ) { + HMODULE ntdll; + void **dispatcher_ptr; + const UNICODE_STRING ntdll_name = RTL_CONSTANT_STRING( L"ntdll.dll" ); + switch (reason) { case DLL_PROCESS_ATTACH: LdrDisableThreadCalloutsForDll( inst ); if (__wine_syscall_dispatcher) break; /* already set through Wow64Transition */ + LdrGetDllHandle( NULL, 0, &ntdll_name, &ntdll ); + dispatcher_ptr = RtlFindExportedRoutineByName( ntdll, "__wine_syscall_dispatcher" ); + __wine_syscall_dispatcher = *dispatcher_ptr; if (!NtQueryVirtualMemory( GetCurrentProcess(), inst, MemoryWineUnixFuncs, &win32u_handle, sizeof(win32u_handle), NULL )) __wine_unix_call( win32u_handle, 0, &__wine_syscall_dispatcher );

1 0

Alexandre Julliard : ntdll: Build the syscall argument array from the generated syscall list.
by Alexandre Julliard 17 Nov '23

17 Nov '23

Module: wine Branch: master Commit: 1693784169d43ee2978066cf276168869a55866d URL: https://gitlab.winehq.org/wine/wine/-/commit/1693784169d43ee2978066cf276168… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Nov 17 09:55:27 2023 +0100 ntdll: Build the syscall argument array from the generated syscall list. --- dlls/ntdll/unix/loader.c | 22 ++++++++++++++++------ 1 file changed, 16 insertions(+), 6 deletions(-) diff --git a/dlls/ntdll/unix/loader.c b/dlls/ntdll/unix/loader.c index 42b5aa9d84e..c5ffbf1eb0a 100644 --- a/dlls/ntdll/unix/loader.c +++ b/dlls/ntdll/unix/loader.c @@ -115,8 +115,6 @@ void *pRtlUserThreadStart = NULL; void *p__wine_ctrl_routine = NULL; SYSTEM_DLL_INIT_BLOCK *pLdrSystemDllInitBlock = NULL; -static void *p__wine_syscall_dispatcher; - static void * const syscalls[] = { #define SYSCALL_ENTRY(id,name,args) name, @@ -128,9 +126,21 @@ static void * const syscalls[] = #undef SYSCALL_ENTRY }; -static BYTE syscall_args[ARRAY_SIZE(syscalls)]; +static BYTE syscall_args[ARRAY_SIZE(syscalls)] = +{ +#define SYSCALL_ENTRY(id,name,args) args, +#ifdef _WIN64 + ALL_SYSCALLS64 +#else + ALL_SYSCALLS32 +#endif +#undef SYSCALL_ENTRY +}; -SYSTEM_SERVICE_TABLE KeServiceDescriptorTable[4]; +SYSTEM_SERVICE_TABLE KeServiceDescriptorTable[4] = +{ + { (ULONG_PTR *)syscalls, NULL, ARRAY_SIZE(syscalls), syscall_args } +}; #ifdef __GNUC__ static void fatal_error( const char *err, ... ) __attribute__((noreturn, format(printf,1,2))); @@ -1518,6 +1528,7 @@ NTSTATUS load_start_exe( WCHAR **image, void **module ) */ static void load_ntdll_functions( HMODULE module ) { + void **p__wine_syscall_dispatcher; void **p__wine_unix_call_dispatcher; unixlib_handle_t *p__wine_unixlib_handle; const IMAGE_EXPORT_DIRECTORY *exports; @@ -1541,6 +1552,7 @@ static void load_ntdll_functions( HMODULE module ) GET_FUNC( __wine_syscall_dispatcher ); GET_FUNC( __wine_unix_call_dispatcher ); GET_FUNC( __wine_unixlib_handle ); + *p__wine_syscall_dispatcher = __wine_syscall_dispatcher; *p__wine_unix_call_dispatcher = __wine_unix_call_dispatcher; *p__wine_unixlib_handle = (UINT_PTR)unix_call_funcs; #undef GET_FUNC @@ -1804,7 +1816,6 @@ static ULONG_PTR get_image_address(void) */ static void start_main_thread(void) { - SYSTEM_SERVICE_TABLE syscall_table = { (ULONG_PTR *)syscalls, NULL, ARRAY_SIZE(syscalls), syscall_args }; TEB *teb = virtual_alloc_first_teb(); signal_init_threading(); @@ -1822,7 +1833,6 @@ static void start_main_thread(void) load_ntdll(); load_wow64_ntdll( main_image_info.Machine ); load_apiset_dll(); - ntdll_init_syscalls( &syscall_table, p__wine_syscall_dispatcher ); server_init_process_done(); }

1 0

Alex Henrie : d3dcompiler_43/tests: Use CRT allocation functions.
by Alexandre Julliard 16 Nov '23

16 Nov '23

Module: wine Branch: master Commit: c64aa0006e4a33d755a57a693cd81dc1ed95fa9d URL: https://gitlab.winehq.org/wine/wine/-/commit/c64aa0006e4a33d755a57a693cd81d… Author: Alex Henrie <alexhenrie24(a)gmail.com> Date: Wed Nov 15 23:35:52 2023 -0700 d3dcompiler_43/tests: Use CRT allocation functions. --- dlls/d3dcompiler_43/tests/asm.c | 17 ++++++----------- dlls/d3dcompiler_43/tests/hlsl_d3d9.c | 15 +++++++-------- 2 files changed, 13 insertions(+), 19 deletions(-) diff --git a/dlls/d3dcompiler_43/tests/asm.c b/dlls/d3dcompiler_43/tests/asm.c index 6d3e7013017..55d84213e1e 100644 --- a/dlls/d3dcompiler_43/tests/asm.c +++ b/dlls/d3dcompiler_43/tests/asm.c @@ -1460,23 +1460,20 @@ static HRESULT WINAPI testD3DInclude_open(ID3DInclude *iface, D3D_INCLUDE_TYPE i if (!strcmp(filename, "incl.vsh")) { - buffer = HeapAlloc(GetProcessHeap(), 0, sizeof(include)); - CopyMemory(buffer, include, sizeof(include)); + buffer = strdup(include); *bytes = sizeof(include); ok(!parent_data, "Wrong parent_data value.\n"); } else if (!strcmp(filename, "incl2.vsh")) { - buffer = HeapAlloc(GetProcessHeap(), 0, sizeof(include2)); - CopyMemory(buffer, include2, sizeof(include2)); + buffer = strdup(include2); *bytes = sizeof(include2); ok(!parent_data, "Wrong parent_data value.\n"); ok(include_type == D3D_INCLUDE_LOCAL, "Wrong include type %d.\n", include_type); } else if (!strcmp(filename, "incl3.vsh")) { - buffer = HeapAlloc(GetProcessHeap(), 0, sizeof(include3)); - CopyMemory(buffer, include3, sizeof(include3)); + buffer = strdup(include3); *bytes = sizeof(include3); /* Also check for the correct parent_data content */ ok(parent_data != NULL @@ -1486,16 +1483,14 @@ static HRESULT WINAPI testD3DInclude_open(ID3DInclude *iface, D3D_INCLUDE_TYPE i } else if (!strcmp(filename, "incl4.vsh")) { - buffer = HeapAlloc(GetProcessHeap(), 0, sizeof(include4)); - CopyMemory(buffer, include4, sizeof(include4)); + buffer = strdup(include4); *bytes = sizeof(include4); ok(parent_data == NULL, "Wrong parent_data value.\n"); ok(include_type == D3D_INCLUDE_SYSTEM, "Wrong include type %d.\n", include_type); } else if (!strcmp(filename, "includes/incl.vsh")) { - buffer = HeapAlloc(GetProcessHeap(), 0, sizeof(include)); - CopyMemory(buffer, include, sizeof(include)); + buffer = strdup(include); *bytes = sizeof(include); ok(!parent_data, "Wrong parent_data value.\n"); } @@ -1512,7 +1507,7 @@ static HRESULT WINAPI testD3DInclude_open(ID3DInclude *iface, D3D_INCLUDE_TYPE i static HRESULT WINAPI testD3DInclude_close(ID3DInclude *iface, const void *data) { - HeapFree(GetProcessHeap(), 0, (void *)data); + free((void *)data); return S_OK; } diff --git a/dlls/d3dcompiler_43/tests/hlsl_d3d9.c b/dlls/d3dcompiler_43/tests/hlsl_d3d9.c index c6e7aa367f0..eed96db7248 100644 --- a/dlls/d3dcompiler_43/tests/hlsl_d3d9.c +++ b/dlls/d3dcompiler_43/tests/hlsl_d3d9.c @@ -18,7 +18,6 @@ */ #define COBJMACROS #include "wine/test.h" -#include "wine/heap.h" #include "d3dx9.h" #include "d3dcompiler.h" @@ -1515,26 +1514,26 @@ static HRESULT WINAPI test_d3dinclude_open(ID3DInclude *iface, D3D_INCLUDE_TYPE if (!strcmp(filename, "include1.h")) { - buffer = heap_alloc(strlen(include1)); - CopyMemory(buffer, include1, strlen(include1)); + buffer = strdup(include1); *bytes = strlen(include1); + buffer[*bytes] = '$'; /* everything should still work without a null terminator */ ok(include_type == D3D_INCLUDE_LOCAL, "Unexpected include type %d.\n", include_type); ok(!strncmp(include2, parent_data, strlen(include2)) || !strncmp(include3, parent_data, strlen(include3)), "Unexpected parent_data value.\n"); } else if (!strcmp(filename, "include\\include2.h")) { - buffer = heap_alloc(strlen(include2)); - CopyMemory(buffer, include2, strlen(include2)); + buffer = strdup(include2); *bytes = strlen(include2); + buffer[*bytes] = '$'; /* everything should still work without a null terminator */ ok(!parent_data, "Unexpected parent_data value.\n"); ok(include_type == D3D_INCLUDE_LOCAL, "Unexpected include type %d.\n", include_type); } else if (!strcmp(filename, "include\\include3.h")) { - buffer = heap_alloc(strlen(include3)); - CopyMemory(buffer, include3, strlen(include3)); + buffer = strdup(include3); *bytes = strlen(include3); + buffer[*bytes] = '$'; /* everything should still work without a null terminator */ ok(!parent_data, "Unexpected parent_data value.\n"); ok(include_type == D3D_INCLUDE_LOCAL, "Unexpected include type %d.\n", include_type); } @@ -1550,7 +1549,7 @@ static HRESULT WINAPI test_d3dinclude_open(ID3DInclude *iface, D3D_INCLUDE_TYPE static HRESULT WINAPI test_d3dinclude_close(ID3DInclude *iface, const void *data) { - heap_free((void *)data); + free((void *)data); return S_OK; }

1 0

Alex Henrie : ntoskrnl/tests: Put test_input on the stack in the main_test function.
by Alexandre Julliard 16 Nov '23

16 Nov '23

Module: wine Branch: master Commit: 6e45c17182224723f0bafa94623b1e37d81e89c8 URL: https://gitlab.winehq.org/wine/wine/-/commit/6e45c17182224723f0bafa94623b1e… Author: Alex Henrie <alexhenrie24(a)gmail.com> Date: Sun Nov 5 22:02:27 2023 -0700 ntoskrnl/tests: Put test_input on the stack in the main_test function. --- dlls/ntoskrnl.exe/tests/ntoskrnl.c | 14 +++++--------- 1 file changed, 5 insertions(+), 9 deletions(-) diff --git a/dlls/ntoskrnl.exe/tests/ntoskrnl.c b/dlls/ntoskrnl.exe/tests/ntoskrnl.c index 6ca4324f11b..13258b05291 100644 --- a/dlls/ntoskrnl.exe/tests/ntoskrnl.c +++ b/dlls/ntoskrnl.exe/tests/ntoskrnl.c @@ -44,7 +44,6 @@ #include "ddk/hidsdi.h" #include "ddk/hidpi.h" #include "wine/test.h" -#include "wine/heap.h" #include "wine/mssign.h" #include "driver.h" @@ -397,21 +396,18 @@ static ULONG64 modified_value; static void main_test(void) { - struct main_test_input *test_input; + struct main_test_input test_input; DWORD size; BOOL res; - test_input = heap_alloc( sizeof(*test_input) ); - test_input->process_id = GetCurrentProcessId(); - test_input->teststr_offset = (SIZE_T)((BYTE *)&teststr - (BYTE *)NtCurrentTeb()->Peb->ImageBaseAddress); - test_input->modified_value = &modified_value; + test_input.process_id = GetCurrentProcessId(); + test_input.teststr_offset = (SIZE_T)((BYTE *)&teststr - (BYTE *)NtCurrentTeb()->Peb->ImageBaseAddress); + test_input.modified_value = &modified_value; modified_value = 0; - res = DeviceIoControl(device, IOCTL_WINETEST_MAIN_TEST, test_input, sizeof(*test_input), NULL, 0, &size, NULL); + res = DeviceIoControl(device, IOCTL_WINETEST_MAIN_TEST, &test_input, sizeof(test_input), NULL, 0, &size, NULL); ok(res, "DeviceIoControl failed: %lu\n", GetLastError()); ok(!size, "got size %lu\n", size); - - heap_free(test_input); } static void test_basic_ioctl(void)

1 0

← Newer
1
...
40
41
42
43
44
45
46
...
99
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-commits November 2023