On 03/10/2017 10:45 AM, Nicolas le bayon wrote:
2017-03-05 1:54 GMT+01:00 Marek Vasut marex@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@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 ...