On 3/24/21 8:38 PM, Simon Glass wrote:
Hi Sean,
On Thu, 11 Mar 2021 at 18:15, Sean Anderson <sean...@gmail.com> wrote:
strlcpy should always return the number of bytes copied. We were
accidentally missing the nul-terminator. We also always used to return a
non-zero value, even if we did not actually copy anything.
Fixes: 23cd138503 ("Integrate USB gadget layer and USB CDC driver layer")
Signed-off-by: Sean Anderson <sean...@gmail.com>
---
Changes in v2:
- New
lib/string.c | 12 ++++++++----
1 file changed, 8 insertions(+), 4 deletions(-)
Please can you add a test while you are here? Might be easier on the
-next branch.
This is tested in patch 3 as strlcat is implemented with strlcpy. Though
it looks like I will need to change the implementation...
--Sean
Regards,
Simon
