1. 31 7月, 2007 2 次提交
  2. 20 7月, 2007 4 次提交
    • J
      USB: Remove pointless conditional in drivers/usb/serial/io_ti.c::edge_shutdown() · 0d46c007
      Jesper Juhl 提交于
      Coverity scan found (CID: 1708) this in
      drivers/usb/serial/io_ti.c::edge_shutdown() :
      
      ...
      2797 		for (i=0; i < serial->num_ports; ++i) {
      2798 			edge_port = usb_get_serial_port_data(serial->port[i]);
      2799 			edge_remove_sysfs_attrs(edge_port->port);
      2800 			if (edge_port) {
      2801 				edge_buf_free(edge_port->ep_out_buf);
      2802 				kfree(edge_port);
      2803 			}
      2804 			usb_set_serial_port_data(serial->port[i], NULL);
      2805 		}
      ...
      
      It's complaining that we dereference 'edge_port' in line 2799 which
      makes the test of that pointer against NULL in 2800 pointless, since if
      edge_port was actually NULL we'd have crashed already before reaching
      line 2800.
      Reading the edge_open() function it seems to me that the pointer
      returned by usb_get_serial_port_data(serial->port[i]) and stored in
      'edge_port' can never actually be NULL here, so the test is entirely
      superfluous (even if it could be NULL it would be pointless here,
      ignoring the then possible crash in that case, since both
      edge_buf_free() and kfree() can handle being passed NULL pointers.
      
      This patch removes the pointless conditional (and also makes a few
      tiny style corrections now that I was in the area anyway).
      Signed-off-by: NJesper Juhl <jesper.juhl@gmail.com>
      Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
      0d46c007
    • F
      USB: mos7720, mos7840: remove redundant urb check · e94fa28f
      Florin Malita 提交于
      Coverity (1709, 1710, 1711, 1712, 1713) actually flagged these as
      REVERSE_INULLs (NULL check performed after dereference). But looking at
      the other drivers I can't see any similar tests and the USB core already
      makes sure urb is non-null - so might as well get rid of the checks.
      Signed-off-by: NFlorin Malita <fmalita@gmail.com>
      Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
      e94fa28f
    • K
      USB: sierra: Add new devices · 9454c46a
      Kevin Lloyd 提交于
      This patch adds new devices to the Sierra Wireless driver. This is being
      resubmitted because the dependent patch (patch 01/02) needed to be
      resubmitted.
      Signed-off-by: NKevin Lloyd <linux@sierrawireless.com>
      Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
      9454c46a
    • K
      USB: sierra: Add TRU-Install (c) Support · 112225b1
      Kevin Lloyd 提交于
      This patch adds compatibility with Sierra Wireless' new TRU-Install
      feature. Future devices that use this feature will not work unless this
      patch has been applied.
      Signed-off-by: NKevin Lloyd <linux@sierrawireless.com>
      Signed-off-by: NGreg Kroah-Hartman <gregkh@suse.de>
      
      112225b1
  3. 13 7月, 2007 34 次提交