|
 |
1b1995d |
From 49a01afb24b925de97074d093fb072bb7de470f9 Mon Sep 17 00:00:00 2001
|
|
|
1b1995d |
From: Hans de Goede <hdegoede@redhat.com>
|
|
|
1b1995d |
Date: Sat, 18 Feb 2012 09:18:57 +0100
|
|
|
1b1995d |
Subject: [PATCH 122/140] usb-redir: Always clear device state on filter
|
|
|
1b1995d |
reject
|
|
|
1b1995d |
|
|
|
1b1995d |
Always call usbredir_device_disconnect() when usbredir_check_filter() fails
|
|
|
1b1995d |
to clean up all the device state (ie received endpoint info).
|
|
|
1b1995d |
|
|
|
1b1995d |
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
|
|
|
1b1995d |
---
|
|
|
1b1995d |
usb-redir.c | 11 +++++++----
|
|
|
1b1995d |
1 file changed, 7 insertions(+), 4 deletions(-)
|
|
|
1b1995d |
|
|
|
1b1995d |
diff --git a/usb-redir.c b/usb-redir.c
|
|
|
1b1995d |
index 9b804e9..fe3b0a3 100644
|
|
|
1b1995d |
--- a/usb-redir.c
|
|
|
1b1995d |
+++ b/usb-redir.c
|
|
|
1b1995d |
@@ -985,7 +985,7 @@ static int usbredir_check_filter(USBRedirDevice *dev)
|
|
|
1b1995d |
{
|
|
|
1b1995d |
if (dev->interface_info.interface_count == 0) {
|
|
|
1b1995d |
ERROR("No interface info for device\n");
|
|
|
1b1995d |
- return -1;
|
|
|
1b1995d |
+ goto error;
|
|
|
1b1995d |
}
|
|
|
1b1995d |
|
|
|
1b1995d |
if (dev->filter_rules) {
|
|
|
1b1995d |
@@ -993,7 +993,7 @@ static int usbredir_check_filter(USBRedirDevice *dev)
|
|
|
1b1995d |
usb_redir_cap_connect_device_version)) {
|
|
|
1b1995d |
ERROR("Device filter specified and peer does not have the "
|
|
|
1b1995d |
"connect_device_version capability\n");
|
|
|
1b1995d |
- return -1;
|
|
|
1b1995d |
+ goto error;
|
|
|
1b1995d |
}
|
|
|
1b1995d |
|
|
|
1b1995d |
if (usbredirfilter_check(
|
|
|
1b1995d |
@@ -1010,11 +1010,15 @@ static int usbredir_check_filter(USBRedirDevice *dev)
|
|
|
1b1995d |
dev->device_info.product_id,
|
|
|
1b1995d |
dev->device_info.device_version_bcd,
|
|
|
1b1995d |
0) != 0) {
|
|
|
1b1995d |
- return -1;
|
|
|
1b1995d |
+ goto error;
|
|
|
1b1995d |
}
|
|
|
1b1995d |
}
|
|
|
1b1995d |
|
|
|
1b1995d |
return 0;
|
|
|
1b1995d |
+
|
|
|
1b1995d |
+error:
|
|
|
1b1995d |
+ usbredir_device_disconnect(dev);
|
|
|
1b1995d |
+ return -1;
|
|
|
1b1995d |
}
|
|
|
1b1995d |
|
|
|
1b1995d |
/*
|
|
|
1b1995d |
@@ -1140,7 +1144,6 @@ static void usbredir_interface_info(void *priv,
|
|
|
1b1995d |
if (usbredir_check_filter(dev)) {
|
|
|
1b1995d |
ERROR("Device no longer matches filter after interface info "
|
|
|
1b1995d |
"change, disconnecting!\n");
|
|
|
1b1995d |
- usbredir_device_disconnect(dev);
|
|
|
1b1995d |
}
|
|
|
1b1995d |
}
|
|
|
1b1995d |
}
|
|
|
1b1995d |
--
|
|
|
1b1995d |
1.7.9.3
|
|
|
1b1995d |
|