From: Yuri Benditovich <yuri.benditov...@daynix.com>

SpiceUsbBackendDevice deals with real and emulated devices but there
is no plans to implement emulated isochronous devices.

This patch adds check to edev (emulated device) in the guard, fix the
return value to FALSE instead of plain 0 and return early in case the
code path is around emulated devices.

This is a preparatory patch to extend emulated devices support.

Signed-off-by: Victor Toso <victort...@redhat.com>

t#
---
 src/usb-backend.c | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/src/usb-backend.c b/src/usb-backend.c
index 5b52a40..3258240 100644
--- a/src/usb-backend.c
+++ b/src/usb-backend.c
@@ -357,11 +357,17 @@ gboolean 
spice_usb_backend_device_isoch(SpiceUsbBackendDevice *dev)
     gint i, j, k;
     int rc;
 
+    g_return_val_if_fail(libdev != NULL || dev->edev != NULL, FALSE);
+
+    if (dev->edev) {
+        /* currently we do not emulate isoch devices */
+        return FALSE;
+    }
+
     if (dev->cached_isochronous_valid) {
         return dev->cached_isochronous;
     }
 
-    g_return_val_if_fail(libdev != NULL, 0);
 
     rc = libusb_get_active_config_descriptor(libdev, &conf_desc);
     if (rc) {
-- 
2.21.0

_______________________________________________
Spice-devel mailing list
Spice-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/spice-devel

Reply via email to