2014-04-01 17:16 GMT+08:00 Marek Vasut <ma...@denx.de>:
> On Tuesday, April 01, 2014 at 10:46:52 AM, Kuo-Jung Su wrote:
>> From: Kuo-Jung Su <dant...@faraday-tech.com>
>>
>> It would be better to have strlcpy() moved to lib/string.c,
>> so that it could be reused by others without enabling
>> USB Gadget Ethernet.
>>
>> Signed-off-by: Kuo-Jung Su <dant...@faraday-tech.com>
>> CC: Albert Aribaud <albert.u.b...@aribaud.net>
>> CC: Wolfgang Denk <w...@denx.de>
>> Cc: Marek Vasut <ma...@denx.de>
>
> Good article on strlcpy() is here [1]. I suggest we remove strlcpy() 
> altogether
> and use standard posix functions like strncpy() instead. I am on Ulrich 
> Drepper
> side on this, strlcpy() is just hiding bugs.
>
> [1] https://lwn.net/Articles/507319/
>

Agree, I'll use strncpy instead of strlcpy in drivers/clk/clkdev.c &
drivers/usb/gadget/ether.c

And the patch for drivers/usb/gadget/ether.c would not be bonded to
this patch series.

-- 
Best wishes,
Kuo-Jung Su
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to