On 03/10/2017 10:45 AM, Nicolas le bayon wrote:
> 2017-03-05 1:54 GMT+01:00 Marek Vasut <ma...@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 <nleba...@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.
Well, then use malloc + snprintf ...
--
Best regards,
Marek Vasut
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
