usb: gadget: composite: fix incorrect handling of OS desc requests
authorChris Dickens <christopher.a.dickens@gmail.com>
Mon, 1 Jan 2018 02:59:42 +0000 (18:59 -0800)
committerFelipe Balbi <felipe.balbi@linux.intel.com>
Thu, 8 Mar 2018 13:12:01 +0000 (15:12 +0200)
commit5d6ae4f0da8a64a185074dabb1b2f8c148efa741
treed61dd8340070101c7c1cdba2d917b78faa885232
parentf3768997013e1c7d625ca427150644f80eb5900e
usb: gadget: composite: fix incorrect handling of OS desc requests

When handling an OS descriptor request, one of the first operations is
to zero out the request buffer using the wLength from the setup packet.
There is no bounds checking, so a wLength > 4096 would clobber memory
adjacent to the request buffer. Fix this by taking the min of wLength
and the request buffer length prior to the memset. While at it, define
the buffer length in a header file so that magic numbers don't appear
throughout the code.

When returning data to the host, the data length should be the min of
the wLength and the valid data we have to return. Currently we are
returning wLength, thus requests for a wLength greater than the amount
of data in the OS descriptor buffer would return invalid (albeit zero'd)
data following the valid descriptor data. Fix this by counting the
number of bytes when constructing the data and using this when
determining the length of the request.

Signed-off-by: Chris Dickens <christopher.a.dickens@gmail.com>
Signed-off-by: Felipe Balbi <felipe.balbi@linux.intel.com>
drivers/usb/gadget/composite.c
include/linux/usb/composite.h