dany/u-boot-megous
1
0
Fork 0
mirror of https://xff.cz/git/u-boot/ synced 2025-09-01 16:52:14 +02:00
Code Issues Packages Projects Releases Wiki Activity

common: usb: fix checking condition

Browse Source 
We support max USB_MAXENDPOINTS, so need to use
"epno >= USB_MAXENDPOINTS", but not "epno > USB_MAXENDPOINTS".
If use ">", we may exceeds the array of if_desc->ep_desc.

Signed-off-by: Peng Fan <peng.fan@nxp.com>
Cc: Simon Glass <sjg@chromium.org>
Cc: Hans de Goede <hdegoede@redhat.com>
Cc: Marek Vasut <marex@denx.de>
Cc: Paul Kocialkowski <contact@paulk.fr>
Cc: "Stefan Brüns" <stefan.bruens@rwth-aachen.de>
Cc: Vincent Palatin <vpalatin@chromium.org>
This commit is contained in:
Peng Fan
2016-01-12 15:57:01 +08:00
committed by Marek Vasut
parent 08c11cb5df
commit 447b9cdf2c
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
common/usb.c
View File

@@ -456,7 +456,7 @@ static int usb_parse_config(struct usb_device *dev,
} }
epno = dev->config.if_desc[ifno].no_of_ep; epno = dev->config.if_desc[ifno].no_of_ep;
if_desc = &dev->config.if_desc[ifno]; if_desc = &dev->config.if_desc[ifno];
if (epno > USB_MAXENDPOINTS) { if (epno >= USB_MAXENDPOINTS) {
printf("Interface %d has too many endpoints!\n", printf("Interface %d has too many endpoints!\n",
if_desc->desc.bInterfaceNumber); if_desc->desc.bInterfaceNumber);
return -EINVAL; return -EINVAL;