diff options
author | Stefan Brüns <stefan.bruens@rwth-aachen.de> | 2016-01-17 04:09:51 +0100 |
---|---|---|
committer | Marek Vasut <marex@denx.de> | 2016-01-23 16:21:10 +0100 |
commit | 56a7bbd7412ae8dedca14877bb7aa45199de730c (patch) | |
tree | 995ae9799c9c46806ffca44a2020eda9de1774c1 /drivers/usb/host/xhci-dwc3.c | |
parent | 12f229ea8f6c8e20f8fd07906eafc853c4c354a9 (diff) | |
download | blackbird-obmc-uboot-56a7bbd7412ae8dedca14877bb7aa45199de730c.tar.gz blackbird-obmc-uboot-56a7bbd7412ae8dedca14877bb7aa45199de730c.zip |
usb: dwc2: Fix out-of-bounds access, fix chunk size
Fix two errors in transfer len calculation, move loop invariant code out
of loop.
If xfer_len is equal to CONFIG_DWC2_MAX_TRANSFER_SIZE (or slightly
smaller), the xfer_len will be to large, e.g.:
xfer_len = MAX_TRANSFER_SIZE = 65535
max packet size = 512
=> num_packets = 128
=> IN xfer_len = 65536
For OUT transactions larger than (65536 - mps) bytes, the xfer_len
determination is quite awkward, it is only correct due to:
- max_packet_size for control/bulk/interrupt is required to be
power-of-two.
- (CONFIG_DWC2_MAX_TRANSFER_SIZE + 1) % max-packet-size is zero
for all allowed (2^3 ... 2^9) packet sizes
As the max xfer len is loop invariant, it can be moved out of the loop.
Signed-off-by: Stefan Brüns <stefan.bruens@rwth-aachen.de>
Diffstat (limited to 'drivers/usb/host/xhci-dwc3.c')
0 files changed, 0 insertions, 0 deletions