[PATCH] user32: Don't read preparsed data on non-HID devices.

[Rémi Bernon]

Signed-off-by: Rémi Bernon <rbernon(a)codeweavers.com>
---

This should fix a segfault in user32 input test introduced with:

  19c6524e48db1d785095953d25591f1e2d2872d9

 dlls/user32/rawinput.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/dlls/user32/rawinput.c b/dlls/user32/rawinput.c
index 2465262f74ed..dd2ac2e208b2 100644
--- a/dlls/user32/rawinput.c
+++ b/dlls/user32/rawinput.c
@@ -492,7 +492,8 @@ UINT WINAPI GetRawInputDeviceInfoW(HANDLE handle, UINT command, void *data, UINT
 
     case RIDI_PREPARSEDDATA:
         avail_bytes = *data_size;
-        if (handle == WINE_MOUSE_HANDLE || handle == WINE_KEYBOARD_HANDLE)
+        if (handle == WINE_MOUSE_HANDLE || handle == WINE_KEYBOARD_HANDLE ||
+            device->info.dwType != RIM_TYPEHID)
         {
             to_copy_bytes = 0;
             *data_size = 0;
-- 
2.26.1

[Marvin]

Hi,

While running your changed tests, I think I found new failures.
Being a bot and all I'm not very good at pattern recognition, so I might be
wrong, but could you please double-check?

Full results can be found at:
https://testbot.winehq.org/JobDetails.pl?Key=70664

Your paranoid android.


=== debiant (32 bit Chinese:China report) ===

user32:
msg.c:17608: Test succeeded inside todo block: SendMessage from other thread 1: marked "todo_wine" but succeeds
msg.c:17612: Test succeeded inside todo block: WaitForSingleObject failed, ret:0
msg.c:17628: Test succeeded inside todo block: SendMessage from other thread 3: marked "todo_wine" but succeeds
msg.c:17636: Test succeeded inside todo block: wrong status 00000000
msg.c:17640: Test succeeded inside todo block: SendMessage from other thread 5: marked "todo_wine" but succeeds