Re: [PATCH] USB: Gadget: Composite: Debug interface comparison

[Jassi Brar <jassisinghbrar@xxxxxxxxx>]

Ping ...


On Sun, Feb 6, 2011 at 5:39 PM, Jassi Brar <jassisinghbrar@xxxxxxxxx> wrote:
> From: Jassi Brar <jassi.brar@xxxxxxxxxxx>
>
> While checking valid interface number we should compare MAX_CONFIG_INTERFACES
> with the variable 'intf' (which holds the lower 8bits of w_index) rather than
> 'w_index'
>
> Signed-off-by: Jassi Brar <jassi.brar@xxxxxxxxxxx>
> ---
> Âdrivers/usb/gadget/composite.c | Â Â6 +++---
> Â1 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/usb/gadget/composite.c b/drivers/usb/gadget/composite.c
> index 92bb61a..53e0496 100644
> --- a/drivers/usb/gadget/composite.c
> +++ b/drivers/usb/gadget/composite.c
> @@ -887,7 +887,7 @@ composite_setup(struct usb_gadget *gadget, const struct usb_ctrlrequest *ctrl)
> Â Â Â Âcase USB_REQ_SET_INTERFACE:
> Â Â Â Â Â Â Â Âif (ctrl->bRequestType != USB_RECIP_INTERFACE)
> Â Â Â Â Â Â Â Â Â Â Â Âgoto unknown;
> - Â Â Â Â Â Â Â if (!cdev->config || w_index >= MAX_CONFIG_INTERFACES)
> + Â Â Â Â Â Â Â if (!cdev->config || intf >= MAX_CONFIG_INTERFACES)
> Â Â Â Â Â Â Â Â Â Â Â Âbreak;
> Â Â Â Â Â Â Â Âf = cdev->config->interface[intf];
> Â Â Â Â Â Â Â Âif (!f)
> @@ -899,7 +899,7 @@ composite_setup(struct usb_gadget *gadget, const struct usb_ctrlrequest *ctrl)
> Â Â Â Âcase USB_REQ_GET_INTERFACE:
> Â Â Â Â Â Â Â Âif (ctrl->bRequestType != (USB_DIR_IN|USB_RECIP_INTERFACE))
> Â Â Â Â Â Â Â Â Â Â Â Âgoto unknown;
> - Â Â Â Â Â Â Â if (!cdev->config || w_index >= MAX_CONFIG_INTERFACES)
> + Â Â Â Â Â Â Â if (!cdev->config || intf >= MAX_CONFIG_INTERFACES)
> Â Â Â Â Â Â Â Â Â Â Â Âbreak;
> Â Â Â Â Â Â Â Âf = cdev->config->interface[intf];
> Â Â Â Â Â Â Â Âif (!f)
> @@ -928,7 +928,7 @@ unknown:
> Â Â Â Â Â Â Â Â */
> Â Â Â Â Â Â Â Âswitch (ctrl->bRequestType & USB_RECIP_MASK) {
> Â Â Â Â Â Â Â Âcase USB_RECIP_INTERFACE:
> - Â Â Â Â Â Â Â Â Â Â Â if (!cdev->config || w_index >= MAX_CONFIG_INTERFACES)
> + Â Â Â Â Â Â Â Â Â Â Â if (!cdev->config || intf >= MAX_CONFIG_INTERFACES)
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âbreak;
> Â Â Â Â Â Â Â Â Â Â Â Âf = cdev->config->interface[intf];
> Â Â Â Â Â Â Â Â Â Â Â Âbreak;
> --
> 1.7.0.1
>
>
ÿô.nlj·Ÿ®‰­†+%ŠË±é¥Šwÿº{.nlj·¥Š{±þë)íèjg¬±¨¶‰šŽŠÝjÿ¾«þG«é¸¢·¦j:+v‰¨Šwèm¶Ÿÿþø®w¥þŠà£¢·hšâÿ†Ù