November 2022 - Wine-commits

Jacek Caban : configure: Use -nostdlib instead of -nostartfiles in WINE_TRY_PE_CFLAGS.
by Alexandre Julliard 28 Nov '22

28 Nov '22

Module: wine Branch: master Commit: c096d6cec7251050ca3c0657d6ed44fdf5cbede1 URL: https://gitlab.winehq.org/wine/wine/-/commit/c096d6cec7251050ca3c0657d6ed44… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Fri Nov 25 13:35:26 2022 +0100 configure: Use -nostdlib instead of -nostartfiles in WINE_TRY_PE_CFLAGS. -nostartfiles is not enough if clang is configured with explicit default rtlib. --- aclocal.m4 | 2 +- configure | 60 ++++++++++++++++++++++++++++++------------------------------ 2 files changed, 31 insertions(+), 31 deletions(-)

1 0

Jacek Caban : winegcc: Use -nostdlib instead of -nostartfiles on Clang MSVC targets.
by Alexandre Julliard 28 Nov '22

28 Nov '22

Module: wine Branch: master Commit: cc91a2f933ed24570aff5d05ec179d6cdeac1026 URL: https://gitlab.winehq.org/wine/wine/-/commit/cc91a2f933ed24570aff5d05ec179d… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Fri Nov 25 14:36:30 2022 +0100 winegcc: Use -nostdlib instead of -nostartfiles on Clang MSVC targets. -nostartfiles is not enough if clang is configured with explicit default rtlib. --- tools/winegcc/winegcc.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tools/winegcc/winegcc.c b/tools/winegcc/winegcc.c index 71d2b93f0c6..a850cce765f 100644 --- a/tools/winegcc/winegcc.c +++ b/tools/winegcc/winegcc.c @@ -482,7 +482,8 @@ static struct strarray get_link_args( struct options *opts, const char *output_n } if (opts->unicode_app) strarray_add( &flags, "-municode" ); if (opts->nodefaultlibs || opts->use_msvcrt) strarray_add( &flags, "-nodefaultlibs" ); - if (opts->nostartfiles || opts->use_msvcrt) strarray_add( &flags, "-nostartfiles" ); + if (opts->nostartfiles) strarray_add( &flags, "-nostartfiles" ); + if (opts->use_msvcrt) strarray_add( &flags, "-nostdlib" ); if (opts->image_base) strarray_add( &flags, strmake("-Wl,-base:%s", opts->image_base )); if (opts->subsystem) strarray_add( &flags, strmake("-Wl,-subsystem:%s", opts->subsystem ));

1 0

Jacek Caban : configure: Use -fuse-ld=lld also for compilation targets.
by Alexandre Julliard 28 Nov '22

28 Nov '22

Module: wine Branch: master Commit: eaae51b11c8c98bb6fd43572f9e120c0a3bb821e URL: https://gitlab.winehq.org/wine/wine/-/commit/eaae51b11c8c98bb6fd43572f9e120… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Fri Nov 25 14:37:30 2022 +0100 configure: Use -fuse-ld=lld also for compilation targets. This is used by clang to distinguish between MSVC and windows-itanium targets. We override default option (usually meant for native builds) to make sure that we use the right target. --- configure | 16 ++++++++-------- configure.ac | 10 +++++----- 2 files changed, 13 insertions(+), 13 deletions(-) diff --git a/configure b/configure index 35b8ff32551..5742119fc7d 100755 --- a/configure +++ b/configure @@ -10835,7 +10835,6 @@ then : done llvm_target=$target - llvm_cflags="-fuse-ld=lld" if test -z "$llvm_target" then case $wine_arch in @@ -10844,19 +10843,20 @@ then : *) llvm_target=$wine_arch-windows ;; esac fi + llvm_extra_cflags="-target $llvm_target -fuse-ld=lld" case $llvm_target in - *windows) llvm_cflags="$llvm_cflags -Wl,-subsystem:console" ;; + *windows) llvm_cflags="-Wl,-subsystem:console" ;; esac - { as_ac_var=`printf "%s\n" "ac_cv_${wine_arch}_cflags_-target $llvm_target $llvm_cflags" | $as_tr_sh` -{ printf "%s\n" "$as_me:${as_lineno-$LINENO}: checking whether $CC supports -target $llvm_target $llvm_cflags" >&5 -printf %s "checking whether $CC supports -target $llvm_target $llvm_cflags... " >&6; } + { as_ac_var=`printf "%s\n" "ac_cv_${wine_arch}_cflags_$llvm_extra_cflags $llvm_cflags" | $as_tr_sh` +{ printf "%s\n" "$as_me:${as_lineno-$LINENO}: checking whether $CC supports $llvm_extra_cflags $llvm_cflags" >&5 +printf %s "checking whether $CC supports $llvm_extra_cflags $llvm_cflags... " >&6; } if eval test \${$as_ac_var+y} then : printf %s "(cached) " >&6 else $as_nop ac_wine_try_cflags_saved=$CFLAGS ac_wine_try_cflags_saved_exeext=$ac_exeext -CFLAGS="$CFLAGS -nostdlib -nodefaultlibs -target $llvm_target $llvm_cflags" +CFLAGS="$CFLAGS -nostdlib -nodefaultlibs $llvm_extra_cflags $llvm_cflags" ac_exeext=".exe" cat confdefs.h - <<_ACEOF >conftest.$ac_ext /* end confdefs.h. */ @@ -10880,8 +10880,8 @@ if eval test \"x\$"$as_ac_var"\" = x"yes" then : target=$llvm_target eval "${wine_arch}_DELAYLOADFLAG=\"-Wl,-delayload,\"" - as_fn_append ${wine_arch}_EXTRACFLAGS " -target $target" - CFLAGS="$CFLAGS -target $target $llvm_cflags" + as_fn_append ${wine_arch}_EXTRACFLAGS " $llvm_extra_cflags" + CFLAGS="$llvm_extra_cflags $llvm_cflags" fi } eval "${wine_arch}_TARGET=\$target" fi diff --git a/configure.ac b/configure.ac index 7313fde9a4e..cb91c9c02b5 100644 --- a/configure.ac +++ b/configure.ac @@ -931,7 +931,6 @@ do dnl Check if cross compiler supports -target argument llvm_target=$target - llvm_cflags="-fuse-ld=lld" if test -z "$llvm_target" then case $wine_arch in @@ -940,14 +939,15 @@ do *) llvm_target=$wine_arch-windows ;; esac fi + llvm_extra_cflags="-target $llvm_target -fuse-ld=lld" case $llvm_target in - *windows) llvm_cflags="$llvm_cflags -Wl,-subsystem:console" ;; + *windows) llvm_cflags="-Wl,-subsystem:console" ;; esac - WINE_TRY_PE_CFLAGS([-target $llvm_target $llvm_cflags], + WINE_TRY_PE_CFLAGS([$llvm_extra_cflags $llvm_cflags], [target=$llvm_target AS_VAR_SET([${wine_arch}_DELAYLOADFLAG],["-Wl,-delayload,"]) - AS_VAR_APPEND([${wine_arch}_EXTRACFLAGS],[" -target $target"]) - CFLAGS="$CFLAGS -target $target $llvm_cflags"]) + AS_VAR_APPEND([${wine_arch}_EXTRACFLAGS],[" $llvm_extra_cflags"]) + CFLAGS="$llvm_extra_cflags $llvm_cflags"]) AS_VAR_SET([${wine_arch}_TARGET],[$target])]) AS_VAR_POPDEF([wine_cv_crosscc])

1 0

Rémi Bernon : joy.cpl: Remove unnecessary joystick forcefeedback member.
by Alexandre Julliard 28 Nov '22

28 Nov '22

Module: wine Branch: master Commit: b274278d0e118f537192c4d8597573fd68e7b2ff URL: https://gitlab.winehq.org/wine/wine/-/commit/b274278d0e118f537192c4d8597573… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Sat Nov 26 20:49:03 2022 +0100 joy.cpl: Remove unnecessary joystick forcefeedback member. --- dlls/joy.cpl/main.c | 14 +++----------- 1 file changed, 3 insertions(+), 11 deletions(-) diff --git a/dlls/joy.cpl/main.c b/dlls/joy.cpl/main.c index 59e5facb76f..bee38fe3553 100644 --- a/dlls/joy.cpl/main.c +++ b/dlls/joy.cpl/main.c @@ -78,7 +78,6 @@ struct effect struct Joystick { IDirectInputDevice8W *device; - BOOL forcefeedback; struct list effects; IDirectInputEffect *effect_selected; @@ -216,10 +215,10 @@ static BOOL CALLBACK enum_effects( const DIEFFECTINFOW *info, void *context ) static BOOL CALLBACK enum_callback(const DIDEVICEINSTANCEW *instance, void *context) { + DIDEVCAPS caps = {.dwSize = sizeof(DIDEVCAPS)}; struct JoystickData *data = context; struct Joystick *joystick; DIPROPRANGE proprange; - DIDEVCAPS caps; if (data->joysticks == NULL) { @@ -233,15 +232,11 @@ static BOOL CALLBACK enum_callback(const DIDEVICEINSTANCEW *instance, void *cont IDirectInput8_CreateDevice(data->di, &instance->guidInstance, &joystick->device, NULL); IDirectInputDevice8_SetDataFormat(joystick->device, &c_dfDIJoystick); - caps.dwSize = sizeof(caps); - IDirectInputDevice8_GetCapabilities(joystick->device, &caps); - - joystick->forcefeedback = caps.dwFlags & DIDC_FORCEFEEDBACK; - list_init( &joystick->effects ); joystick->effect_selected = NULL; - if (joystick->forcefeedback) data->num_ff++; + IDirectInputDevice8_GetCapabilities(joystick->device, &caps); + if (caps.dwFlags & DIDC_FORCEFEEDBACK) data->num_ff++; /* Set axis range to ease the GUI visualization */ proprange.diph.dwSize = sizeof(DIPROPRANGE); @@ -252,9 +247,6 @@ static BOOL CALLBACK enum_callback(const DIDEVICEINSTANCEW *instance, void *cont proprange.lMax = TEST_AXIS_MAX; IDirectInputDevice_SetProperty(joystick->device, DIPROP_RANGE, &proprange.diph); - - if (!joystick->forcefeedback) return DIENUM_CONTINUE; - IDirectInputDevice8_EnumEffects( joystick->device, enum_effects, (void *)joystick, 0 ); return DIENUM_CONTINUE;

1 0

Rémi Bernon : joy.cpl: Avoid race condition when accessing selected effect.
by Alexandre Julliard 28 Nov '22

28 Nov '22

Module: wine Branch: master Commit: c6cdea71cc7789790917f9f2977cb97878d17c95 URL: https://gitlab.winehq.org/wine/wine/-/commit/c6cdea71cc7789790917f9f2977cb9… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Sat Nov 26 20:34:56 2022 +0100 joy.cpl: Avoid race condition when accessing selected effect. --- dlls/joy.cpl/main.c | 40 ++++++++++++++++++++++++++++++++++++++-- 1 file changed, 38 insertions(+), 2 deletions(-) diff --git a/dlls/joy.cpl/main.c b/dlls/joy.cpl/main.c index bee38fe3553..ec8299753f1 100644 --- a/dlls/joy.cpl/main.c +++ b/dlls/joy.cpl/main.c @@ -105,6 +105,15 @@ struct JoystickData static HMODULE hcpl; +static CRITICAL_SECTION joy_cs; +static CRITICAL_SECTION_DEBUG joy_cs_debug = +{ + 0, 0, &joy_cs, + { &joy_cs_debug.ProcessLocksList, &joy_cs_debug.ProcessLocksList }, + 0, 0, { (DWORD_PTR)(__FILE__ ": joy_cs") } +}; +static CRITICAL_SECTION joy_cs = { &joy_cs_debug, -1, 0, 0, 0, 0 }; + /********************************************************************* * DllMain */ @@ -213,6 +222,27 @@ static BOOL CALLBACK enum_effects( const DIEFFECTINFOW *info, void *context ) return DIENUM_CONTINUE; } +static void set_selected_effect( struct Joystick *joystick, IDirectInputEffect *effect ) +{ + IDirectInputEffect *previous; + + EnterCriticalSection( &joy_cs ); + if ((previous = joystick->effect_selected)) IDirectInputEffect_Release( previous ); + if ((joystick->effect_selected = effect)) IDirectInput_AddRef( effect ); + LeaveCriticalSection( &joy_cs ); +} + +static IDirectInputEffect *get_selected_effect( struct Joystick *joystick ) +{ + IDirectInputEffect *effect; + + EnterCriticalSection( &joy_cs ); + if ((effect = joystick->effect_selected)) IDirectInputEffect_AddRef( effect ); + LeaveCriticalSection( &joy_cs ); + + return effect; +} + static BOOL CALLBACK enum_callback(const DIDEVICEINSTANCEW *instance, void *context) { DIDEVCAPS caps = {.dwSize = sizeof(DIDEVCAPS)}; @@ -277,6 +307,8 @@ static void destroy_joysticks(struct JoystickData *data) { struct effect *effect, *next_effect; + set_selected_effect( &data->joysticks[i], NULL ); + LIST_FOR_EACH_ENTRY_SAFE( effect, next_effect, &data->joysticks[i].effects, struct effect, entry ) { list_remove( &effect->entry ); @@ -867,6 +899,8 @@ static void ff_handle_effectchange(HWND hwnd, struct Joystick *joy) struct list *entry; int sel; + set_selected_effect( joy, NULL ); + sel = SendDlgItemMessageW(hwnd, IDC_FFEFFECTLIST, LB_GETCURSEL, 0, 0); if (sel < 0) return; @@ -874,7 +908,7 @@ static void ff_handle_effectchange(HWND hwnd, struct Joystick *joy) while (sel-- && entry) entry = list_next( &joy->effects, entry ); if (!entry) return; - joy->effect_selected = LIST_ENTRY( entry, struct effect, entry )->effect; + set_selected_effect( joy, LIST_ENTRY( entry, struct effect, entry )->effect ); IDirectInputDevice8_Unacquire(joy->device); IDirectInputDevice8_SetCooperativeLevel(joy->device, GetAncestor(hwnd, GA_ROOT), DISCL_BACKGROUND|DISCL_EXCLUSIVE); @@ -908,7 +942,7 @@ static DWORD WINAPI ff_input_thread(void *param) Sleep(TEST_POLL_TIME); - if (!(effect = joy->effect_selected)) continue; + if (!(effect = get_selected_effect( joy ))) continue; poll_input(joy, &state); @@ -930,6 +964,8 @@ static DWORD WINAPI ff_input_thread(void *param) IDirectInputEffect_Start( effect, 1, 0 ); break; } + + IDirectInputEffect_Release( effect ); } return 0;

1 0

Alex Henrie : wldap32: Use the strdup function instead of reimplementing it.
by Alexandre Julliard 28 Nov '22

28 Nov '22

Module: wine Branch: master Commit: b072bb494b31c5dbfde6039716bb1ce278f0da37 URL: https://gitlab.winehq.org/wine/wine/-/commit/b072bb494b31c5dbfde6039716bb1c… Author: Alex Henrie <alexhenrie24(a)gmail.com> Date: Sun Nov 27 17:02:03 2022 -0700 wldap32: Use the strdup function instead of reimplementing it. --- dlls/wldap32/ber.c | 2 +- dlls/wldap32/control.c | 2 +- dlls/wldap32/init.c | 6 +++--- dlls/wldap32/winldap_private.h | 10 +--------- 4 files changed, 6 insertions(+), 14 deletions(-) diff --git a/dlls/wldap32/ber.c b/dlls/wldap32/ber.c index 07013ad35ae..d3e8adc8433 100644 --- a/dlls/wldap32/ber.c +++ b/dlls/wldap32/ber.c @@ -266,7 +266,7 @@ ULONG WINAPIV WLDAP32_ber_scanf( WLDAP32_BerElement *ber, char *fmt, ... ) { char *str, **ptr = va_arg( list, char ** ); if ((ret = ber_scanf( BER(ber), new_fmt, &str )) == -1) break; - *ptr = strdupU( str ); + *ptr = strdup( str ); ldap_memfree( str ); break; } diff --git a/dlls/wldap32/control.c b/dlls/wldap32/control.c index ba1cefe9a45..131ea4dad64 100644 --- a/dlls/wldap32/control.c +++ b/dlls/wldap32/control.c @@ -197,7 +197,7 @@ ULONG CDECL ldap_encode_sort_controlA( LDAP *ld, LDAPSortKeyA **sortkeys, LDAPCo if ((result = ldap_create_sort_controlA( ld, sortkeys, critical, &control )) == WLDAP32_LDAP_SUCCESS) { - ret->ldctl_oid = strdupU(control->ldctl_oid); + ret->ldctl_oid = strdup( control->ldctl_oid ); bv_val_dup( &control->ldctl_value, &ret->ldctl_value ); ret->ldctl_iscritical = control->ldctl_iscritical; ldap_control_freeA( control ); diff --git a/dlls/wldap32/init.c b/dlls/wldap32/init.c index 512c558936d..b8010a2da12 100644 --- a/dlls/wldap32/init.c +++ b/dlls/wldap32/init.c @@ -36,7 +36,7 @@ static char **split_hostnames( const char *hostnames ) char **res, *str, *p, *q; unsigned int i = 0; - str = strdupU( hostnames ); + str = strdup( hostnames ); if (!str) return NULL; p = str; @@ -72,7 +72,7 @@ static char **split_hostnames( const char *hostnames ) if (isspace( *p )) { *p = '\0'; p++; - res[i] = strdupU( q ); + res[i] = strdup( q ); if (!res[i]) goto oom; i++; @@ -82,7 +82,7 @@ static char **split_hostnames( const char *hostnames ) } else { - res[i] = strdupU( q ); + res[i] = strdup( q ); if (!res[i]) goto oom; i++; } diff --git a/dlls/wldap32/winldap_private.h b/dlls/wldap32/winldap_private.h index 886c7933e9f..2902daea974 100644 --- a/dlls/wldap32/winldap_private.h +++ b/dlls/wldap32/winldap_private.h @@ -534,14 +534,6 @@ struct WLDAP32_berval ** CDECL ldap_get_values_lenW( LDAP *, LDAPMessage *, WCHA ULONG map_error( int ) DECLSPEC_HIDDEN; -static inline char *strdupU( const char *src ) -{ - char *dst; - if (!src) return NULL; - if ((dst = malloc( strlen( src ) + 1 ))) strcpy( dst, src ); - return dst; -} - static inline char *strWtoU( const WCHAR *str ) { char *ret = NULL; @@ -1314,7 +1306,7 @@ static inline char **strarrayUtoU( char **strarray ) { char **p = strarray, **q = ret; - while (*p) *q++ = strdupU( *p++ ); + while (*p) *q++ = strdup( *p++ ); *q = NULL; } }

1 0

Alexandre Julliard : avicap32: Use the Unix call helpers.
by Alexandre Julliard 28 Nov '22

28 Nov '22

Module: wine Branch: master Commit: 414304658e5183d1dcea89e3728908625bef40f8 URL: https://gitlab.winehq.org/wine/wine/-/commit/414304658e5183d1dcea89e3728908… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Nov 28 11:39:16 2022 +0100 avicap32: Use the Unix call helpers. --- dlls/avicap32/avicap32_main.c | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/dlls/avicap32/avicap32_main.c b/dlls/avicap32/avicap32_main.c index 1a1c514e931..641e3896027 100644 --- a/dlls/avicap32/avicap32_main.c +++ b/dlls/avicap32/avicap32_main.c @@ -31,7 +31,6 @@ WINE_DEFAULT_DEBUG_CHANNEL(avicap); static HINSTANCE avicap_instance; -static unixlib_handle_t unix_handle; static const WCHAR class_name[] = L"wine_avicap_class"; @@ -125,8 +124,7 @@ BOOL VFWAPI capGetDriverDescriptionW(WORD index, WCHAR *name, int name_len, WCHA struct get_device_desc_params params; params.index = index; - if (!unix_handle || __wine_unix_call(unix_handle, unix_get_device_desc, &params)) - return FALSE; + if (WINE_UNIX_CALL(unix_get_device_desc, &params)) return FALSE; TRACE("Found device name %s, version %s.\n", debugstr_w(params.name), debugstr_w(params.version)); lstrcpynW(name, params.name, name_len); @@ -139,9 +137,8 @@ BOOL WINAPI DllMain(HINSTANCE instance, DWORD reason, void *reserved) switch (reason) { case DLL_PROCESS_ATTACH: - NtQueryVirtualMemory(GetCurrentProcess(), instance, - MemoryWineUnixFuncs, &unix_handle, sizeof(unix_handle), NULL); DisableThreadLibraryCalls(instance); + __wine_init_unix_call(); register_class(); avicap_instance = instance; break;

1 0

Alexandre Julliard : capi2032: Use the Unix call helpers.
by Alexandre Julliard 28 Nov '22

28 Nov '22

Module: wine Branch: master Commit: 6677255d665e83b4f8d67f019232ef5ecac8e15f URL: https://gitlab.winehq.org/wine/wine/-/commit/6677255d665e83b4f8d67f019232ef… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Nov 28 11:39:33 2022 +0100 capi2032: Use the Unix call helpers. --- dlls/capi2032/cap20wxx.c | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/dlls/capi2032/cap20wxx.c b/dlls/capi2032/cap20wxx.c index 96007f51e96..c8c008d7c5f 100644 --- a/dlls/capi2032/cap20wxx.c +++ b/dlls/capi2032/cap20wxx.c @@ -29,9 +29,7 @@ WINE_DEFAULT_DEBUG_CHANNEL(capi); -static unixlib_handle_t capi_handle; - -#define CAPI_CALL( func, params ) __wine_unix_call( capi_handle, unix_ ## func, params ) +#define CAPI_CALL( func, params ) WINE_UNIX_CALL( unix_ ## func, params ) BOOL WINAPI DllMain( HINSTANCE instance, DWORD reason, LPVOID reserved ) @@ -40,8 +38,7 @@ BOOL WINAPI DllMain( HINSTANCE instance, DWORD reason, LPVOID reserved ) { case DLL_PROCESS_ATTACH: DisableThreadLibraryCalls( instance ); - return !NtQueryVirtualMemory( GetCurrentProcess(), instance, MemoryWineUnixFuncs, - &capi_handle, sizeof(capi_handle), NULL ); + return !__wine_init_unix_call(); } return TRUE; }

1 0

Alexandre Julliard : ctapi32: Use the Unix call helpers.
by Alexandre Julliard 28 Nov '22

28 Nov '22

Module: wine Branch: master Commit: 81493d0f8c5e1c38b96481e0af31f168d46f6e91 URL: https://gitlab.winehq.org/wine/wine/-/commit/81493d0f8c5e1c38b96481e0af31f1… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Nov 28 11:39:45 2022 +0100 ctapi32: Use the Unix call helpers. --- dlls/ctapi32/ctapi32.c | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/dlls/ctapi32/ctapi32.c b/dlls/ctapi32/ctapi32.c index 0f0ee399e1c..64ae904ff41 100644 --- a/dlls/ctapi32/ctapi32.c +++ b/dlls/ctapi32/ctapi32.c @@ -32,9 +32,7 @@ WINE_DEFAULT_DEBUG_CHANNEL(ctapi32); #define FALLBACK_LIBCTAPI "libctapi.so" -static unixlib_handle_t ctapi_handle; - -#define CTAPI_CALL( func, params ) __wine_unix_call( ctapi_handle, unix_ ## func, params ) +#define CTAPI_CALL( func, params ) WINE_UNIX_CALL( unix_ ## func, params ) static BOOL load_functions(void) { @@ -104,8 +102,7 @@ BOOL WINAPI DllMain (HINSTANCE hinstDLL, DWORD fdwReason, LPVOID lpvReserved) { case DLL_PROCESS_ATTACH: DisableThreadLibraryCalls(hinstDLL); - if (NtQueryVirtualMemory( GetCurrentProcess(), hinstDLL, MemoryWineUnixFuncs, - &ctapi_handle, sizeof(ctapi_handle), NULL )) + if (__wine_init_unix_call()) return FALSE; if (!load_functions()) return FALSE;

1 0

Alexandre Julliard : dnsapi: Use the Unix call helpers.
by Alexandre Julliard 28 Nov '22

28 Nov '22

Module: wine Branch: master Commit: a62d3ab841c21561e0ab1f860b2c592934520cca URL: https://gitlab.winehq.org/wine/wine/-/commit/a62d3ab841c21561e0ab1f860b2c59… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Nov 28 11:40:07 2022 +0100 dnsapi: Use the Unix call helpers. --- dlls/dnsapi/dnsapi.h | 4 +--- dlls/dnsapi/main.c | 5 +---- 2 files changed, 2 insertions(+), 7 deletions(-) diff --git a/dlls/dnsapi/dnsapi.h b/dlls/dnsapi/dnsapi.h index 250b0a1b4c3..89324dd9c29 100644 --- a/dlls/dnsapi/dnsapi.h +++ b/dlls/dnsapi/dnsapi.h @@ -129,6 +129,4 @@ enum unix_funcs unix_query, }; -extern unixlib_handle_t resolv_handle; - -#define RESOLV_CALL( func, params ) __wine_unix_call( resolv_handle, unix_ ## func, params ) +#define RESOLV_CALL( func, params ) WINE_UNIX_CALL( unix_ ## func, params ) diff --git a/dlls/dnsapi/main.c b/dlls/dnsapi/main.c index 23738bf36fe..6324d46e173 100644 --- a/dlls/dnsapi/main.c +++ b/dlls/dnsapi/main.c @@ -32,8 +32,6 @@ WINE_DEFAULT_DEBUG_CHANNEL(dnsapi); -unixlib_handle_t resolv_handle = 0; - BOOL WINAPI DllMain( HINSTANCE hinst, DWORD reason, LPVOID reserved ) { TRACE( "(%p, %lu, %p)\n", hinst, reason, reserved ); @@ -42,8 +40,7 @@ BOOL WINAPI DllMain( HINSTANCE hinst, DWORD reason, LPVOID reserved ) { case DLL_PROCESS_ATTACH: DisableThreadLibraryCalls( hinst ); - if (NtQueryVirtualMemory( GetCurrentProcess(), hinst, MemoryWineUnixFuncs, - &resolv_handle, sizeof(resolv_handle), NULL )) + if (__wine_init_unix_call()) ERR( "No libresolv support, expect problems\n" ); break; case DLL_PROCESS_DETACH:

1 0