USB: rio500: refuse more than one device at a time

commit 3864d33943b4a76c6e64616280e98d2410b1190f upstream.

This driver is using a global variable. It cannot handle more than
one device at a time. The issue has been existing since the dawn
of the driver.

Signed-off-by: Oliver Neukum <oneukum@suse.com>
Reported-by: syzbot+35f04d136fc975a70da4@syzkaller.appspotmail.com
Cc: stable <stable@vger.kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/drivers/usb/misc/rio500.c b/drivers/usb/misc/rio500.c
index ddfebb144aaa44fd928c3177d378628a5538f6bc..86e0af5842daba8c624bf77141d52fa92228b735 100644 (file)
--- a/drivers/usb/misc/rio500.c
+++ b/drivers/usb/misc/rio500.c
@@ -460,15 +460,23 @@ static int probe_rio(struct usb_interface *intf,
 {
        struct usb_device *dev = interface_to_usbdev(intf);
        struct rio_usb_data *rio = &rio_instance;
-       int retval;
+       int retval = 0;
 
-       dev_info(&intf->dev, "USB Rio found at address %d\n", dev->devnum);
+       mutex_lock(&rio500_mutex);
+       if (rio->present) {
+               dev_info(&intf->dev, "Second USB Rio at address %d refused\n", dev->devnum);
+               retval = -EBUSY;
+               goto bail_out;
+       } else {
+               dev_info(&intf->dev, "USB Rio found at address %d\n", dev->devnum);
+       }
 
        retval = usb_register_dev(intf, &usb_rio_class);
        if (retval) {
                dev_err(&dev->dev,
                        "Not able to get a minor for this device.\n");
-               return -ENOMEM;
+               retval = -ENOMEM;
+               goto bail_out;
        }
 
        rio->rio_dev = dev;
@@ -477,7 +485,8 @@ static int probe_rio(struct usb_interface *intf,
                dev_err(&dev->dev,
                        "probe_rio: Not enough memory for the output buffer\n");
                usb_deregister_dev(intf, &usb_rio_class);
-               return -ENOMEM;
+               retval = -ENOMEM;
+               goto bail_out;
        }
        dev_dbg(&intf->dev, "obuf address:%p\n", rio->obuf);
 
@@ -486,7 +495,8 @@ static int probe_rio(struct usb_interface *intf,
                        "probe_rio: Not enough memory for the input buffer\n");
                usb_deregister_dev(intf, &usb_rio_class);
                kfree(rio->obuf);
-               return -ENOMEM;
+               retval = -ENOMEM;
+               goto bail_out;
        }
        dev_dbg(&intf->dev, "ibuf address:%p\n", rio->ibuf);
 
@@ -494,8 +504,10 @@ static int probe_rio(struct usb_interface *intf,
 
        usb_set_intfdata (intf, rio);
        rio->present = 1;
+bail_out:
+       mutex_unlock(&rio500_mutex);
 
-       return 0;
+       return retval;
 }
 
 static void disconnect_rio(struct usb_interface *intf)