[U-Boot] [PATCH] cmd: usb_mass_storage: add protection for block_dev
Marek Vasut
marex at denx.de
Wed Oct 18 20:22:38 UTC 2017
On 10/18/2017 06:27 PM, Tom Rini wrote:
> On Wed, Oct 18, 2017 at 04:03:21PM +0200, Patrick Delaunay wrote:
>
>> solve data abort for the command "ums 0 ubi 0"
>> because result of case blk_get_device_part_str() result is OK
>> but with block_dev = 0 when CONFIG_CMD_UBIFS is activate and
>> ubi volume is mounted
>>
>> Signed-off-by: Patrick Delaunay <patrick.delaunay at st.com>
This is UMS, so Lukasz ... please review _thoroughly_ , as I suspect
there might be some weird underlying issue .
>> ---
>>
>> cmd/usb_mass_storage.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/cmd/usb_mass_storage.c b/cmd/usb_mass_storage.c
>> index cfeecb7..c0563ca 100644
>> --- a/cmd/usb_mass_storage.c
>> +++ b/cmd/usb_mass_storage.c
>> @@ -85,7 +85,7 @@ static int ums_init(const char *devtype, const char *devnums_part_str)
>> partnum = 0;
>>
>> /* f_mass_storage.c assumes SECTOR_SIZE sectors */
>> - if (block_dev->blksz != SECTOR_SIZE)
>> + if (!block_dev || block_dev->blksz != SECTOR_SIZE)
>> goto cleanup;
>>
>> ums_new = realloc(ums, (ums_count + 1) * sizeof(*ums));
>
> Adding in Marek..
>
--
Best regards,
Marek Vasut
More information about the U-Boot
mailing list