diff options
author | Maulik Mankad <maulik@ti.com> | 2011-01-13 18:19:56 +0530 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2011-01-22 18:53:27 -0800 |
commit | 3c47eb06f08eb970ea9d696bcdb57a175d37b470 (patch) | |
tree | 9cc8df6c428c7973caa5b28e19ffc75622ef2c6a /drivers/usb | |
parent | d14fc1a74e846d7851f24fc9519fe87dc12a1231 (diff) | |
download | kernel_samsung_crespo-3c47eb06f08eb970ea9d696bcdb57a175d37b470.zip kernel_samsung_crespo-3c47eb06f08eb970ea9d696bcdb57a175d37b470.tar.gz kernel_samsung_crespo-3c47eb06f08eb970ea9d696bcdb57a175d37b470.tar.bz2 |
usb: gadget: composite: avoid access beyond array max length
One of the USB CV MSC tests issues Get Max LUN request with
invalid wIndex (wIndex = 65535) parameter.
Add proper handling to prevent array index out of bounds issue.
Signed-off-by: Maulik Mankad <maulik@ti.com>
Cc: David Brownell <david-b@pacbell.net>
Acked-by: Alan Stern <stern@rowland.harvard.edu>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/usb')
-rw-r--r-- | drivers/usb/gadget/composite.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/drivers/usb/gadget/composite.c b/drivers/usb/gadget/composite.c index f6ff845..1ba4bef 100644 --- a/drivers/usb/gadget/composite.c +++ b/drivers/usb/gadget/composite.c @@ -928,8 +928,9 @@ unknown: */ switch (ctrl->bRequestType & USB_RECIP_MASK) { case USB_RECIP_INTERFACE: - if (cdev->config) - f = cdev->config->interface[intf]; + if (!cdev->config || w_index >= MAX_CONFIG_INTERFACES) + break; + f = cdev->config->interface[intf]; break; case USB_RECIP_ENDPOINT: |