March 2024 - Wine-commits - list.winehq.org

Zebediah Figura : wined3d: Avoid leaking string buffers in shader_glsl_load_bindless_samplers().
by Alexandre Julliard 21 Mar '24

21 Mar '24

Module: wine Branch: master Commit: c38df594f5c35ef28986cfd1340ecfd3eb5969b0 URL: https://gitlab.winehq.org/wine/wine/-/commit/c38df594f5c35ef28986cfd1340ecf… Author: Zebediah Figura <zfigura(a)codeweavers.com> Date: Mon Mar 18 18:10:41 2024 -0500 wined3d: Avoid leaking string buffers in shader_glsl_load_bindless_samplers(). Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=56445 --- dlls/wined3d/glsl_shader.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/dlls/wined3d/glsl_shader.c b/dlls/wined3d/glsl_shader.c index 9ba9b2e1d4c..42cdb585c50 100644 --- a/dlls/wined3d/glsl_shader.c +++ b/dlls/wined3d/glsl_shader.c @@ -10760,17 +10760,19 @@ static void shader_glsl_update_graphics_program(struct shader_glsl_priv *priv, static void shader_glsl_load_bindless_samplers(struct shader_glsl_priv *priv, struct wined3d_context_gl *context_gl, const struct wined3d_state *state, enum wined3d_shader_type shader_type) { - struct wined3d_string_buffer *sampler_name = string_buffer_get(&priv->string_buffers); const struct wined3d_device_gl *device_gl = wined3d_device_gl(context_gl->c.device); const struct glsl_context_data *ctx_data = context_gl->c.shader_backend_data; const struct wined3d_shader *shader = state->shader[shader_type]; const struct wined3d_gl_info *gl_info = context_gl->gl_info; const char *prefix = shader_glsl_get_prefix(shader_type); + struct wined3d_string_buffer *sampler_name; /* Note that we don't use bindless samplers for FFP shaders. */ if (!shader) return; + sampler_name = string_buffer_get(&priv->string_buffers); + for (unsigned int i = 0; i < shader->reg_maps.sampler_map.count; ++i) { const struct wined3d_shader_sampler_map_entry *entry = &shader->reg_maps.sampler_map.entries[i];

1 0

Pavel Ondračka : d3d9/tests: Define enums outside of struct.
by Alexandre Julliard 21 Mar '24

21 Mar '24

Module: wine Branch: master Commit: d0e37372cf393184ad9f25697a40c16ab624b98f URL: https://gitlab.winehq.org/wine/wine/-/commit/d0e37372cf393184ad9f25697a40c1… Author: Pavel Ondračka <pavel.ondracka(a)gmail.com> Date: Fri Mar 15 17:54:16 2024 +0100 d3d9/tests: Define enums outside of struct. Co-authored-by: David Heidelberg <david(a)ixit.cz> Signed-off-by: David Heidelberg <david(a)ixit.cz> --- dlls/d3d9/tests/visual.c | 30 ++++++++++++++++-------------- 1 file changed, 16 insertions(+), 14 deletions(-) diff --git a/dlls/d3d9/tests/visual.c b/dlls/d3d9/tests/visual.c index d05649ab92e..36af662f19b 100644 --- a/dlls/d3d9/tests/visual.c +++ b/dlls/d3d9/tests/visual.c @@ -1667,16 +1667,17 @@ static void color_fill_test(void) {D3DPOOL_MANAGED, 0, D3DERR_INVALIDCALL}, {D3DPOOL_SCRATCH, 0, D3DERR_INVALIDCALL}, }; + enum format_flags + { + CHECK_FILL_VALUE = 0x1, + BLOCKS = 0x2, + FLOAT_VALUES = 0x4, + }; static const struct { D3DFORMAT format; const char *name; - enum - { - CHECK_FILL_VALUE = 0x1, - BLOCKS = 0x2, - FLOAT_VALUES = 0x4, - } flags; + enum format_flags flags; unsigned int fill_i[4]; float fill_f[4]; } @@ -23249,16 +23250,17 @@ static void test_texture_blending(void) DWORD value; }; + enum texture_stage_texture + { + TEXTURE_INVALID, + TEXTURE_NONE, + TEXTURE_BUMPMAP, + TEXTURE_RED, + }; + struct texture_stage { - enum - { - TEXTURE_INVALID, - TEXTURE_NONE, - TEXTURE_BUMPMAP, - TEXTURE_RED, - } - texture; + enum texture_stage_texture texture; struct texture_stage_state state[20]; };

1 0

David Heidelberg : d3d9/tests: Replace LPDWORD cast with float_to_int function.
by Alexandre Julliard 21 Mar '24

21 Mar '24

Module: wine Branch: master Commit: a63af560dd108ca603505fd030229ba2fd83fba1 URL: https://gitlab.winehq.org/wine/wine/-/commit/a63af560dd108ca603505fd030229b… Author: David Heidelberg <david(a)ixit.cz> Date: Tue Mar 19 15:53:37 2024 +0100 d3d9/tests: Replace LPDWORD cast with float_to_int function. Borrowed from dlls/wined3d/wined3d_private.h Signed-off-by: David Heidelberg <david(a)ixit.cz> --- dlls/d3d9/tests/visual.c | 45 +++++++++++++++++++++++++++++---------------- 1 file changed, 29 insertions(+), 16 deletions(-) diff --git a/dlls/d3d9/tests/visual.c b/dlls/d3d9/tests/visual.c index 36af662f19b..20df3d29b1b 100644 --- a/dlls/d3d9/tests/visual.c +++ b/dlls/d3d9/tests/visual.c @@ -29,6 +29,7 @@ */ #include <stdbool.h> +#include <stdint.h> #include <limits.h> #include <math.h> @@ -107,6 +108,18 @@ static BOOL compare_vec4(const struct vec4 *vec, float x, float y, float z, floa && compare_float(vec->w, w, ulps); } +static uint32_t float_to_int(float f) +{ + union + { + uint32_t u; + float f; + } u; + + u.f = f; + return u.u; +} + static BOOL adapter_is_warp(const D3DADAPTER_IDENTIFIER9 *identifier) { return !strcmp(identifier->Driver, "d3d10warp.dll"); @@ -3511,10 +3524,10 @@ static void texbem_test(void) generate_bumpmap_textures(device); - IDirect3DDevice9_SetTextureStageState(device, 1, D3DTSS_BUMPENVMAT00, *(LPDWORD)&bumpenvmat[0]); - IDirect3DDevice9_SetTextureStageState(device, 1, D3DTSS_BUMPENVMAT01, *(LPDWORD)&bumpenvmat[1]); - IDirect3DDevice9_SetTextureStageState(device, 1, D3DTSS_BUMPENVMAT10, *(LPDWORD)&bumpenvmat[2]); - hr = IDirect3DDevice9_SetTextureStageState(device, 1, D3DTSS_BUMPENVMAT11, *(LPDWORD)&bumpenvmat[3]); + IDirect3DDevice9_SetTextureStageState(device, 1, D3DTSS_BUMPENVMAT00, float_to_int(bumpenvmat[0])); + IDirect3DDevice9_SetTextureStageState(device, 1, D3DTSS_BUMPENVMAT01, float_to_int(bumpenvmat[1])); + IDirect3DDevice9_SetTextureStageState(device, 1, D3DTSS_BUMPENVMAT10, float_to_int(bumpenvmat[2])); + hr = IDirect3DDevice9_SetTextureStageState(device, 1, D3DTSS_BUMPENVMAT11, float_to_int(bumpenvmat[3])); ok(hr == S_OK, "Got hr %#lx.\n", hr); hr = IDirect3DDevice9_SetVertexShader(device, NULL); @@ -3665,24 +3678,24 @@ static void texbem_test(void) bumpenvmat[0] =-1.0; bumpenvmat[2] = 2.0; bumpenvmat[1] = 0.0; bumpenvmat[3] = 0.0; - hr = IDirect3DDevice9_SetTextureStageState(device, 1, D3DTSS_BUMPENVMAT00, *(LPDWORD)&bumpenvmat[0]); + hr = IDirect3DDevice9_SetTextureStageState(device, 1, D3DTSS_BUMPENVMAT00, float_to_int(bumpenvmat[0])); ok(hr == S_OK, "Got hr %#lx.\n", hr); - hr = IDirect3DDevice9_SetTextureStageState(device, 1, D3DTSS_BUMPENVMAT01, *(LPDWORD)&bumpenvmat[1]); + hr = IDirect3DDevice9_SetTextureStageState(device, 1, D3DTSS_BUMPENVMAT01, float_to_int(bumpenvmat[1])); ok(hr == S_OK, "Got hr %#lx.\n", hr); - hr = IDirect3DDevice9_SetTextureStageState(device, 1, D3DTSS_BUMPENVMAT10, *(LPDWORD)&bumpenvmat[2]); + hr = IDirect3DDevice9_SetTextureStageState(device, 1, D3DTSS_BUMPENVMAT10, float_to_int(bumpenvmat[2])); ok(hr == S_OK, "Got hr %#lx.\n", hr); - hr = IDirect3DDevice9_SetTextureStageState(device, 1, D3DTSS_BUMPENVMAT11, *(LPDWORD)&bumpenvmat[3]); + hr = IDirect3DDevice9_SetTextureStageState(device, 1, D3DTSS_BUMPENVMAT11, float_to_int(bumpenvmat[3])); ok(hr == S_OK, "Got hr %#lx.\n", hr); bumpenvmat[0] = 1.5; bumpenvmat[2] = 0.0; bumpenvmat[1] = 0.0; bumpenvmat[3] = 0.5; - hr = IDirect3DDevice9_SetTextureStageState(device, 3, D3DTSS_BUMPENVMAT00, *(LPDWORD)&bumpenvmat[0]); + hr = IDirect3DDevice9_SetTextureStageState(device, 3, D3DTSS_BUMPENVMAT00, float_to_int(bumpenvmat[0])); ok(hr == S_OK, "Got hr %#lx.\n", hr); - hr = IDirect3DDevice9_SetTextureStageState(device, 3, D3DTSS_BUMPENVMAT01, *(LPDWORD)&bumpenvmat[1]); + hr = IDirect3DDevice9_SetTextureStageState(device, 3, D3DTSS_BUMPENVMAT01, float_to_int(bumpenvmat[1])); ok(hr == S_OK, "Got hr %#lx.\n", hr); - hr = IDirect3DDevice9_SetTextureStageState(device, 3, D3DTSS_BUMPENVMAT10, *(LPDWORD)&bumpenvmat[2]); + hr = IDirect3DDevice9_SetTextureStageState(device, 3, D3DTSS_BUMPENVMAT10, float_to_int(bumpenvmat[2])); ok(hr == S_OK, "Got hr %#lx.\n", hr); - hr = IDirect3DDevice9_SetTextureStageState(device, 3, D3DTSS_BUMPENVMAT11, *(LPDWORD)&bumpenvmat[3]); + hr = IDirect3DDevice9_SetTextureStageState(device, 3, D3DTSS_BUMPENVMAT11, float_to_int(bumpenvmat[3])); ok(hr == S_OK, "Got hr %#lx.\n", hr); hr = IDirect3DDevice9_SetSamplerState(device, 0, D3DSAMP_ADDRESSU, D3DTADDRESS_CLAMP); @@ -11028,13 +11041,13 @@ static void fixed_function_bumpmap_test(void) /* Generate the textures */ generate_bumpmap_textures(device); - hr = IDirect3DDevice9_SetTextureStageState(device, 0, D3DTSS_BUMPENVMAT00, *(LPDWORD)&bumpenvmat[0]); + hr = IDirect3DDevice9_SetTextureStageState(device, 0, D3DTSS_BUMPENVMAT00, float_to_int(bumpenvmat[0])); ok(hr == S_OK, "Got hr %#lx.\n", hr); - hr = IDirect3DDevice9_SetTextureStageState(device, 0, D3DTSS_BUMPENVMAT01, *(LPDWORD)&bumpenvmat[1]); + hr = IDirect3DDevice9_SetTextureStageState(device, 0, D3DTSS_BUMPENVMAT01, float_to_int(bumpenvmat[1])); ok(hr == S_OK, "Got hr %#lx.\n", hr); - hr = IDirect3DDevice9_SetTextureStageState(device, 0, D3DTSS_BUMPENVMAT10, *(LPDWORD)&bumpenvmat[2]); + hr = IDirect3DDevice9_SetTextureStageState(device, 0, D3DTSS_BUMPENVMAT10, float_to_int(bumpenvmat[2])); ok(hr == S_OK, "Got hr %#lx.\n", hr); - hr = IDirect3DDevice9_SetTextureStageState(device, 0, D3DTSS_BUMPENVMAT11, *(LPDWORD)&bumpenvmat[3]); + hr = IDirect3DDevice9_SetTextureStageState(device, 0, D3DTSS_BUMPENVMAT11, float_to_int(bumpenvmat[3])); ok(hr == S_OK, "Got hr %#lx.\n", hr); hr = IDirect3DDevice9_SetTextureStageState(device, 0, D3DTSS_COLOROP, D3DTOP_BUMPENVMAP);

1 0

David Heidelberg : mailmap: Add a mailmap entry for myself with the proper name.
by Alexandre Julliard 21 Mar '24

21 Mar '24

Module: wine Branch: master Commit: 5d8e8804d485357156f346ceb840d2c42308cfae URL: https://gitlab.winehq.org/wine/wine/-/commit/5d8e8804d485357156f346ceb840d2… Author: David Heidelberg <david(a)ixit.cz> Date: Fri Mar 15 18:05:54 2024 +0100 mailmap: Add a mailmap entry for myself with the proper name. Signed-off-by: David Heidelberg <david(a)ixit.cz> --- .mailmap | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.mailmap b/.mailmap index 4bd0cee2b2c..7cf13e67174 100644 --- a/.mailmap +++ b/.mailmap @@ -18,7 +18,7 @@ Charles Davis <cdavis(a)codeweavers.com> Charles Davis <cdavis5x(a)gmail.com> Christopher Gautier <krys(a)via.ecp.fr> David A. Cuthbert <dacut(a)ece.cmu.edu> -David Heidelberger <david(a)ixit.cz> +David Heidelberg <david(a)ixit.cz> Dennis Björklund <db(a)zigo.dhs.org> Dennis Björklund <dennisb(a)cs.chalmers.se> Dimitrie O. Paun <dimi(a)bigfoot.com>

1 0

Rémi Bernon : winex11: Accept key and mouse events with QS_RAWINPUT.
by Alexandre Julliard 21 Mar '24

21 Mar '24

Module: wine Branch: master Commit: f1dca8914ca8a93aa28a6856b5ebfc540aea8ca7 URL: https://gitlab.winehq.org/wine/wine/-/commit/f1dca8914ca8a93aa28a6856b5ebfc… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Mon Mar 18 11:41:32 2024 +0100 winex11: Accept key and mouse events with QS_RAWINPUT. --- dlls/winex11.drv/event.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/dlls/winex11.drv/event.c b/dlls/winex11.drv/event.c index 8b02361aaff..379b27b3f70 100644 --- a/dlls/winex11.drv/event.c +++ b/dlls/winex11.drv/event.c @@ -229,14 +229,14 @@ static Bool filter_event( Display *display, XEvent *event, char *arg ) case KeyRelease: case KeymapNotify: case MappingNotify: - return (mask & (QS_KEY|QS_HOTKEY)) != 0; + return (mask & (QS_KEY | QS_HOTKEY | QS_RAWINPUT)) != 0; case ButtonPress: case ButtonRelease: - return (mask & QS_MOUSEBUTTON) != 0; + return (mask & (QS_MOUSEBUTTON | QS_RAWINPUT)) != 0; case MotionNotify: case EnterNotify: case LeaveNotify: - return (mask & QS_MOUSEMOVE) != 0; + return (mask & (QS_MOUSEMOVE | QS_RAWINPUT)) != 0; case Expose: return (mask & QS_PAINT) != 0; case FocusIn:

1 0

Rémi Bernon : server: Send WM_WINE_CLIPCURSOR message only when necessary.
by Alexandre Julliard 21 Mar '24

21 Mar '24

Module: wine Branch: master Commit: 1604d8a2025bc56d9a5a7619294119fa2fa2015e URL: https://gitlab.winehq.org/wine/wine/-/commit/1604d8a2025bc56d9a5a7619294119… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Thu Mar 14 16:22:21 2024 +0100 server: Send WM_WINE_CLIPCURSOR message only when necessary. --- server/queue.c | 9 +++++++-- server/user.h | 1 + 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/server/queue.c b/server/queue.c index 0e8653bedf0..5eb6c1edafb 100644 --- a/server/queue.c +++ b/server/queue.c @@ -527,6 +527,7 @@ static void get_message_defaults( struct msg_queue *queue, int *x, int *y, unsig void set_clip_rectangle( struct desktop *desktop, const rectangle_t *rect, unsigned int flags, int reset ) { rectangle_t top_rect; + unsigned int old_flags; int x, y; get_top_window_rectangle( desktop, &top_rect ); @@ -542,6 +543,9 @@ void set_clip_rectangle( struct desktop *desktop, const rectangle_t *rect, unsig } else desktop->cursor.clip = top_rect; + old_flags = desktop->cursor.clip_flags; + desktop->cursor.clip_flags = flags; + /* warp the mouse to be inside the clip rect */ x = max( min( desktop->cursor.x, desktop->cursor.clip.right - 1 ), desktop->cursor.clip.left ); y = max( min( desktop->cursor.y, desktop->cursor.clip.bottom - 1 ), desktop->cursor.clip.top ); @@ -550,8 +554,9 @@ void set_clip_rectangle( struct desktop *desktop, const rectangle_t *rect, unsig /* request clip cursor rectangle reset to the desktop thread */ if (reset) post_desktop_message( desktop, WM_WINE_CLIPCURSOR, flags, FALSE ); - /* notify foreground thread, of reset, or to apply new cursor clipping rect */ - queue_cursor_message( desktop, 0, WM_WINE_CLIPCURSOR, flags, reset ); + /* notify foreground thread of reset, clipped, or released cursor rect */ + if (reset || flags != SET_CURSOR_NOCLIP || old_flags != SET_CURSOR_NOCLIP) + queue_cursor_message( desktop, 0, WM_WINE_CLIPCURSOR, flags, reset ); } /* change the foreground input and reset the cursor clip rect */ diff --git a/server/user.h b/server/user.h index 309a0966104..d805a179d16 100644 --- a/server/user.h +++ b/server/user.h @@ -59,6 +59,7 @@ struct global_cursor int x; /* cursor position */ int y; rectangle_t clip; /* cursor clip rectangle */ + unsigned int clip_flags; /* last cursor clip flags */ unsigned int last_change; /* time of last position change */ user_handle_t win; /* window that contains the cursor */ };

1 0

Rémi Bernon : server: Send WM_WINE_SETCURSOR message only when necessary.
by Alexandre Julliard 21 Mar '24

21 Mar '24

Module: wine Branch: master Commit: 994021da996f06f3fd65670b19c301ed77441c5c URL: https://gitlab.winehq.org/wine/wine/-/commit/994021da996f06f3fd65670b19c301… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Thu Mar 14 18:09:17 2024 +0100 server: Send WM_WINE_SETCURSOR message only when necessary. --- server/queue.c | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/server/queue.c b/server/queue.c index 5eb6c1edafb..5ae54e743a4 100644 --- a/server/queue.c +++ b/server/queue.c @@ -481,11 +481,11 @@ static int update_desktop_cursor_pos( struct desktop *desktop, user_handle_t win return updated; } -static void update_desktop_cursor_handle( struct desktop *desktop, struct thread_input *input ) +static void update_desktop_cursor_handle( struct desktop *desktop, struct thread_input *input, user_handle_t handle ) { if (input == get_desktop_cursor_thread_input( desktop )) { - user_handle_t handle = input->cursor_count < 0 ? 0 : input->cursor, win = desktop->cursor.win; + user_handle_t win = desktop->cursor.win; /* when clipping send the message to the foreground window as well, as some driver have an artificial overlay window */ if (is_cursor_clipped( desktop )) queue_cursor_message( desktop, 0, WM_WINE_SETCURSOR, win, handle ); queue_cursor_message( desktop, win, WM_WINE_SETCURSOR, win, handle ); @@ -3636,12 +3636,14 @@ DECL_HANDLER(get_last_input_time) DECL_HANDLER(set_cursor) { struct msg_queue *queue = get_current_queue(); + user_handle_t prev_cursor, new_cursor; struct thread_input *input; struct desktop *desktop; if (!queue) return; input = queue->input; desktop = input->desktop; + prev_cursor = input->cursor_count < 0 ? 0 : input->cursor; reply->prev_handle = input->cursor; reply->prev_count = input->cursor_count; @@ -3666,8 +3668,8 @@ DECL_HANDLER(set_cursor) if (req->flags & SET_CURSOR_CLIP) set_clip_rectangle( desktop, &req->clip, req->flags, 0 ); if (req->flags & SET_CURSOR_NOCLIP) set_clip_rectangle( desktop, NULL, SET_CURSOR_NOCLIP, 0 ); - if (req->flags & (SET_CURSOR_HANDLE | SET_CURSOR_COUNT)) - update_desktop_cursor_handle( desktop, input ); + new_cursor = input->cursor_count < 0 ? 0 : input->cursor; + if (prev_cursor != new_cursor) update_desktop_cursor_handle( desktop, input, new_cursor ); reply->new_x = desktop->cursor.x; reply->new_y = desktop->cursor.y;

1 0

Rémi Bernon : win32u: Use a structure to pass peek_message arguments.
by Alexandre Julliard 21 Mar '24

21 Mar '24

Module: wine Branch: master Commit: 8405fe85d4525ae8ae1161d4318a008d397c0c00 URL: https://gitlab.winehq.org/wine/wine/-/commit/8405fe85d4525ae8ae1161d4318a00… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Fri Mar 15 13:51:18 2024 +0100 win32u: Use a structure to pass peek_message arguments. --- dlls/win32u/message.c | 49 +++++++++++++++++++++++++++++++++++++------------ 1 file changed, 37 insertions(+), 12 deletions(-) diff --git a/dlls/win32u/message.c b/dlls/win32u/message.c index 44a27763bdc..25b8c3c48f0 100644 --- a/dlls/win32u/message.c +++ b/dlls/win32u/message.c @@ -2695,6 +2695,15 @@ static BOOL process_hardware_message( MSG *msg, UINT hw_id, const struct hardwar return ret; } +struct peek_message_filter +{ + HWND hwnd; + UINT first; + UINT last; + UINT mask; + UINT flags; +}; + /*********************************************************************** * peek_message * @@ -2702,9 +2711,11 @@ static BOOL process_hardware_message( MSG *msg, UINT hw_id, const struct hardwar * available; -1 on error. * All pending sent messages are processed before returning. */ -static int peek_message( MSG *msg, HWND hwnd, UINT first, UINT last, UINT flags, UINT changed_mask ) +static int peek_message( MSG *msg, const struct peek_message_filter *filter ) { LRESULT result; + HWND hwnd = filter->hwnd; + UINT first = filter->first, last = filter->last, flags = filter->flags; struct user_thread_info *thread_info = get_user_thread_info(); INPUT_MESSAGE_SOURCE prev_source = thread_info->client_info.msg_source; struct received_message_info info; @@ -2732,8 +2743,8 @@ static int peek_message( MSG *msg, HWND hwnd, UINT first, UINT last, UINT flags, req->get_first = first; req->get_last = last; req->hw_id = hw_id; - req->wake_mask = changed_mask & (QS_SENDMESSAGE | QS_SMRESULT); - req->changed_mask = changed_mask; + req->wake_mask = filter->mask & (QS_SENDMESSAGE | QS_SMRESULT); + req->changed_mask = filter->mask; wine_server_set_reply( req, buffer, buffer_size ); if (!(res = wine_server_call( req ))) { @@ -2758,8 +2769,8 @@ static int peek_message( MSG *msg, HWND hwnd, UINT first, UINT last, UINT flags, free( buffer ); if (res == STATUS_PENDING) { - thread_info->wake_mask = changed_mask & (QS_SENDMESSAGE | QS_SMRESULT); - thread_info->changed_mask = changed_mask; + thread_info->wake_mask = filter->mask & (QS_SENDMESSAGE | QS_SMRESULT); + thread_info->changed_mask = filter->mask; return 0; } if (res != STATUS_BUFFER_OVERFLOW) @@ -2904,8 +2915,17 @@ static int peek_message( MSG *msg, HWND hwnd, UINT first, UINT last, UINT flags, } } else - peek_message( msg, info.msg.hwnd, info.msg.message, - info.msg.message, flags | PM_REMOVE, changed_mask ); + { + struct peek_message_filter new_filter = + { + .hwnd = info.msg.hwnd, + .flags = flags | PM_REMOVE, + .first = info.msg.message, + .last = info.msg.message, + .mask = filter->mask, + }; + peek_message( msg, &new_filter ); + } continue; } if (info.msg.message >= WM_DDE_FIRST && info.msg.message <= WM_DDE_LAST) @@ -2956,7 +2976,7 @@ static int peek_message( MSG *msg, HWND hwnd, UINT first, UINT last, UINT flags, info.msg.wParam, info.msg.lParam ); /* if some PM_QS* flags were specified, only handle sent messages from now on */ - if (HIWORD(flags) && !changed_mask) flags = PM_QS_SENDMESSAGE | LOWORD(flags); + if (HIWORD(flags) && !filter->mask) flags = PM_QS_SENDMESSAGE | LOWORD(flags); } } @@ -2967,8 +2987,9 @@ static int peek_message( MSG *msg, HWND hwnd, UINT first, UINT last, UINT flags, */ static void process_sent_messages(void) { + struct peek_message_filter filter = {.flags = PM_REMOVE | PM_QS_SENDMESSAGE}; MSG msg; - peek_message( &msg, 0, 0, 0, PM_REMOVE | PM_QS_SENDMESSAGE, 0 ); + peek_message( &msg, &filter ); } /*********************************************************************** @@ -3191,13 +3212,14 @@ BOOL WINAPI NtUserWaitMessage(void) */ BOOL WINAPI NtUserPeekMessage( MSG *msg_out, HWND hwnd, UINT first, UINT last, UINT flags ) { + struct peek_message_filter filter = {.hwnd = hwnd, .first = first, .last = last, .flags = flags}; MSG msg; int ret; user_check_not_lock(); check_for_driver_events( 0 ); - ret = peek_message( &msg, hwnd, first, last, flags, 0 ); + ret = peek_message( &msg, &filter ); if (ret < 0) return FALSE; if (!ret) @@ -3205,7 +3227,7 @@ BOOL WINAPI NtUserPeekMessage( MSG *msg_out, HWND hwnd, UINT first, UINT last, U flush_window_surfaces( TRUE ); ret = wait_message( 0, NULL, 0, QS_ALLINPUT, 0 ); /* if we received driver events, check again for a pending message */ - if (ret == WAIT_TIMEOUT || peek_message( &msg, hwnd, first, last, flags, 0 ) <= 0) return FALSE; + if (ret == WAIT_TIMEOUT || peek_message( &msg, &filter ) <= 0) return FALSE; } check_for_driver_events( msg.message ); @@ -3228,6 +3250,7 @@ BOOL WINAPI NtUserPeekMessage( MSG *msg_out, HWND hwnd, UINT first, UINT last, U */ BOOL WINAPI NtUserGetMessage( MSG *msg, HWND hwnd, UINT first, UINT last ) { + struct peek_message_filter filter = {.hwnd = hwnd, .first = first, .last = last}; HANDLE server_queue = get_server_queue_handle(); unsigned int mask = QS_POSTMESSAGE | QS_SENDMESSAGE; /* Always selected */ int ret; @@ -3246,7 +3269,9 @@ BOOL WINAPI NtUserGetMessage( MSG *msg, HWND hwnd, UINT first, UINT last ) } else mask = QS_ALLINPUT; - while (!(ret = peek_message( msg, hwnd, first, last, PM_REMOVE | (mask << 16), mask ))) + filter.mask = mask; + filter.flags = PM_REMOVE | (mask << 16); + while (!(ret = peek_message( msg, &filter ))) { wait_objects( 1, &server_queue, INFINITE, mask & (QS_SENDMESSAGE | QS_SMRESULT), mask, 0 ); }

1 0

Rémi Bernon : server: Check for internal hardware messages before others.
by Alexandre Julliard 21 Mar '24

21 Mar '24

Module: wine Branch: master Commit: a84517418e6fd1882c1c0d60448018d7bc6f3bec URL: https://gitlab.winehq.org/wine/wine/-/commit/a84517418e6fd1882c1c0d60448018… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Fri Mar 15 13:11:09 2024 +0100 server: Check for internal hardware messages before others. --- server/queue.c | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/server/queue.c b/server/queue.c index 5ae54e743a4..c884e9f7f55 100644 --- a/server/queue.c +++ b/server/queue.c @@ -2988,6 +2988,12 @@ DECL_HANDLER(get_message) } if (!queue) return; + + /* check for any hardware internal message */ + if (get_hardware_message( current, req->hw_id, get_win, WM_WINE_FIRST_DRIVER_MSG, + WM_WINE_LAST_DRIVER_MSG, req->flags, reply )) + return; + queue->last_get_msg = current_time; if (!filter) filter = QS_ALLINPUT; @@ -3028,11 +3034,6 @@ DECL_HANDLER(get_message) get_hardware_message( current, req->hw_id, get_win, req->get_first, req->get_last, req->flags, reply )) return; - /* check for any internal driver message */ - if (get_hardware_message( current, req->hw_id, get_win, WM_WINE_FIRST_DRIVER_MSG, - WM_WINE_LAST_DRIVER_MSG, req->flags, reply )) - return; - /* now check for WM_PAINT */ if ((filter & QS_PAINT) && queue->paint_count &&

1 0

Rémi Bernon : server: Process internal messages when checking queue status.
by Alexandre Julliard 21 Mar '24

21 Mar '24

Module: wine Branch: master Commit: b053e924e8e13b3637f2a5a8ffe88d84c2d17075 URL: https://gitlab.winehq.org/wine/wine/-/commit/b053e924e8e13b3637f2a5a8ffe88d… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Fri Mar 15 13:11:09 2024 +0100 server: Process internal messages when checking queue status. --- dlls/dinput/tests/device8.c | 11 +++++++++++ dlls/win32u/input.c | 9 +++++++++ dlls/win32u/message.c | 13 +++---------- dlls/win32u/ntuser_private.h | 11 +++++++++++ include/wine/server_protocol.h | 4 +++- server/protocol.def | 1 + server/queue.c | 6 ++++++ server/request.h | 3 ++- server/trace.c | 1 + 9 files changed, 47 insertions(+), 12 deletions(-) diff --git a/dlls/dinput/tests/device8.c b/dlls/dinput/tests/device8.c index 7fb0ef538e0..6bfc88f01cc 100644 --- a/dlls/dinput/tests/device8.c +++ b/dlls/dinput/tests/device8.c @@ -2038,6 +2038,9 @@ static void test_hid_touch_screen(void) ret = RegisterRawInputDevices( &rawdevice, 1, sizeof(RAWINPUTDEVICE) ); ok( ret, "RegisterRawInputDevices failed, error %lu\n", GetLastError() ); + res = GetQueueStatus( QS_RAWINPUT ); + ok( res == 0, "got res %#lx\n", res ); + bus_send_hid_input( file, &desc, &touch_multiple, sizeof(touch_multiple) ); bus_wait_hid_input( file, &desc, 5000 ); bus_send_hid_input( file, &desc, &touch_release, sizeof(touch_release) ); @@ -2046,6 +2049,10 @@ static void test_hid_touch_screen(void) res = MsgWaitForMultipleObjects( 0, NULL, FALSE, 500, QS_POINTER ); ok( !res, "MsgWaitForMultipleObjects returned %#lx\n", res ); + res = GetQueueStatus( QS_RAWINPUT ); + ok( LOWORD(res) == QS_RAWINPUT, "got res %#lx\n", res ); + ok( HIWORD(res) == QS_RAWINPUT, "got res %#lx\n", res ); + memset( rawbuffer, 0, sizeof(rawbuffer) ); rawinput = (RAWINPUT *)rawbuffer; rawbuffer_size = sizeof(rawbuffer); @@ -2064,6 +2071,10 @@ static void test_hid_touch_screen(void) ok( !memcmp( rawinput->data.hid.bRawData, touch_multiple.report_buf, desc.caps.InputReportByteLength ), "got unexpected report data\n" ); + res = GetQueueStatus( QS_RAWINPUT ); + ok( LOWORD(res) == 0, "got res %#lx\n", res ); + ok( HIWORD(res) == 0, "got res %#lx\n", res ); + rawdevice.dwFlags = RIDEV_REMOVE; rawdevice.hwndTarget = 0; ret = RegisterRawInputDevices( &rawdevice, 1, sizeof(RAWINPUTDEVICE) ); diff --git a/dlls/win32u/input.c b/dlls/win32u/input.c index 4a7c04f66c5..1886ff979d7 100644 --- a/dlls/win32u/input.c +++ b/dlls/win32u/input.c @@ -791,8 +791,17 @@ BOOL WINAPI NtUserGetCursorInfo( CURSORINFO *info ) static void check_for_events( UINT flags ) { + struct peek_message_filter filter = + { + .internal = TRUE, + .flags = PM_REMOVE, + }; + MSG msg; + if (!user_driver->pProcessEvents( flags )) flush_window_surfaces( TRUE ); + + peek_message( &msg, &filter ); } /********************************************************************** diff --git a/dlls/win32u/message.c b/dlls/win32u/message.c index 25b8c3c48f0..2b416f39241 100644 --- a/dlls/win32u/message.c +++ b/dlls/win32u/message.c @@ -2695,15 +2695,6 @@ static BOOL process_hardware_message( MSG *msg, UINT hw_id, const struct hardwar return ret; } -struct peek_message_filter -{ - HWND hwnd; - UINT first; - UINT last; - UINT mask; - UINT flags; -}; - /*********************************************************************** * peek_message * @@ -2711,7 +2702,7 @@ struct peek_message_filter * available; -1 on error. * All pending sent messages are processed before returning. */ -static int peek_message( MSG *msg, const struct peek_message_filter *filter ) +int peek_message( MSG *msg, const struct peek_message_filter *filter ) { LRESULT result; HWND hwnd = filter->hwnd; @@ -2738,6 +2729,7 @@ static int peek_message( MSG *msg, const struct peek_message_filter *filter ) SERVER_START_REQ( get_message ) { + req->internal = filter->internal; req->flags = flags; req->get_win = wine_server_user_handle( hwnd ); req->get_first = first; @@ -2923,6 +2915,7 @@ static int peek_message( MSG *msg, const struct peek_message_filter *filter ) .first = info.msg.message, .last = info.msg.message, .mask = filter->mask, + .internal = filter->internal, }; peek_message( msg, &new_filter ); } diff --git a/dlls/win32u/ntuser_private.h b/dlls/win32u/ntuser_private.h index 3b6cab5bdc9..bb2169998b6 100644 --- a/dlls/win32u/ntuser_private.h +++ b/dlls/win32u/ntuser_private.h @@ -238,6 +238,17 @@ extern void free_dce( struct dce *dce, HWND hwnd ); extern void invalidate_dce( WND *win, const RECT *extra_rect ); /* message.c */ +struct peek_message_filter +{ + HWND hwnd; + UINT first; + UINT last; + UINT mask; + UINT flags; + BOOL internal; +}; + +extern int peek_message( MSG *msg, const struct peek_message_filter *filter ); extern BOOL set_keyboard_auto_repeat( BOOL enable ); /* systray.c */ diff --git a/include/wine/server_protocol.h b/include/wine/server_protocol.h index f9b591ef207..d5b9867a0a7 100644 --- a/include/wine/server_protocol.h +++ b/include/wine/server_protocol.h @@ -2877,6 +2877,8 @@ struct get_message_request unsigned int hw_id; unsigned int wake_mask; unsigned int changed_mask; + unsigned int internal; + char __pad_44[4]; }; struct get_message_reply { @@ -6507,7 +6509,7 @@ union generic_reply /* ### protocol_version begin ### */ -#define SERVER_PROTOCOL_VERSION 797 +#define SERVER_PROTOCOL_VERSION 798 /* ### protocol_version end ### */ diff --git a/server/protocol.def b/server/protocol.def index 814ad741f9a..55dda2d7636 100644 --- a/server/protocol.def +++ b/server/protocol.def @@ -2150,6 +2150,7 @@ enum message_type unsigned int hw_id; /* id of the previous hardware message (or 0) */ unsigned int wake_mask; /* wakeup bits mask */ unsigned int changed_mask; /* changed bits mask */ + unsigned int internal; /* get internal messages only */ @REPLY user_handle_t win; /* window handle */ unsigned int msg; /* message code */ diff --git a/server/queue.c b/server/queue.c index c884e9f7f55..ed099b3b989 100644 --- a/server/queue.c +++ b/server/queue.c @@ -2994,6 +2994,12 @@ DECL_HANDLER(get_message) WM_WINE_LAST_DRIVER_MSG, req->flags, reply )) return; + if (req->internal) /* check for internal messages only, leave queue flags unchanged */ + { + set_error( STATUS_PENDING ); + return; + } + queue->last_get_msg = current_time; if (!filter) filter = QS_ALLINPUT; diff --git a/server/request.h b/server/request.h index 69d10bcb138..0f5c03023bb 100644 --- a/server/request.h +++ b/server/request.h @@ -1412,7 +1412,8 @@ C_ASSERT( FIELD_OFFSET(struct get_message_request, get_last) == 24 ); C_ASSERT( FIELD_OFFSET(struct get_message_request, hw_id) == 28 ); C_ASSERT( FIELD_OFFSET(struct get_message_request, wake_mask) == 32 ); C_ASSERT( FIELD_OFFSET(struct get_message_request, changed_mask) == 36 ); -C_ASSERT( sizeof(struct get_message_request) == 40 ); +C_ASSERT( FIELD_OFFSET(struct get_message_request, internal) == 40 ); +C_ASSERT( sizeof(struct get_message_request) == 48 ); C_ASSERT( FIELD_OFFSET(struct get_message_reply, win) == 8 ); C_ASSERT( FIELD_OFFSET(struct get_message_reply, msg) == 12 ); C_ASSERT( FIELD_OFFSET(struct get_message_reply, wparam) == 16 ); diff --git a/server/trace.c b/server/trace.c index 2852770a5d2..fea0f2b3784 100644 --- a/server/trace.c +++ b/server/trace.c @@ -2735,6 +2735,7 @@ static void dump_get_message_request( const struct get_message_request *req ) fprintf( stderr, ", hw_id=%08x", req->hw_id ); fprintf( stderr, ", wake_mask=%08x", req->wake_mask ); fprintf( stderr, ", changed_mask=%08x", req->changed_mask ); + fprintf( stderr, ", internal=%08x", req->internal ); } static void dump_get_message_reply( const struct get_message_reply *req )

1 0