Changeset d0a6e54 in mainline for uspace/drv/usbhid/usbhid.c

Timestamp:

2011-04-21T17:16:53Z (14 years ago)

Author:

Lubos Slovak <lubos.slovak@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

3bcac68

Parents:

020559c9

Message:

Product and Vendor ID may be 0 - changed type to int.

Fixes #193

File:

• uspace/drv/usbhid/usbhid.c (modified) (2 diffs)

uspace/drv/usbhid/usbhid.c

@@ -251 +251 @@
         while (count < USB_HID_MAX_SUBDRIVERS &&
             (mapping->usage_path != NULL
-            || mapping->vendor_id != 0 || mapping->product_id != 0)) {
+            || mapping->vendor_id >= 0 || mapping->product_id >= 0)) {
                 // check the vendor & product ID
-                if (mapping->vendor_id != 0 && mapping->product_id == 0) {
-                        usb_log_warning("Missing Product ID for Vendor ID %u\n",
+                if (mapping->vendor_id >= 0 && mapping->product_id < 0) {
+                        usb_log_warning("Missing Product ID for Vendor ID %d\n",
                             mapping->vendor_id);
                         return EINVAL;
                 }
-                if (mapping->product_id != 0 && mapping->vendor_id == 0) {
-                        usb_log_warning("Missing Vendor ID for Product ID %u\n",
+                if (mapping->product_id >= 0 && mapping->vendor_id < 0) {
+                        usb_log_warning("Missing Vendor ID for Product ID %d\n",
                             mapping->product_id);
                         return EINVAL;
@@ -267 +267 @@
                 matched = false;
                 
-                if (mapping->vendor_id != 0) {
-                        assert(mapping->product_id != 0);
+                if (mapping->vendor_id >= 0) {
+                        assert(mapping->product_id >= 0);
                         usb_log_debug("Comparing device against vendor ID %u"
                             " and product ID %u.\n", mapping->vendor_id,