winebus.sys: Read irp->IoStatus.Status before leaving the CS.

Or we may have a race condition where a pending IRP may be completed after we left the CS but before we read its status. Signed-off-by: Rémi Bernon <rbernon@codeweavers.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>

winebus.sys: Read irp->IoStatus.Status before leaving the CS.
28f797ec · Rémi Bernon · Alexandre Julliard · faaad364 · 28f797ec
Commit 28f797ec authored Aug 13, 2021 by Rémi Bernon Committed by Alexandre Julliard Aug 13, 2021
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

main.c dlls/winebus.sys/main.c +1 -1

No files found.
--- a/dlls/winebus.sys/main.c
+++ b/dlls/winebus.sys/main.c
@@ -982,9 +982,9 @@ static NTSTATUS WINAPI hid_internal_dispatch(DEVICE_OBJECT *device, IRP *irp)
            break;
    }

+    status = irp->IoStatus.Status;
    LeaveCriticalSection(&ext->cs);

-    status = irp->IoStatus.Status;
    if (status != STATUS_PENDING) IoCompleteRequest(irp, IO_NO_INCREMENT);
    return status;
 }