diff options
author | Satyam Sharma <satyam@infradead.org> | 2007-09-02 16:07:31 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2007-10-12 17:55:25 -0400 |
commit | e6d42f0e962390e956545aa282efb05a9d6a41d1 (patch) | |
tree | 77dec05024647ac82094bd5e62fb6baf7d098307 /drivers/usb | |
parent | 5adceac5da3d353dd1318d0482cab94a79472adf (diff) |
USB: sisusbvga: Fix bug
drivers/usb/misc/sisusbvga/sisusb.c: In function sisusb_open
drivers/usb/misc/sisusbvga/sisusb.c:2444: warning: sisusb is used uninitialized in this function
is a genuine bug (which will cause oops). We cannot use "sisusb" in
error path for (!interface), because sisusb will itself be derived
from "interface" later.
Signed-off-by: Satyam Sharma <satyam@infradead.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/usb')
-rw-r--r-- | drivers/usb/misc/sisusbvga/sisusb.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/usb/misc/sisusbvga/sisusb.c b/drivers/usb/misc/sisusbvga/sisusb.c index 4d6b89336e63..033bdfe9b215 100644 --- a/drivers/usb/misc/sisusbvga/sisusb.c +++ b/drivers/usb/misc/sisusbvga/sisusb.c | |||
@@ -2440,10 +2440,8 @@ sisusb_open(struct inode *inode, struct file *file) | |||
2440 | struct usb_interface *interface; | 2440 | struct usb_interface *interface; |
2441 | int subminor = iminor(inode); | 2441 | int subminor = iminor(inode); |
2442 | 2442 | ||
2443 | if (!(interface = usb_find_interface(&sisusb_driver, subminor))) { | 2443 | if (!(interface = usb_find_interface(&sisusb_driver, subminor))) |
2444 | dev_err(&sisusb->sisusb_dev->dev, "Failed to find interface\n"); | ||
2445 | return -ENODEV; | 2444 | return -ENODEV; |
2446 | } | ||
2447 | 2445 | ||
2448 | if (!(sisusb = usb_get_intfdata(interface))) | 2446 | if (!(sisusb = usb_get_intfdata(interface))) |
2449 | return -ENODEV; | 2447 | return -ENODEV; |