Staging: line6: fix leaks in line6_probe()
authorKulikov Vasiliy <segooon@gmail.com>
Tue, 13 Jul 2010 11:22:46 +0000 (15:22 +0400)
committerGreg Kroah-Hartman <gregkh@suse.de>
Thu, 22 Jul 2010 18:30:49 +0000 (11:30 -0700)
If error occurs line6_probe() must put interface and usbdev that were
got before.

Signed-off-by: Kulikov Vasiliy <segooon@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/line6/driver.c

index 1d5a4730276363877f3f6e26f8e9710fc8811dd7..27b986a50a0390788e002871d741daa183e97b4d 100644 (file)
@@ -679,8 +679,10 @@ static int line6_probe(struct usb_interface *interface, const struct usb_device_
        usb_get_dev(usbdev);
 
        /* we don't handle multiple configurations */
-       if (usbdev->descriptor.bNumConfigurations != 1)
-               return -ENODEV;
+       if (usbdev->descriptor.bNumConfigurations != 1) {
+               ret = -ENODEV;
+               goto err_put;
+       }
 
        /* check vendor and product id */
        for (devtype = ARRAY_SIZE(line6_id_table) - 1; devtype--;) {
@@ -692,16 +694,20 @@ static int line6_probe(struct usb_interface *interface, const struct usb_device_
                        break;
        }
 
-       if (devtype < 0)
-               return -ENODEV;
+       if (devtype < 0) {
+               ret = -ENODEV;
+               goto err_put;
+       }
 
        /* find free slot in device table: */
        for (devnum = 0; devnum < LINE6_MAX_DEVICES; ++devnum)
                if (line6_devices[devnum] == NULL)
                        break;
 
-       if (devnum == LINE6_MAX_DEVICES)
-               return -ENODEV;
+       if (devnum == LINE6_MAX_DEVICES) {
+               ret = -ENODEV;
+               goto err_put;
+       }
 
        /* initialize device info: */
        properties = &line6_properties_table[devtype];
@@ -762,13 +768,14 @@ static int line6_probe(struct usb_interface *interface, const struct usb_device_
 
        default:
                MISSING_CASE;
-               return -ENODEV;
+               ret = -ENODEV;
+               goto err_put;
        }
 
        ret = usb_set_interface(usbdev, interface_number, alternate);
        if (ret < 0) {
                dev_err(&interface->dev, "set_interface failed\n");
-               return ret;
+               goto err_put;
        }
 
        /* initialize device data based on product id: */
@@ -815,7 +822,8 @@ static int line6_probe(struct usb_interface *interface, const struct usb_device_
                        break;
 
                default:
-                       return -ENODEV;
+                       ret = -ENODEV;
+                       goto err_put;
                }
                break;
 
@@ -827,19 +835,22 @@ static int line6_probe(struct usb_interface *interface, const struct usb_device_
 
        default:
                MISSING_CASE;
-               return -ENODEV;
+               ret = -ENODEV;
+               goto err_put;
        }
 
        if (size == 0) {
                dev_err(line6->ifcdev, "driver bug: interface data size not set\n");
-               return -ENODEV;
+               ret = -ENODEV;
+               goto err_put;
        }
 
        line6 = kzalloc(size, GFP_KERNEL);
 
        if (line6 == NULL) {
                dev_err(&interface->dev, "Out of memory\n");
-               return -ENOMEM;
+               ret = -ENODEV;
+               goto err_put;
        }
 
        /* store basic data: */
@@ -875,16 +886,16 @@ static int line6_probe(struct usb_interface *interface, const struct usb_device_
 
                if (line6->buffer_listen == NULL) {
                        dev_err(&interface->dev, "Out of memory\n");
-                       line6_destruct(interface);
-                       return -ENOMEM;
+                       ret = -ENOMEM;
+                       goto err_destruct;
                }
 
                line6->buffer_message = kmalloc(LINE6_MESSAGE_MAXLEN, GFP_KERNEL);
 
                if (line6->buffer_message == NULL) {
                        dev_err(&interface->dev, "Out of memory\n");
-                       line6_destruct(interface);
-                       return -ENOMEM;
+                       ret = -ENOMEM;
+                       goto err_destruct;
                }
 
                line6->urb_listen = usb_alloc_urb(0, GFP_KERNEL);
@@ -892,15 +903,15 @@ static int line6_probe(struct usb_interface *interface, const struct usb_device_
                if (line6->urb_listen == NULL) {
                        dev_err(&interface->dev, "Out of memory\n");
                        line6_destruct(interface);
-                       return -ENOMEM;
+                       ret = -ENOMEM;
+                       goto err_destruct;
                }
 
                ret = line6_start_listen(line6);
                if (ret < 0) {
                        dev_err(&interface->dev, "%s: usb_submit_urb failed\n",
                                __func__);
-                       line6_destruct(interface);
-                       return ret;
+                       goto err_destruct;
                }
        }
 
@@ -952,22 +963,25 @@ static int line6_probe(struct usb_interface *interface, const struct usb_device_
                ret = -ENODEV;
        }
 
-       if (ret < 0) {
-               line6_destruct(interface);
-               return ret;
-       }
+       if (ret < 0)
+               goto err_destruct;
 
        ret = sysfs_create_link(&interface->dev.kobj, &usbdev->dev.kobj,
                                "usb_device");
-       if (ret < 0) {
-               line6_destruct(interface);
-               return ret;
-       }
+       if (ret < 0)
+               goto err_destruct;
 
        dev_info(&interface->dev, "Line6 %s now attached\n",
                 line6->properties->name);
        line6_devices[devnum] = line6;
        line6_list_devices();
+       return 0;
+
+err_destruct:
+       line6_destruct(interface);
+err_put:
+       usb_put_intf(interface);
+       usb_put_dev(usbdev);
        return ret;
 }