summaryrefslogtreecommitdiffstats
path: root/libusb-1.0/libusb/os/windows_usb.c
diff options
context:
space:
mode:
Diffstat (limited to 'libusb-1.0/libusb/os/windows_usb.c')
-rw-r--r--libusb-1.0/libusb/os/windows_usb.c40
1 files changed, 9 insertions, 31 deletions
diff --git a/libusb-1.0/libusb/os/windows_usb.c b/libusb-1.0/libusb/os/windows_usb.c
index 6ad6229..9223afb 100644
--- a/libusb-1.0/libusb/os/windows_usb.c
+++ b/libusb-1.0/libusb/os/windows_usb.c
@@ -571,6 +571,7 @@ unsigned long htab_hash(char* str)
/*
* Returns the session ID of a device's nth level ancestor
+ * If there's no device at the nth level, return 0
*/
static unsigned long get_ancestor_session_id(DWORD devinst, unsigned level)
{
@@ -590,11 +591,6 @@ static unsigned long get_ancestor_session_id(DWORD devinst, unsigned level)
if (CM_Get_Device_IDA(devinst, path, MAX_PATH_LENGTH, 0) != CR_SUCCESS) {
return 0;
}
- // Return a special invalid session ID if the GP is not USB (as can be the case for HID)
- if ((level == 2) && (path[0] != 'U') && (path[1] != 'S') && (path[2] != 'B')) {
- usbi_dbg("detected non USB parent '%s'", path);
- return ~0;
- }
// TODO (post hotplug): try without sanitizing
sanitized_path = sanitize_path(path);
if (sanitized_path == NULL) {
@@ -1267,7 +1263,7 @@ static int windows_get_device_list(struct libusb_context *ctx, struct discovered
#define DEV_PASS 3
#define HID_PASS 4
int r = LIBUSB_SUCCESS;
- unsigned int nb_guids, pass, i, j;
+ unsigned int nb_guids, pass, i, j, ancestor;
char path[MAX_PATH_LENGTH];
char strbuf[MAX_PATH_LENGTH];
struct libusb_device *dev, *parent_dev;
@@ -1456,35 +1452,17 @@ static int windows_get_device_list(struct libusb_context *ctx, struct discovered
case HUB_PASS:
break;
default:
- session_id = get_ancestor_session_id(dev_info_data.DevInst, 1);
- if (session_id == 0) {
- usbi_err(ctx, "program assertion failed: orphan device '%s'", dev_id_path);
- LOOP_BREAK(LIBUSB_ERROR_NO_DEVICE);
- }
- parent_dev = usbi_get_device_by_session_id(ctx, session_id);
- // Composite HID devices have double indirection => Check grandparent
- if (pass == HID_PASS) {
- session_id = get_ancestor_session_id(dev_info_data.DevInst, 2);
+ // Go through the ancestors until we see a face we recognize
+ parent_dev = NULL;
+ for (ancestor = 1; parent_dev == NULL; ancestor++) {
+ session_id = get_ancestor_session_id(dev_info_data.DevInst, ancestor);
if (session_id == 0) {
- usbi_err(ctx, "program assertion failed: no grandparent for '%s'", dev_id_path);
- LOOP_BREAK(LIBUSB_ERROR_NO_DEVICE);
- } else if (session_id == ~0) {
- usbi_dbg("skipping non USB HID interface '%s'", dev_id_path);
- continue;
- }
- dev = usbi_get_device_by_session_id(ctx, session_id);
- if (dev == NULL) {
- usbi_err(ctx, "program assertion failed: unlisted grandparent for '%s'", dev_id_path);
- LOOP_BREAK(LIBUSB_ERROR_NO_DEVICE);
- }
- if (__device_priv(dev)->apib->id == USB_API_COMPOSITE) {
- parent_dev = dev;
+ break;
}
+ parent_dev = usbi_get_device_by_session_id(ctx, session_id);
}
if (parent_dev == NULL) {
- // This can occur if the OS only reports a newly plugged device after we started enum,
- // eg. HID parent not listed on GEN pass, but children listed on HID pass
- usbi_warn(ctx, "unlisted parent for '%s' (newly connected device?) - ignoring", dev_id_path);
+ usbi_dbg("unlisted ancestor for '%s' (non USB HID, newly connected, etc.) - ignoring", dev_id_path);
continue;
}
parent_priv = __device_priv(parent_dev);