[U-Boot] [PATCH] usb gadget: increase envstr size in cb_getvar

[Nicolas le bayon]

2017-03-05 1:54 GMT+01:00 Marek Vasut <marex at denx.de>:

> On 03/03/2017 01:26 PM, Lukasz Majewski wrote:
>
>> Hi Nicolas,
>>
>> +Marek on CC
>>
>
> Lukasz, this is gadget stuff, so it's up to you to handle it.
>
>
>
>> Hi,
>>>
>>> would it be possible to consider this patch please?
>>>
>>> Best Regards
>>> Nicolas
>>>
>>> 2016-09-21 10:43 GMT+02:00 Nicolas le bayon <nlebayon at gmail.com>:
>>>
>>> Hi,
>>>>
>>>> I propose you this patch in order to take into account strings
>>>> larger than actual size (32). I recently faced to
>>>> 'fastboot.partition-type:userdata' (it was in Android context) but
>>>> this could be the case for other purposes.
>>>>
>>>> I propose to simply double the size. Is it ok for you?
>>>>
>>>> Regards
>>>>
>>>> ---
>>>>  drivers/usb/gadget/f_fastboot.c | 2 +-
>>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>>
>>>> diff --git a/drivers/usb/gadget/f_fastboot.c b/drivers/usb/gadget/f_
>>>> fastboot.c
>>>> index 2160b1c..84e22e4 100644
>>>> --- a/drivers/usb/gadget/f_fastboot.c
>>>> +++ b/drivers/usb/gadget/f_fastboot.c
>>>> @@ -432,7 +432,7 @@ static void cb_getvar(struct usb_ep *ep, struct
>>>> usb_request *req)
>>>>   else
>>>>   strcpy(response, "FAILValue not set");
>>>>   } else {
>>>> - char envstr[32];
>>>> + char envstr[64];
>>>>
>>>>   snprintf(envstr, sizeof(envstr) - 1, "fastboot.%s", cmd);
>>>>
>>>
> What about using asprintf() or somesuch ?


It would be a nice idea, but the asprintf function is not available
unfortunately.


>
>   s = getenv(envstr);
>>>> --
>>>> 1.9.1
>>>>
>>>>
>>>>
>>>> _______________________________________________
>>> U-Boot mailing list
>>> U-Boot at lists.denx.de
>>> http://lists.denx.de/listinfo/u-boot
>>>
>>
>>
>>
>>
>> Best regards,
>>
>> Lukasz Majewski
>>
>> --
>>
>> DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
>> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
>> Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
>>
>>
>