]> git.proxmox.com Git - mirror_ubuntu-kernels.git/commitdiff
USB: r8a66597: clean up. remove unneeded null checks
authorDan Carpenter <error27@gmail.com>
Tue, 10 Nov 2009 08:53:58 +0000 (10:53 +0200)
committerGreg Kroah-Hartman <gregkh@suse.de>
Fri, 11 Dec 2009 19:55:22 +0000 (11:55 -0800)
td and dev can not be null.

Also they are dereferenced in list_for_each_entry_safe and list_for_each
before the check happens so we would have an oops if it were possible
for them to be null.

Found using the smatch static checker.

Signed-off-by: Dan Carpenter <error27@gmail.com>
Acked-by: Yoshihiro Shimoda <shimoda.yoshihiro@renesas.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/usb/host/r8a66597-hcd.c

index e33d36256350b8d0b3a4c23577d9f59a92ed1ef3..41dbc70ae752eaf41332807d4be86905801c47a8 100644 (file)
@@ -822,8 +822,6 @@ static void force_dequeue(struct r8a66597 *r8a66597, u16 pipenum, u16 address)
                return;
 
        list_for_each_entry_safe(td, next, list, queue) {
-               if (!td)
-                       continue;
                if (td->address != address)
                        continue;
 
@@ -2025,8 +2023,6 @@ static struct r8a66597_device *get_r8a66597_device(struct r8a66597 *r8a66597,
        struct list_head *list = &r8a66597->child_device;
 
        list_for_each_entry(dev, list, device_list) {
-               if (!dev)
-                       continue;
                if (dev->usb_address != addr)
                        continue;