hidclass.sys: Call IoInvalidateDeviceRelations before IoSetDevicePropertyData.

Otherwise setupapi may fail to find the device on which to set the property. This triggers messages like the following the first time a device is created (on prefix creation for mouse and keyboard devices): IoSetDevicePropertyData Failed to open device, error 0xe000020b. Some IoSetDevicePropertyData implementation detail makes it always return successfully but it's not doing anything and user32 will then ignore the device when it enumerates rawinput devices. Signed-off-by: Rémi Bernon <rbernon@codeweavers.com> Signed-off-by: Zebediah Figura <zfigura@codeweavers.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>

hidclass.sys: Call IoInvalidateDeviceRelations before IoSetDevicePropertyData.
af2b2d04 · Rémi Bernon · Alexandre Julliard · caa5f68e · af2b2d04
Commit af2b2d04 authored May 05, 2021 by Rémi Bernon Committed by Alexandre Julliard May 05, 2021
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

pnp.c dlls/hidclass.sys/pnp.c +2 -2

No files found.
--- a/dlls/hidclass.sys/pnp.c
+++ b/dlls/hidclass.sys/pnp.c
@@ -245,6 +245,8 @@ static void create_child(minidriver *minidriver, DEVICE_OBJECT *fdo)
    else
        pdo_ext->u.pdo.rawinput_handle = alloc_rawinput_handle();

+    IoInvalidateDeviceRelations(fdo_ext->u.fdo.hid_ext.PhysicalDeviceObject, BusRelations);
+
    if ((status = IoSetDevicePropertyData(child_pdo, &DEVPROPKEY_HID_HANDLE, LOCALE_NEUTRAL,
                                          PLUGPLAY_PROPERTY_PERSISTENT, DEVPROP_TYPE_UINT32,
                                          sizeof(pdo_ext->u.pdo.rawinput_handle), &pdo_ext->u.pdo.rawinput_handle)))
@@ -254,8 +256,6 @@ static void create_child(minidriver *minidriver, DEVICE_OBJECT *fdo)
        return;
    }

-    IoInvalidateDeviceRelations(fdo_ext->u.fdo.hid_ext.PhysicalDeviceObject, BusRelations);
-
    pdo_ext->u.pdo.poll_interval = DEFAULT_POLL_INTERVAL;

    pdo_ext->u.pdo.ring_buffer = RingBuffer_Create(