diff options
| author | Matthias Schiffer <[email protected]> | 2023-07-14 13:24:50 +0200 |
|---|---|---|
| committer | Tom Rini <[email protected]> | 2023-08-08 17:05:43 -0400 |
| commit | 615828721abfe8c73b5103d4436402ecbf9b9897 (patch) | |
| tree | bdd4cb11ef01155c17dfa51115eeff976bba05e7 /scripts/checkpatch.pl | |
| parent | a169438411f9277cc689c14078151aa1d1caae3c (diff) | |
Revert "lib: string: Fix strlcpy return value", fix callers
Both the Linux kernel and libbsd agree that strlcpy() should always
return strlen(src) and not include the NUL termination. The incorrect
U-Boot implementation makes it impossible to check the return value for
truncation, and breaks code written with the usual implementation in
mind (for example, fdtdec_add_reserved_memory() was subtly broken).
I reviewed all callers of strlcpy() and strlcat() and fixed them
according to my understanding of the intended function.
This reverts commit d3358ecc54be0bc3b4dd11f7a63eab0a2842f772 and adds
related fixes.
Fixes: d3358ecc54be ("lib: string: Fix strlcpy return value")
Signed-off-by: Matthias Schiffer <[email protected]>
Reviewed-by: Simon Glass <[email protected]>
Reviewed-by: Sean Anderson <[email protected]>
Diffstat (limited to 'scripts/checkpatch.pl')
0 files changed, 0 insertions, 0 deletions
