usbredirhost: Ensure all in flight urbs are cancelled before a reset

The guest should have cancelled any pending urbs already, but the cancellations
have not necessary completed when we receive a reset.

If we then continue with the reset pending urbs may complete with an error code
of LIBUSB_TRANSFER_NO_DEVICE, causing us to see a device disconnect.

We also need to cleanly shutdown any streams and let the guest know they
should be restarted after the reset.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>

1 files changed, 11 insertions, 0 deletions

diff --git a/usbredirhost/usbredirhost.c b/usbredirhost/usbredirhost.c
index 6d361d2..5ec3664 100644
--- a/usbredirhost/usbredirhost.c
+++ b/usbredirhost/usbredirhost.c
@@ -1548,6 +1548,17 @@ static void usbredirhost_reset(void *priv)
         return;
     }
 
+    /*
+     * The guest should have cancelled any pending urbs already, but the
+     * cancellations may be awaiting completion, and if we then do a reset
+     * they will complete with an error code of LIBUSB_TRANSFER_NO_DEVICE.
+     *
+     * And we also need to cleanly shutdown any streams (and let the guest
+     * know they should be restarted after the reset).
+     */
+    if (usbredirhost_cancel_pending_urbs(host, 1))
+        usbredirhost_wait_for_cancel_completion(host);
+
     r = usbredirhost_reset_device(host);
     if (r != 0) {
         host->read_status = usbredirhost_read_device_lost;