chiark / gitweb /
fix randonm findings from llvm-clang-analyzer
authorKay Sievers <kay.sievers@vrfy.org>
Tue, 8 Sep 2009 20:11:04 +0000 (22:11 +0200)
committerKay Sievers <kay.sievers@vrfy.org>
Tue, 8 Sep 2009 20:11:04 +0000 (22:11 +0200)
Thanks to Lennart for the log file!

extras/collect/collect.c
extras/fstab_import/fstab_import.c
extras/modem-modeswitch/ma8280p_us.c
extras/scsi_id/scsi_serial.c
libudev/libudev-enumerate.c
udev/udev-rules.c
udev/udevadm-info.c

index c4523f1..0069dea 100644 (file)
@@ -331,8 +331,8 @@ static void everybody(void)
                case STATE_CONFIRMED:
                        state = "confirmed";
                        break;
-               fprintf(stderr, "ID: %s=%s\n", him->name, state);
                }
+               fprintf(stderr, "ID: %s=%s\n", him->name, state);
        }
 }
 
index 98ee75f..c66bffb 100644 (file)
@@ -152,7 +152,7 @@ int main(int argc, char *argv[])
                                pos[0] = '\0';
                                label = str;
                        }
-                       if (matches_device_list(udev, devices, str)) {
+                       if (matches_device_list(udev, devices, label)) {
                                print_fstab_entry(udev, mnt);
                                rc = 0;
                                break;
@@ -176,7 +176,7 @@ int main(int argc, char *argv[])
                                pos[0] = '\0';
                                uuid = str;
                        }
-                       if (matches_device_list(udev, devices, str)) {
+                       if (matches_device_list(udev, devices, uuid)) {
                                print_fstab_entry(udev, mnt);
                                rc = 0;
                                break;
index 3cf9eb9..d7cf4f4 100644 (file)
@@ -103,7 +103,7 @@ static void msleep(long msec)
 int ma8280p_switch (struct usb_dev_handle *devh, struct usb_device *dev)
 {
        int ret = 1;
-       char *buf = NULL;
+       char *buf = NULL;
 
        buf = malloc (65535);
        if (!buf) {
@@ -111,350 +111,350 @@ int ma8280p_switch (struct usb_dev_handle *devh, struct usb_device *dev)
                return -1;
        }
 
-       ret = usb_get_descriptor(devh, 0x0000001, 0x0000000, buf, 0x0000012);
+       usb_get_descriptor(devh, 0x0000001, 0x0000000, buf, 0x0000012);
        msleep(6);
-       ret = usb_get_descriptor(devh, 0x0000001, 0x0000000, buf, 0x0000012);
+       usb_get_descriptor(devh, 0x0000001, 0x0000000, buf, 0x0000012);
        msleep(6);
-       ret = usb_get_descriptor(devh, 0x0000002, 0x0000000, buf, 0x0000400);
+       usb_get_descriptor(devh, 0x0000002, 0x0000000, buf, 0x0000400);
        msleep(10);
        ret = usb_release_interface(devh, 0);
        if (ret != 0) 
                debug ("%s: failed to release interface before set_configuration: %d", dev->filename, ret);
-       ret = usb_set_configuration(devh, 0x0000001);
+       usb_set_configuration(devh, 0x0000001);
        ret = usb_claim_interface(devh, 0);
        if (ret != 0) 
                debug ("%s: claim after set_configuration failed with error %d", dev->filename, ret);
-       //ret = usb_set_altinterface(devh, 0);
+       //usb_set_altinterface(devh, 0);
        //msleep(33);
-       ret = usb_control_msg(devh, USB_TYPE_CLASS + USB_RECIP_INTERFACE + USB_ENDPOINT_IN, 0x0000001, 0x0000300, 0x0000000, buf, 0x0000008, 1000);
+       usb_control_msg(devh, USB_TYPE_CLASS + USB_RECIP_INTERFACE + USB_ENDPOINT_IN, 0x0000001, 0x0000300, 0x0000000, buf, 0x0000008, 1000);
        msleep(5);
        memcpy(buf, "\xb0\x04\x00\x00\x02\x90\x26\x86", 0x0000008);
-       ret = usb_control_msg(devh, USB_TYPE_CLASS + USB_RECIP_INTERFACE, 0x0000009, 0x0000300, 0x0000000, buf, 0x0000008, 1000);
+       usb_control_msg(devh, USB_TYPE_CLASS + USB_RECIP_INTERFACE, 0x0000009, 0x0000300, 0x0000000, buf, 0x0000008, 1000);
        msleep(4);
        memcpy(buf, "\xb0\x04\x00\x00\x02\x90\x26\x86", 0x0000008);
-       ret = usb_control_msg(devh, USB_TYPE_CLASS + USB_RECIP_INTERFACE, 0x0000009, 0x0000300, 0x0000000, buf, 0x0000008, 1000);
+       usb_control_msg(devh, USB_TYPE_CLASS + USB_RECIP_INTERFACE, 0x0000009, 0x0000300, 0x0000000, buf, 0x0000008, 1000);
        msleep(4);
 
        msleep(4);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(6);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(1);
        //URB_FUNCTION_GET_STATUS_FROM_ENDPOINT skipped
        msleep(4);
        memcpy(buf, "\x37\x01\xfe\xdb\xc1\x33\x1f\x83", 0x0000008);
-       ret = usb_interrupt_write(devh, 0x00000002, buf, 0x0000008, 1000);
+       usb_interrupt_write(devh, 0x00000002, buf, 0x0000008, 1000);
        msleep(3);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(1);
        memcpy(buf, "\x37\x0e\xb5\x9d\x3b\x8a\x91\x51", 0x0000008);
-       ret = usb_interrupt_write(devh, 0x00000002, buf, 0x0000008, 1000);
+       usb_interrupt_write(devh, 0x00000002, buf, 0x0000008, 1000);
        msleep(7);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(1);
        memcpy(buf, "\x34\x87\xba\x0d\xfc\x8a\x91\x51", 0x0000008);
-       ret = usb_interrupt_write(devh, 0x00000002, buf, 0x0000008, 1000);
+       usb_interrupt_write(devh, 0x00000002, buf, 0x0000008, 1000);
        msleep(7);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(2);
        //URB_FUNCTION_GET_STATUS_FROM_ENDPOINT skipped
        msleep(4);
        memcpy(buf, "\x37\x01\xfe\xdb\xc1\x33\x1f\x83", 0x0000008);
-       ret = usb_interrupt_write(devh, 0x00000002, buf, 0x0000008, 1000);
+       usb_interrupt_write(devh, 0x00000002, buf, 0x0000008, 1000);
        msleep(2);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(1);
        memcpy(buf, "\x37\x0e\xb5\x9d\x3b\x8a\x91\x51", 0x0000008);
-       ret = usb_interrupt_write(devh, 0x00000002, buf, 0x0000008, 1000);
+       usb_interrupt_write(devh, 0x00000002, buf, 0x0000008, 1000);
        msleep(7);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(1);
        memcpy(buf, "\x34\x87\xba\x0d\xfc\x8a\x91\x51", 0x0000008);
-       ret = usb_interrupt_write(devh, 0x00000002, buf, 0x0000008, 1000);
+       usb_interrupt_write(devh, 0x00000002, buf, 0x0000008, 1000);
        msleep(7);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(8);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(1);
        //URB_FUNCTION_GET_STATUS_FROM_ENDPOINT skipped
        msleep(4);
        memcpy(buf, "\x33\x04\xfe\x00\xf4\x6c\x1f\xf0", 0x0000008);
-       ret = usb_interrupt_write(devh, 0x00000002, buf, 0x0000008, 1000);
+       usb_interrupt_write(devh, 0x00000002, buf, 0x0000008, 1000);
        msleep(3);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
        msleep(1);
        //URB_FUNCTION_GET_STATUS_FROM_ENDPOINT skipped
        msleep(4);
        memcpy(buf, "\x32\x07\xfe\xf0\x29\xb9\x3a\xf0", 0x0000008);
-       ret = usb_interrupt_write(devh, 0x00000002, buf, 0x0000008, 1000);
+       usb_interrupt_write(devh, 0x00000002, buf, 0x0000008, 1000);
        msleep(3);
-       ret = usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
+       usb_interrupt_read(devh, 0x00000081, buf, 0x0000008, 1000);
 
        if (buf)
                free (buf);
index 2e9168f..7b308a9 100644 (file)
@@ -170,7 +170,6 @@ static int scsi_dump_sense(struct udev *udev,
        int code;
        int sense_class;
        int sense_key;
-       int descriptor_format;
        int asc, ascq;
 #ifdef DUMP_SENSE
        char out_buffer[256];
@@ -210,7 +209,6 @@ static int scsi_dump_sense(struct udev *udev,
                        return -1;
                }
                if ((code == 0x0) || (code == 0x1)) {
-                       descriptor_format = 0;
                        sense_key = sense_buffer[2] & 0xf;
                        if (s < 14) {
                                /*
@@ -223,7 +221,6 @@ static int scsi_dump_sense(struct udev *udev,
                        asc = sense_buffer[12];
                        ascq = sense_buffer[13];
                } else if ((code == 0x2) || (code == 0x3)) {
-                       descriptor_format = 1;
                        sense_key = sense_buffer[1] & 0xf;
                        asc = sense_buffer[2];
                        ascq = sense_buffer[3];
index e30a406..186abb9 100644 (file)
@@ -559,7 +559,7 @@ static int scan_dir_and_add_devices(struct udev_enumerate *udev_enumerate,
        if (subdir1 != NULL)
                l = util_strpcpyl(&s, l, "/", subdir1, NULL);
        if (subdir2 != NULL)
-               l = util_strpcpyl(&s, l, "/", subdir2, NULL);
+               util_strpcpyl(&s, l, "/", subdir2, NULL);
        dir = opendir(path);
        if (dir == NULL)
                return -1;
index bc22404..a85800a 100644 (file)
@@ -1127,7 +1127,7 @@ static int sort_token(struct udev_rules *rules, struct rule_tmp *rule_tmp)
 
        for (i = 0; i < rule_tmp->token_cur; i++) {
                enum token_type next_val = TK_UNSET;
-               unsigned int next_idx;
+               unsigned int next_idx = 0;
                unsigned int j;
 
                /* find smallest value */
index 519dc4a..d95091c 100644 (file)
@@ -60,7 +60,7 @@ static void print_all_attributes(struct udev_device *device, const char *key)
                        value = udev_device_get_sysattr_value(device, dent->d_name);
                        if (value == NULL)
                                continue;
-                       dbg(udev, "attr '%s'='%s'(%zi)\n", dent->d_name, value, len);
+                       dbg(udev, "attr '%s'='%s'\n", dent->d_name, value);
 
                        /* skip nonprintable attributes */
                        len = strlen(value);