[U-Boot] [PATCH] usb: storage: Fix overwritten in usb_stor_set_max_xfer_blk()

[Bin Meng]

Hi Marek,

On Fri, Sep 29, 2017 at 5:36 PM, Marek Vasut <marex at denx.de> wrote:
> On 09/29/2017 11:26 AM, Bin Meng wrote:
>> Hi Marek,
>>
>> On Fri, Sep 29, 2017 at 10:30 AM, Marek Vasut <marex at denx.de> wrote:
>>> On 09/29/2017 01:36 AM, Bin Meng wrote:
>>>> Hi Marek,
>>>>
>>>> On Thu, Sep 28, 2017 at 11:24 PM, Marek Vasut <marex at denx.de> wrote:
>>>>> On 09/28/2017 06:50 AM, Bin Meng wrote:
>>>>>> The stored 'blk' value is overwritten to 'size / 512' before it can
>>>>>> be used in usb_stor_set_max_xfer_blk(). This is not what we want.
>>>>>> In fact, when 'size' exceeds the upper limit (USHRT_MAX * 512), we
>>>>>> should simply assign 'size' to the upper limit.
>>>>>>
>>>>>> Reported-by: Coverity (CID: 167250)
>>>>>> Signed-off-by: Bin Meng <bmeng.cn at gmail.com>
>>>>>
>>>>> Still failing
>>>>> https://travis-ci.org/marex/u-boot-usb/builds/280835848
>>>>>
>>>>
>>>> This fixes the Coverity issue, not the 'make tests' issue. Please hold
>>>> on apply the xHCI patchset and when the fix is ready I will send v2.
>>>
>>> Can't you send me fix on top of current set ? If not, OK, tell me what
>>> to drop and what to pick.
>>>
>>
>> OK, will do.
>
> Thanks!
>

I just sent additional patches
(http://patchwork.ozlabs.org/project/uboot/list/?series=5892) to fix
the 'make tests' issues.

Please do the following on the u-boot-usb/master branch:

1). Drop http://git.denx.de/?p=u-boot/u-boot-usb.git;a=commit;h=198db64035a589ef19920b2cec58419b4b338ce7
2). Drop http://git.denx.de/?p=u-boot/u-boot-usb.git;a=commit;h=9a462a3af216d77fee689f119419506218531f77
3). Apply http://patchwork.ozlabs.org/project/uboot/list/?series=5892
on top of u-boot-usb/master

Then everything should be fine.

Regards,
Bin