- Wine-Commits - list.winehq.org

Alexandre Julliard : winegstreamer: Use the Unix call helpers.
by Alexandre Julliard Nov. 25, 2022

Nov. 25, 2022

Module: wine Branch: master Commit: 86554e8199a6c2044b5f68eef4f3d44ddf880278 URL: https://gitlab.winehq.org/wine/wine/-/commit/86554e8199a6c2044b5f68eef4f3d4… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Nov 25 10:19:30 2022 +0100 winegstreamer: Use the Unix call helpers. --- dlls/winegstreamer/main.c | 49 ++++++++++++++++++++++------------------------- 1 file changed, 23 insertions(+), 26 deletions(-) diff --git a/dlls/winegstreamer/main.c b/dlls/winegstreamer/main.c index ea52797712e..fc7344df50c 100644 --- a/dlls/winegstreamer/main.c +++ b/dlls/winegstreamer/main.c @@ -29,8 +29,6 @@ #include "gst_guids.h" #include "wmcodecdsp.h" -static unixlib_handle_t unix_handle; - WINE_DEFAULT_DEBUG_CHANNEL(quartz); DEFINE_GUID(GUID_NULL, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0); @@ -75,7 +73,7 @@ struct wg_parser *wg_parser_create(enum wg_parser_type type, bool unlimited_buff TRACE("type %#x, unlimited_buffering %d.\n", type, unlimited_buffering); - if (__wine_unix_call(unix_handle, unix_wg_parser_create, &params)) + if (WINE_UNIX_CALL(unix_wg_parser_create, &params)) return NULL; TRACE("Returning parser %p.\n", params.parser); @@ -87,7 +85,7 @@ void wg_parser_destroy(struct wg_parser *parser) { TRACE("parser %p.\n", parser); - __wine_unix_call(unix_handle, unix_wg_parser_destroy, parser); + WINE_UNIX_CALL(unix_wg_parser_destroy, parser); } HRESULT wg_parser_connect(struct wg_parser *parser, uint64_t file_size) @@ -100,14 +98,14 @@ HRESULT wg_parser_connect(struct wg_parser *parser, uint64_t file_size) TRACE("parser %p, file_size %I64u.\n", parser, file_size); - return __wine_unix_call(unix_handle, unix_wg_parser_connect, &params); + return WINE_UNIX_CALL(unix_wg_parser_connect, &params); } void wg_parser_disconnect(struct wg_parser *parser) { TRACE("parser %p.\n", parser); - __wine_unix_call(unix_handle, unix_wg_parser_disconnect, parser); + WINE_UNIX_CALL(unix_wg_parser_disconnect, parser); } bool wg_parser_get_next_read_offset(struct wg_parser *parser, uint64_t *offset, uint32_t *size) @@ -119,7 +117,7 @@ bool wg_parser_get_next_read_offset(struct wg_parser *parser, uint64_t *offset, TRACE("parser %p, offset %p, size %p.\n", parser, offset, size); - if (__wine_unix_call(unix_handle, unix_wg_parser_get_next_read_offset, &params)) + if (WINE_UNIX_CALL(unix_wg_parser_get_next_read_offset, &params)) return false; *offset = params.offset; *size = params.size; @@ -137,7 +135,7 @@ void wg_parser_push_data(struct wg_parser *parser, const void *data, uint32_t si TRACE("parser %p, data %p, size %u.\n", parser, data, size); - __wine_unix_call(unix_handle, unix_wg_parser_push_data, &params); + WINE_UNIX_CALL(unix_wg_parser_push_data, &params); } uint32_t wg_parser_get_stream_count(struct wg_parser *parser) @@ -149,7 +147,7 @@ uint32_t wg_parser_get_stream_count(struct wg_parser *parser) TRACE("parser %p.\n", parser); - __wine_unix_call(unix_handle, unix_wg_parser_get_stream_count, &params); + WINE_UNIX_CALL(unix_wg_parser_get_stream_count, &params); return params.count; } @@ -163,7 +161,7 @@ struct wg_parser_stream *wg_parser_get_stream(struct wg_parser *parser, uint32_t TRACE("parser %p, index %u.\n", parser, index); - __wine_unix_call(unix_handle, unix_wg_parser_get_stream, &params); + WINE_UNIX_CALL(unix_wg_parser_get_stream, &params); TRACE("Returning stream %p.\n", params.stream); return params.stream; @@ -179,7 +177,7 @@ void wg_parser_stream_get_preferred_format(struct wg_parser_stream *stream, stru TRACE("stream %p, format %p.\n", stream, format); - __wine_unix_call(unix_handle, unix_wg_parser_stream_get_preferred_format, &params); + WINE_UNIX_CALL(unix_wg_parser_stream_get_preferred_format, &params); } void wg_parser_stream_enable(struct wg_parser_stream *stream, const struct wg_format *format) @@ -192,14 +190,14 @@ void wg_parser_stream_enable(struct wg_parser_stream *stream, const struct wg_fo TRACE("stream %p, format %p.\n", stream, format); - __wine_unix_call(unix_handle, unix_wg_parser_stream_enable, &params); + WINE_UNIX_CALL(unix_wg_parser_stream_enable, &params); } void wg_parser_stream_disable(struct wg_parser_stream *stream) { TRACE("stream %p.\n", stream); - __wine_unix_call(unix_handle, unix_wg_parser_stream_disable, stream); + WINE_UNIX_CALL(unix_wg_parser_stream_disable, stream); } bool wg_parser_stream_get_buffer(struct wg_parser *parser, struct wg_parser_stream *stream, @@ -214,7 +212,7 @@ bool wg_parser_stream_get_buffer(struct wg_parser *parser, struct wg_parser_stre TRACE("parser %p, stream %p, buffer %p.\n", parser, stream, buffer); - return !__wine_unix_call(unix_handle, unix_wg_parser_stream_get_buffer, &params); + return !WINE_UNIX_CALL(unix_wg_parser_stream_get_buffer, &params); } bool wg_parser_stream_copy_buffer(struct wg_parser_stream *stream, @@ -230,14 +228,14 @@ bool wg_parser_stream_copy_buffer(struct wg_parser_stream *stream, TRACE("stream %p, data %p, offset %u, size %u.\n", stream, data, offset, size); - return !__wine_unix_call(unix_handle, unix_wg_parser_stream_copy_buffer, &params); + return !WINE_UNIX_CALL(unix_wg_parser_stream_copy_buffer, &params); } void wg_parser_stream_release_buffer(struct wg_parser_stream *stream) { TRACE("stream %p.\n", stream); - __wine_unix_call(unix_handle, unix_wg_parser_stream_release_buffer, stream); + WINE_UNIX_CALL(unix_wg_parser_stream_release_buffer, stream); } void wg_parser_stream_notify_qos(struct wg_parser_stream *stream, @@ -255,7 +253,7 @@ void wg_parser_stream_notify_qos(struct wg_parser_stream *stream, TRACE("stream %p, underflow %d, proportion %.16e, diff %I64d, timestamp %I64u.\n", stream, underflow, proportion, diff, timestamp); - __wine_unix_call(unix_handle, unix_wg_parser_stream_notify_qos, &params); + WINE_UNIX_CALL(unix_wg_parser_stream_notify_qos, &params); } uint64_t wg_parser_stream_get_duration(struct wg_parser_stream *stream) @@ -267,7 +265,7 @@ uint64_t wg_parser_stream_get_duration(struct wg_parser_stream *stream) TRACE("stream %p.\n", stream); - __wine_unix_call(unix_handle, unix_wg_parser_stream_get_duration, &params); + WINE_UNIX_CALL(unix_wg_parser_stream_get_duration, &params); TRACE("Returning duration %I64u.\n", params.duration); return params.duration; @@ -289,7 +287,7 @@ void wg_parser_stream_seek(struct wg_parser_stream *stream, double rate, TRACE("stream %p, rate %.16e, start_pos %I64u, stop_pos %I64u, start_flags %#lx, stop_flags %#lx.\n", stream, rate, start_pos, stop_pos, start_flags, stop_flags); - __wine_unix_call(unix_handle, unix_wg_parser_stream_seek, &params); + WINE_UNIX_CALL(unix_wg_parser_stream_seek, &params); } struct wg_transform *wg_transform_create(const struct wg_format *input_format, @@ -303,7 +301,7 @@ struct wg_transform *wg_transform_create(const struct wg_format *input_format, TRACE("input_format %p, output_format %p.\n", input_format, output_format); - if (__wine_unix_call(unix_handle, unix_wg_transform_create, &params)) + if (WINE_UNIX_CALL(unix_wg_transform_create, &params)) return NULL; TRACE("Returning transform %p.\n", params.transform); @@ -314,7 +312,7 @@ void wg_transform_destroy(struct wg_transform *transform) { TRACE("transform %p.\n", transform); - __wine_unix_call(unix_handle, unix_wg_transform_destroy, transform); + WINE_UNIX_CALL(unix_wg_transform_destroy, transform); } HRESULT wg_transform_push_data(struct wg_transform *transform, struct wg_sample *sample) @@ -328,7 +326,7 @@ HRESULT wg_transform_push_data(struct wg_transform *transform, struct wg_sample TRACE("transform %p, sample %p.\n", transform, sample); - if ((status = __wine_unix_call(unix_handle, unix_wg_transform_push_data, &params))) + if ((status = WINE_UNIX_CALL(unix_wg_transform_push_data, &params))) return HRESULT_FROM_NT(status); return params.result; @@ -347,7 +345,7 @@ HRESULT wg_transform_read_data(struct wg_transform *transform, struct wg_sample TRACE("transform %p, sample %p, format %p.\n", transform, sample, format); - if ((status = __wine_unix_call(unix_handle, unix_wg_transform_read_data, &params))) + if ((status = WINE_UNIX_CALL(unix_wg_transform_read_data, &params))) return HRESULT_FROM_NT(status); return params.result; @@ -363,7 +361,7 @@ bool wg_transform_set_output_format(struct wg_transform *transform, struct wg_fo TRACE("transform %p, format %p.\n", transform, format); - return !__wine_unix_call(unix_handle, unix_wg_transform_set_output_format, &params); + return !WINE_UNIX_CALL(unix_wg_transform_set_output_format, &params); } BOOL WINAPI DllMain(HINSTANCE instance, DWORD reason, void *reserved) @@ -371,8 +369,7 @@ BOOL WINAPI DllMain(HINSTANCE instance, DWORD reason, void *reserved) if (reason == DLL_PROCESS_ATTACH) { DisableThreadLibraryCalls(instance); - NtQueryVirtualMemory(GetCurrentProcess(), instance, MemoryWineUnixFuncs, - &unix_handle, sizeof(unix_handle), NULL); + __wine_init_unix_call(); } return TRUE; }

1 0

Alexandre Julliard : opencl: Use the Unix call helpers.
by Alexandre Julliard Nov. 25, 2022

Nov. 25, 2022

Module: wine Branch: master Commit: 3b50ca0ff8794d5b1eb9d0d8591234f927461463 URL: https://gitlab.winehq.org/wine/wine/-/commit/3b50ca0ff8794d5b1eb9d0d8591234… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Nov 25 10:14:03 2022 +0100 opencl: Use the Unix call helpers. --- dlls/opencl/opencl_private.h | 4 +--- dlls/opencl/pe_wrappers.c | 5 +---- 2 files changed, 2 insertions(+), 7 deletions(-) diff --git a/dlls/opencl/opencl_private.h b/dlls/opencl/opencl_private.h index 4c582093b6d..28968296221 100644 --- a/dlls/opencl/opencl_private.h +++ b/dlls/opencl/opencl_private.h @@ -33,8 +33,6 @@ BOOL extension_is_supported( const char *name, size_t len ) DECLSPEC_HIDDEN; -extern unixlib_handle_t opencl_handle DECLSPEC_HIDDEN; - -#define OPENCL_CALL( func, params ) __wine_unix_call( opencl_handle, unix_ ## func, params ) +#define OPENCL_CALL( func, params ) WINE_UNIX_CALL( unix_ ## func, params ) #endif diff --git a/dlls/opencl/pe_wrappers.c b/dlls/opencl/pe_wrappers.c index 29ae1afaaff..3b6846009c1 100644 --- a/dlls/opencl/pe_wrappers.c +++ b/dlls/opencl/pe_wrappers.c @@ -25,8 +25,6 @@ WINE_DEFAULT_DEBUG_CHANNEL(opencl); -unixlib_handle_t opencl_handle = 0; - static cl_int filter_extensions( const char *unix_exts, SIZE_T size, char *win_exts, size_t *ret_size ) { char *p = win_exts; @@ -284,8 +282,7 @@ BOOL WINAPI DllMain( HINSTANCE instance, DWORD reason, void *reserved ) if (reason == DLL_PROCESS_ATTACH) { DisableThreadLibraryCalls( instance ); - return !NtQueryVirtualMemory( GetCurrentProcess(), instance, MemoryWineUnixFuncs, - &opencl_handle, sizeof(opencl_handle), NULL ); + return !__wine_init_unix_call(); } return TRUE; }

1 0

Alexandre Julliard : kerberos: Use the Unix call helpers.
by Alexandre Julliard Nov. 25, 2022

Nov. 25, 2022

Module: wine Branch: master Commit: f7b55ffafaac0a32545d948f62363ec2f5b64ba4 URL: https://gitlab.winehq.org/wine/wine/-/commit/f7b55ffafaac0a32545d948f62363e… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Nov 25 10:12:17 2022 +0100 kerberos: Use the Unix call helpers. --- dlls/kerberos/krb5_ap.c | 30 ++++-------------------------- dlls/kerberos/unixlib.h | 4 +--- 2 files changed, 5 insertions(+), 29 deletions(-) diff --git a/dlls/kerberos/krb5_ap.c b/dlls/kerberos/krb5_ap.c index 84d742079e6..42d3358a20d 100644 --- a/dlls/kerberos/krb5_ap.c +++ b/dlls/kerberos/krb5_ap.c @@ -39,10 +39,6 @@ WINE_DEFAULT_DEBUG_CHANNEL(kerberos); -static HINSTANCE instance; - -unixlib_handle_t krb5_handle = 0; - #define KERBEROS_CAPS \ ( SECPKG_FLAG_INTEGRITY \ | SECPKG_FLAG_PRIVACY \ @@ -134,11 +130,9 @@ static NTSTATUS NTAPI kerberos_LsaApInitializePackage(ULONG package_id, PLSA_DIS { char *kerberos_name; - if (!krb5_handle) + if (!__wine_unixlib_handle) { - if (NtQueryVirtualMemory( GetCurrentProcess(), instance, MemoryWineUnixFuncs, - &krb5_handle, sizeof(krb5_handle), NULL ) || - KRB5_CALL( process_attach, NULL )) + if (__wine_init_unix_call() || KRB5_CALL( process_attach, NULL )) ERR( "no Kerberos support, expect problems\n" ); } @@ -569,11 +563,9 @@ static NTSTATUS NTAPI kerberos_SpInitialize(ULONG_PTR package_id, SECPKG_PARAMET { TRACE("%Iu, %p, %p\n", package_id, params, lsa_function_table); - if (!krb5_handle) + if (!__wine_unixlib_handle) { - if (NtQueryVirtualMemory( GetCurrentProcess(), instance, MemoryWineUnixFuncs, - &krb5_handle, sizeof(krb5_handle), NULL ) || - KRB5_CALL( process_attach, NULL )) + if (__wine_init_unix_call() || KRB5_CALL( process_attach, NULL )) WARN( "no Kerberos support\n" ); return STATUS_UNSUCCESSFUL; } @@ -781,17 +773,3 @@ NTSTATUS NTAPI SpUserModeInitialize(ULONG lsa_version, PULONG package_version, *table_count = 1; return STATUS_SUCCESS; } - -BOOL WINAPI DllMain( HINSTANCE hinst, DWORD reason, void *reserved ) -{ - switch (reason) - { - case DLL_PROCESS_ATTACH: - instance = hinst; - DisableThreadLibraryCalls( hinst ); - break; - case DLL_PROCESS_DETACH: - break; - } - return TRUE; -} diff --git a/dlls/kerberos/unixlib.h b/dlls/kerberos/unixlib.h index f811f583bb2..1725f813a16 100644 --- a/dlls/kerberos/unixlib.h +++ b/dlls/kerberos/unixlib.h @@ -139,6 +139,4 @@ enum unix_funcs unix_verify_signature, }; -extern unixlib_handle_t krb5_handle DECLSPEC_HIDDEN; - -#define KRB5_CALL( func, params ) __wine_unix_call( krb5_handle, unix_ ## func, params ) +#define KRB5_CALL( func, params ) WINE_UNIX_CALL( unix_ ## func, params )

1 0

Alexandre Julliard : dwrite: Use the Unix call helpers.
by Alexandre Julliard Nov. 25, 2022

Nov. 25, 2022

Module: wine Branch: master Commit: 5392bd303efc0073b49f53d54a637a919527d542 URL: https://gitlab.winehq.org/wine/wine/-/commit/5392bd303efc0073b49f53d54a637a… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Nov 25 10:11:36 2022 +0100 dwrite: Use the Unix call helpers. --- dlls/dwrite/main.c | 6 ++---- dlls/dwrite/unixlib.h | 4 +--- 2 files changed, 3 insertions(+), 7 deletions(-) diff --git a/dlls/dwrite/main.c b/dlls/dwrite/main.c index 8cc5d4dccf7..6776b1ce974 100644 --- a/dlls/dwrite/main.c +++ b/dlls/dwrite/main.c @@ -36,7 +36,6 @@ WINE_DEFAULT_DEBUG_CHANNEL(dwrite); HMODULE dwrite_module = 0; -unixlib_handle_t unixlib_handle = 0; static IDWriteFactory7 *shared_factory; static void release_shared_factory(IDWriteFactory7 *factory); @@ -47,8 +46,7 @@ BOOL WINAPI DllMain(HINSTANCE hinstDLL, DWORD reason, LPVOID reserved) case DLL_PROCESS_ATTACH: dwrite_module = hinstDLL; DisableThreadLibraryCalls( hinstDLL ); - if (!NtQueryVirtualMemory(GetCurrentProcess(), hinstDLL, MemoryWineUnixFuncs, - &unixlib_handle, sizeof(unixlib_handle), NULL)) + if (!__wine_init_unix_call()) UNIX_CALL(process_attach, NULL); init_local_fontfile_loader(); break; @@ -56,7 +54,7 @@ BOOL WINAPI DllMain(HINSTANCE hinstDLL, DWORD reason, LPVOID reserved) if (reserved) break; release_shared_factory(shared_factory); release_system_fallback_data(); - if (unixlib_handle) UNIX_CALL(process_detach, NULL); + UNIX_CALL(process_detach, NULL); } return TRUE; } diff --git a/dlls/dwrite/unixlib.h b/dlls/dwrite/unixlib.h index 533c3168b99..c63d5bea722 100644 --- a/dlls/dwrite/unixlib.h +++ b/dlls/dwrite/unixlib.h @@ -108,6 +108,4 @@ enum font_backend_funcs unix_get_design_glyph_metrics, }; -extern unixlib_handle_t unixlib_handle DECLSPEC_HIDDEN; - -#define UNIX_CALL( func, params ) __wine_unix_call( unixlib_handle, unix_ ## func, params ) +#define UNIX_CALL( func, params ) WINE_UNIX_CALL( unix_ ## func, params )

1 0

Alexandre Julliard : crypt32: Use the Unix call helpers.
by Alexandre Julliard Nov. 25, 2022

Nov. 25, 2022

Module: wine Branch: master Commit: f340406417954fdc421d7102365235d71a01ae9a URL: https://gitlab.winehq.org/wine/wine/-/commit/f340406417954fdc421d7102365235… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Nov 25 10:10:59 2022 +0100 crypt32: Use the Unix call helpers. --- dlls/crypt32/crypt32_private.h | 4 +--- dlls/crypt32/main.c | 4 +--- 2 files changed, 2 insertions(+), 6 deletions(-) diff --git a/dlls/crypt32/crypt32_private.h b/dlls/crypt32/crypt32_private.h index f4131bf2694..41c5ec523be 100644 --- a/dlls/crypt32/crypt32_private.h +++ b/dlls/crypt32/crypt32_private.h @@ -508,8 +508,6 @@ enum unix_funcs unix_enum_root_certs, }; -extern unixlib_handle_t crypt32_handle; - -#define CRYPT32_CALL( func, params ) __wine_unix_call( crypt32_handle, unix_ ## func, params ) +#define CRYPT32_CALL( func, params ) WINE_UNIX_CALL( unix_ ## func, params ) #endif diff --git a/dlls/crypt32/main.c b/dlls/crypt32/main.c index 4a94bed9b6b..b320c895e15 100644 --- a/dlls/crypt32/main.c +++ b/dlls/crypt32/main.c @@ -35,7 +35,6 @@ WINE_DEFAULT_DEBUG_CHANNEL(crypt); static HCRYPTPROV hDefProv; HINSTANCE hInstance; -unixlib_handle_t crypt32_handle = 0; static CRITICAL_SECTION prov_param_cs; static CRITICAL_SECTION_DEBUG prov_param_cs_debug = @@ -56,8 +55,7 @@ BOOL WINAPI DllMain(HINSTANCE hInst, DWORD reason, PVOID pvReserved) DisableThreadLibraryCalls(hInst); init_empty_store(); crypt_oid_init(); - if (NtQueryVirtualMemory( GetCurrentProcess(), hInst, MemoryWineUnixFuncs, - &crypt32_handle, sizeof(crypt32_handle), NULL )) + if (__wine_init_unix_call()) return FALSE; CRYPT32_CALL( process_attach, NULL ); break;

1 0

Ivo Ivanov : windows.gaming.input: Implement the number of FFB axes according to the SupportedAxes property.
by Alexandre Julliard Nov. 25, 2022

Nov. 25, 2022

Module: wine Branch: master Commit: a1ce348c9512a5be108dfc9596a160e991b65c61 URL: https://gitlab.winehq.org/wine/wine/-/commit/a1ce348c9512a5be108dfc9596a160… Author: Ivo Ivanov <logos128(a)gmail.com> Date: Sat Jul 23 16:54:13 2022 +0300 windows.gaming.input: Implement the number of FFB axes according to the SupportedAxes property. --- dlls/windows.gaming.input/force_feedback.c | 62 +++++++++++++++++++++++------- 1 file changed, 48 insertions(+), 14 deletions(-) diff --git a/dlls/windows.gaming.input/force_feedback.c b/dlls/windows.gaming.input/force_feedback.c index f24f0627d6f..f7a233b46d4 100644 --- a/dlls/windows.gaming.input/force_feedback.c +++ b/dlls/windows.gaming.input/force_feedback.c @@ -113,6 +113,7 @@ static HRESULT WINAPI effect_impl_put_Parameters( IWineForceFeedbackEffectImpl * WineForceFeedbackEffectEnvelope *envelope ) { struct effect *impl = impl_from_IWineForceFeedbackEffectImpl( iface ); + DWORD count = 0; HRESULT hr; TRACE( "iface %p, params %p, envelope %p.\n", iface, &params, envelope ); @@ -125,9 +126,9 @@ static HRESULT WINAPI effect_impl_put_Parameters( IWineForceFeedbackEffectImpl * impl->constant_force.lMagnitude = round( params.constant.gain * params.constant.direction.X * 10000 ); impl->params.dwDuration = params.constant.duration.Duration / 10; impl->params.dwStartDelay = params.constant.start_delay.Duration / 10; - impl->directions[0] = round( -params.constant.direction.X * 10000 ); - impl->directions[1] = round( -params.constant.direction.Y * 10000 ); - impl->directions[2] = round( -params.constant.direction.Z * 10000 ); + if (impl->axes[count] == DIJOFS_X) impl->directions[count++] = round( -params.constant.direction.X * 10000 ); + if (impl->axes[count] == DIJOFS_Y) impl->directions[count++] = round( -params.constant.direction.Y * 10000 ); + if (impl->axes[count] == DIJOFS_Z) impl->directions[count++] = round( -params.constant.direction.Z * 10000 ); break; case WineForceFeedbackEffectType_Ramp: @@ -136,9 +137,9 @@ static HRESULT WINAPI effect_impl_put_Parameters( IWineForceFeedbackEffectImpl * impl->ramp_force.lEnd = round( params.ramp.gain * params.ramp.end_vector.X * 10000 ); impl->params.dwDuration = params.ramp.duration.Duration / 10; impl->params.dwStartDelay = params.ramp.start_delay.Duration / 10; - impl->directions[0] = round( -params.ramp.start_vector.X * 10000 ); - impl->directions[1] = round( -params.ramp.start_vector.Y * 10000 ); - impl->directions[2] = round( -params.ramp.start_vector.Z * 10000 ); + if (impl->axes[count] == DIJOFS_X) impl->directions[count++] = round( -params.ramp.start_vector.X * 10000 ); + if (impl->axes[count] == DIJOFS_Y) impl->directions[count++] = round( -params.ramp.start_vector.Y * 10000 ); + if (impl->axes[count] == DIJOFS_Z) impl->directions[count++] = round( -params.ramp.start_vector.Z * 10000 ); break; case WineForceFeedbackEffectType_Periodic_SineWave: @@ -153,9 +154,9 @@ static HRESULT WINAPI effect_impl_put_Parameters( IWineForceFeedbackEffectImpl * impl->periodic.lOffset = round( params.periodic.bias * 10000 ); impl->params.dwDuration = params.periodic.duration.Duration / 10; impl->params.dwStartDelay = params.periodic.start_delay.Duration / 10; - impl->directions[0] = round( -params.periodic.direction.X * 10000 ); - impl->directions[1] = round( -params.periodic.direction.Y * 10000 ); - impl->directions[2] = round( -params.periodic.direction.Z * 10000 ); + if (impl->axes[count] == DIJOFS_X) impl->directions[count++] = round( -params.periodic.direction.X * 10000 ); + if (impl->axes[count] == DIJOFS_Y) impl->directions[count++] = round( -params.periodic.direction.Y * 10000 ); + if (impl->axes[count] == DIJOFS_Z) impl->directions[count++] = round( -params.periodic.direction.Z * 10000 ); break; case WineForceFeedbackEffectType_Condition_Spring: @@ -171,9 +172,9 @@ static HRESULT WINAPI effect_impl_put_Parameters( IWineForceFeedbackEffectImpl * impl->condition.lOffset = round( params.condition.bias * 10000 ); impl->params.dwDuration = -1; impl->params.dwStartDelay = 0; - impl->directions[0] = round( params.condition.direction.X * 10000 ); - impl->directions[1] = round( params.condition.direction.Y * 10000 ); - impl->directions[2] = round( params.condition.direction.Z * 10000 ); + if (impl->axes[count] == DIJOFS_X) impl->directions[count++] = round( params.condition.direction.X * 10000 ); + if (impl->axes[count] == DIJOFS_Y) impl->directions[count++] = round( params.condition.direction.Y * 10000 ); + if (impl->axes[count] == DIJOFS_Z) impl->directions[count++] = round( params.condition.direction.Z * 10000 ); break; } @@ -374,7 +375,7 @@ HRESULT force_feedback_effect_create( enum WineForceFeedbackEffectType type, IIn impl->params.dwTriggerButton = -1; impl->params.dwGain = 10000; impl->params.dwFlags = DIEFF_CARTESIAN|DIEFF_OBJECTOFFSETS; - impl->params.cAxes = 2; + impl->params.cAxes = -1; impl->axes[0] = DIJOFS_X; impl->axes[1] = DIJOFS_Y; impl->axes[2] = DIJOFS_Z; @@ -563,11 +564,43 @@ static HRESULT WINAPI motor_get_SupportedAxes( IForceFeedbackMotor *iface, enum static HRESULT WINAPI motor_load_effect_async( IUnknown *invoker, IUnknown *param, PROPVARIANT *result ) { struct effect *effect = impl_from_IForceFeedbackEffect( (IForceFeedbackEffect *)param ); - struct motor *impl = impl_from_IForceFeedbackMotor( (IForceFeedbackMotor *)invoker ); + IForceFeedbackMotor *motor = (IForceFeedbackMotor *)invoker; + struct motor *impl = impl_from_IForceFeedbackMotor( motor ); + ForceFeedbackEffectAxes supported_axes = 0; IDirectInputEffect *dinput_effect; HRESULT hr; EnterCriticalSection( &effect->cs ); + + if (FAILED(hr = IForceFeedbackMotor_get_SupportedAxes( motor, &supported_axes ))) + { + WARN( "get_SupportedAxes for motor %p returned %#lx\n", motor, hr ); + effect->params.cAxes = 0; + } + else if (effect->params.cAxes == -1) + { + DWORD count = 0; + + /* initialize axis mapping and re-map directions that were set with the initial mapping */ + if (supported_axes & ForceFeedbackEffectAxes_X) + { + effect->directions[count] = effect->directions[0]; + effect->axes[count++] = DIJOFS_X; + } + if (supported_axes & ForceFeedbackEffectAxes_Y) + { + effect->directions[count] = effect->directions[1]; + effect->axes[count++] = DIJOFS_Y; + } + if (supported_axes & ForceFeedbackEffectAxes_Z) + { + effect->directions[count] = effect->directions[2]; + effect->axes[count++] = DIJOFS_Z; + } + + effect->params.cAxes = count; + } + if (SUCCEEDED(hr = IDirectInputDevice8_CreateEffect( impl->device, &effect->type, &effect->params, &dinput_effect, NULL ))) { @@ -575,6 +608,7 @@ static HRESULT WINAPI motor_load_effect_async( IUnknown *invoker, IUnknown *para effect->effect = dinput_effect; IDirectInputEffect_AddRef( effect->effect ); } + LeaveCriticalSection( &effect->cs ); result->vt = VT_UI4;

1 0

Ivo Ivanov : dinput/tests: Reduce the available FFB axes to X and Y in test_windows_gaming_input.
by Alexandre Julliard Nov. 25, 2022

Nov. 25, 2022

Module: wine Branch: master Commit: c8217873383a7f9b09efd84b6d448f4132953e47 URL: https://gitlab.winehq.org/wine/wine/-/commit/c8217873383a7f9b09efd84b6d448f… Author: Ivo Ivanov <logos128(a)gmail.com> Date: Tue Aug 9 14:49:07 2022 +0300 dinput/tests: Reduce the available FFB axes to X and Y in test_windows_gaming_input. Windows.Gaming.Input on Windows always uses the X and Y axes for FFB, ignoring what is declared in the HID report descriptor, and the Axes Enable collection. Since we have the correct behavior on Wine, this allows the test to complete on both platforms without issues. --- dlls/dinput/tests/force_feedback.c | 22 ++++++++++++++-------- 1 file changed, 14 insertions(+), 8 deletions(-) diff --git a/dlls/dinput/tests/force_feedback.c b/dlls/dinput/tests/force_feedback.c index d743f959f55..b841e4bb8c7 100644 --- a/dlls/dinput/tests/force_feedback.c +++ b/dlls/dinput/tests/force_feedback.c @@ -4967,19 +4967,18 @@ static void test_windows_gaming_input(void) COLLECTION(1, Logical), USAGE(4, (HID_USAGE_PAGE_GENERIC << 16)|HID_USAGE_GENERIC_X), USAGE(4, (HID_USAGE_PAGE_GENERIC << 16)|HID_USAGE_GENERIC_Y), - USAGE(4, (HID_USAGE_PAGE_GENERIC << 16)|HID_USAGE_GENERIC_Z), LOGICAL_MINIMUM(1, 0), LOGICAL_MAXIMUM(1, 1), PHYSICAL_MINIMUM(1, 0), PHYSICAL_MAXIMUM(1, 1), REPORT_SIZE(1, 1), - REPORT_COUNT(1, 3), + REPORT_COUNT(1, 2), OUTPUT(1, Data|Var|Abs), END_COLLECTION, USAGE(1, PID_USAGE_DIRECTION_ENABLE), REPORT_COUNT(1, 1), OUTPUT(1, Data|Var|Abs), - REPORT_COUNT(1, 4), + REPORT_COUNT(1, 5), OUTPUT(1, Cnst|Var|Abs), USAGE(1, PID_USAGE_DURATION), @@ -5566,7 +5565,7 @@ static void test_windows_gaming_input(void) .code = IOCTL_HID_WRITE_REPORT, .report_id = 3, .report_len = 18, - .report_buf = {3,0x01,0x02,0x08,0x78,0x00,0x00,0x00,0x00,0x00,0x0a,0x00,0xff,0xff,0x4e,0x01,0x00,0x00}, + .report_buf = {3,0x01,0x02,0x04,0x78,0x00,0x00,0x00,0x00,0x00,0x0a,0x00,0xff,0xff,0x4e,0x01,0x00,0x00}, }, }; struct hid_expect expect_create_condition[] = @@ -5597,7 +5596,7 @@ static void test_windows_gaming_input(void) .code = IOCTL_HID_WRITE_REPORT, .report_id = 3, .report_len = 18, - .report_buf = {3,0x01,0x03,0x08,0xff,0xff,0x00,0x00,0x00,0x00,0x00,0x00,0xff,0xff,0x99,0x00,0x00,0x00}, + .report_buf = {3,0x01,0x03,0x04,0xff,0xff,0x00,0x00,0x00,0x00,0x00,0x00,0xff,0xff,0x99,0x00,0x00,0x00}, }, }; struct hid_expect expect_create_constant[] = @@ -5635,7 +5634,7 @@ static void test_windows_gaming_input(void) .code = IOCTL_HID_WRITE_REPORT, .report_id = 3, .report_len = 18, - .report_buf = {3,0x01,0x04,0x08,0x5a,0x00,0x00,0x00,0x00,0x00,0x0a,0x00,0xff,0x7f,0x4e,0x01,0x00,0x00}, + .report_buf = {3,0x01,0x04,0x04,0x5a,0x00,0x00,0x00,0x00,0x00,0x0a,0x00,0xff,0x7f,0x4e,0x01,0x00,0x00}, }, }; struct hid_expect expect_create_ramp[] = @@ -5673,7 +5672,7 @@ static void test_windows_gaming_input(void) .code = IOCTL_HID_WRITE_REPORT, .report_id = 3, .report_len = 18, - .report_buf = {3,0x01,0x05,0x08,0x5a,0x00,0x00,0x00,0x00,0x00,0x0a,0x00,0xff,0xff,0x4e,0x01,0x00,0x00}, + .report_buf = {3,0x01,0x05,0x04,0x5a,0x00,0x00,0x00,0x00,0x00,0x0a,0x00,0xff,0xff,0x4e,0x01,0x00,0x00}, }, }; struct hid_expect expect_effect_start = @@ -5864,7 +5863,7 @@ static void test_windows_gaming_input(void) supported_axes = 0xdeadbeef; hr = IForceFeedbackMotor_get_SupportedAxes( motor, &supported_axes ); ok( hr == S_OK, "get_SupportedAxes returned %#lx\n", hr ); - axes = ForceFeedbackEffectAxes_X | ForceFeedbackEffectAxes_Y | ForceFeedbackEffectAxes_Z; + axes = ForceFeedbackEffectAxes_X | ForceFeedbackEffectAxes_Y; ok( supported_axes == axes || broken( supported_axes == ForceFeedbackEffectAxes_X ), "got axes %#x\n", supported_axes ); @@ -6170,6 +6169,13 @@ static void test_windows_gaming_input(void) ok( hr == S_OK, "SetParametersWithEnvelope returned %#lx\n", hr ); IPeriodicForceEffect_Release( periodic_effect ); + /* Windows.Gaming.Input always uses the X and Y directions on Windows, + * ignoring what is declared in the Axes Enable collection at the + * Set Effects report, or even the existence of the axes in the HID + * report. It ignores the Z direction, at least on HID PID devices. + * DirectInput works properly in such cases on Windows. + */ + set_hid_expect( file, expect_create_periodic, sizeof(expect_create_periodic) ); hr = IForceFeedbackMotor_LoadEffectAsync( motor, effect, &result_async ); ok( hr == S_OK, "LoadEffectAsync returned %#lx\n", hr );

1 0

Ivo Ivanov : windows.gaming.input: Assume that joysticks with single FFB axis are racing wheels.
by Alexandre Julliard Nov. 25, 2022

Nov. 25, 2022

Module: wine Branch: master Commit: 496f7c61348825d735e10878d04c190cc92b8729 URL: https://gitlab.winehq.org/wine/wine/-/commit/496f7c61348825d735e10878d04c19… Author: Ivo Ivanov <logos128(a)gmail.com> Date: Sun Jul 17 18:02:41 2022 +0300 windows.gaming.input: Assume that joysticks with single FFB axis are racing wheels. The HID PID steering wheels always declare one force feedback axis, while the joysticks always have two or more. So it is safe to assume that joysticks with single FFB axis are racing wheels. Fixes FH5 not having force feedback with a Simucube 2 steering wheel, when using the hidraw backend. --- dlls/windows.gaming.input/provider.c | 16 +++++++++++++++- 1 file changed, 15 insertions(+), 1 deletion(-) diff --git a/dlls/windows.gaming.input/provider.c b/dlls/windows.gaming.input/provider.c index 39229a35c7d..6908d733a53 100644 --- a/dlls/windows.gaming.input/provider.c +++ b/dlls/windows.gaming.input/provider.c @@ -141,6 +141,13 @@ static HRESULT WINAPI wine_provider_GetTrustLevel( IWineGameControllerProvider * return E_NOTIMPL; } +static BOOL CALLBACK count_ffb_axes( const DIDEVICEOBJECTINSTANCEW *obj, void *args ) +{ + DWORD *count = args; + if (obj->dwType & DIDFT_FFACTUATOR) (*count)++; + return DIENUM_CONTINUE; +} + static HRESULT WINAPI wine_provider_get_Type( IWineGameControllerProvider *iface, WineGameControllerType *value ) { struct provider *impl = impl_from_IWineGameControllerProvider( iface ); @@ -155,7 +162,14 @@ static HRESULT WINAPI wine_provider_get_Type( IWineGameControllerProvider *iface { case DI8DEVTYPE_DRIVING: *value = WineGameControllerType_RacingWheel; break; case DI8DEVTYPE_GAMEPAD: *value = WineGameControllerType_Gamepad; break; - default: *value = WineGameControllerType_Joystick; break; + default: + { + DWORD count = 0; + hr = IDirectInputDevice8_EnumObjects( impl->dinput_device, count_ffb_axes, &count, DIDFT_AXIS ); + if (SUCCEEDED(hr) && count == 1) *value = WineGameControllerType_RacingWheel; + else *value = WineGameControllerType_Joystick; + break; + } } return S_OK;

1 0

Alexandre Julliard : bcrypt: Use the Unix call helpers.
by Alexandre Julliard Nov. 25, 2022

Nov. 25, 2022

Module: wine Branch: master Commit: a4f8d28de77148f7e8a5ad0fe9376afaabb44d49 URL: https://gitlab.winehq.org/wine/wine/-/commit/a4f8d28de77148f7e8a5ad0fe9376a… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Nov 25 10:10:34 2022 +0100 bcrypt: Use the Unix call helpers. --- dlls/bcrypt/bcrypt_main.c | 15 ++++++--------- 1 file changed, 6 insertions(+), 9 deletions(-) diff --git a/dlls/bcrypt/bcrypt_main.c b/dlls/bcrypt/bcrypt_main.c index 266c30fa426..6f6d713c6ac 100644 --- a/dlls/bcrypt/bcrypt_main.c +++ b/dlls/bcrypt/bcrypt_main.c @@ -34,9 +34,7 @@ WINE_DEFAULT_DEBUG_CHANNEL(bcrypt); -static unixlib_handle_t bcrypt_handle; - -#define UNIX_CALL( func, params ) __wine_unix_call( bcrypt_handle, unix_ ## func, params ) +#define UNIX_CALL( func, params ) WINE_UNIX_CALL( unix_ ## func, params ) NTSTATUS WINAPI BCryptAddContextFunction( ULONG table, const WCHAR *ctx, ULONG iface, const WCHAR *func, ULONG pos ) @@ -1107,7 +1105,7 @@ static NTSTATUS key_asymmetric_create( enum alg_id alg_id, ULONG bitlen, struct { struct key *key; - if (!bcrypt_handle) + if (!__wine_unixlib_handle) { ERR( "no encryption support\n" ); return STATUS_NOT_IMPLEMENTED; @@ -1786,7 +1784,7 @@ NTSTATUS WINAPI BCryptGenerateSymmetricKey( BCRYPT_ALG_HANDLE handle, BCRYPT_KEY TRACE( "%p, %p, %p, %lu, %p, %lu, %#lx\n", handle, ret_handle, object, object_len, secret, secret_len, flags ); if (object) FIXME( "ignoring object buffer\n" ); - if (!bcrypt_handle) + if (!__wine_unixlib_handle) { ERR( "no encryption support\n" ); return STATUS_NOT_IMPLEMENTED; @@ -2342,15 +2340,14 @@ BOOL WINAPI DllMain( HINSTANCE hinst, DWORD reason, LPVOID reserved ) { case DLL_PROCESS_ATTACH: DisableThreadLibraryCalls( hinst ); - if (!NtQueryVirtualMemory( GetCurrentProcess(), hinst, MemoryWineUnixFuncs, - &bcrypt_handle, sizeof(bcrypt_handle), NULL )) + if (!__wine_init_unix_call()) { - if (UNIX_CALL( process_attach, NULL)) bcrypt_handle = 0; + if (UNIX_CALL( process_attach, NULL)) __wine_unixlib_handle = 0; } break; case DLL_PROCESS_DETACH: if (reserved) break; - if (bcrypt_handle) UNIX_CALL( process_detach, NULL ); + if (__wine_unixlib_handle) UNIX_CALL( process_detach, NULL ); break; } return TRUE;

1 0

Alexandre Julliard : odbc32: Use the Unix call helpers.
by Alexandre Julliard Nov. 25, 2022

Nov. 25, 2022

Module: wine Branch: master Commit: 7f808e564d712c111764ec0a43d8e74395c6e3c7 URL: https://gitlab.winehq.org/wine/wine/-/commit/7f808e564d712c111764ec0a43d8e7… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Nov 25 10:13:24 2022 +0100 odbc32: Use the Unix call helpers. --- dlls/odbc32/proxyodbc.c | 16 ++++++---------- 1 file changed, 6 insertions(+), 10 deletions(-) diff --git a/dlls/odbc32/proxyodbc.c b/dlls/odbc32/proxyodbc.c index 714819993ee..679f3d7380e 100644 --- a/dlls/odbc32/proxyodbc.c +++ b/dlls/odbc32/proxyodbc.c @@ -45,9 +45,7 @@ WINE_DEFAULT_DEBUG_CHANNEL(odbc); WINE_DECLARE_DEBUG_CHANNEL(winediag); -static unixlib_handle_t odbc_handle; - -#define ODBC_CALL( func, params ) __wine_unix_call( odbc_handle, unix_ ## func, params ) +#define ODBC_CALL( func, params ) WINE_UNIX_CALL( unix_ ## func, params ) /*********************************************************************** * ODBC_ReplicateODBCInstToRegistry @@ -2641,18 +2639,16 @@ BOOL WINAPI DllMain(HINSTANCE hinstDLL, DWORD reason, LPVOID reserved) switch (reason) { case DLL_PROCESS_ATTACH: - DisableThreadLibraryCalls(hinstDLL); - if (!NtQueryVirtualMemory( GetCurrentProcess(), hinstDLL, MemoryWineUnixFuncs, - &odbc_handle, sizeof(odbc_handle), NULL ) && - !__wine_unix_call( odbc_handle, process_attach, NULL)) + DisableThreadLibraryCalls(hinstDLL); + if (!__wine_init_unix_call() && !WINE_UNIX_CALL( process_attach, NULL )) { ODBC_ReplicateToRegistry(); } - break; + break; case DLL_PROCESS_DETACH: - if (reserved) break; - __wine_unix_call( odbc_handle, process_detach, NULL ); + if (reserved) break; + WINE_UNIX_CALL( process_detach, NULL ); } return TRUE;

1 0