diff options
author | 2020-12-29 15:14:31 -0800 | |
---|---|---|
committer | 2020-12-29 15:36:49 -0800 | |
commit | 5dbdb2d87c294401a22e6a6002f08ebc9fbea38b (patch) | |
tree | edf57809079633ee9522eec5a268e0ac39d3fe64 | |
parent | Revert "kbuild: avoid static_assert for genksyms" (diff) | |
download | linux-5dbdb2d87c294401a22e6a6002f08ebc9fbea38b.tar.gz linux-5dbdb2d87c294401a22e6a6002f08ebc9fbea38b.tar.bz2 linux-5dbdb2d87c294401a22e6a6002f08ebc9fbea38b.zip |
checkpatch: prefer strscpy to strlcpy
Prefer strscpy over the deprecated strlcpy function.
Link: https://lkml.kernel.org/r/19fe91084890e2c16fe56f960de6c570a93fa99b.camel@perches.com
Signed-off-by: Joe Perches <joe@perches.com>
Requested-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rwxr-xr-x | scripts/checkpatch.pl | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index 00085308ed9d..92e888ed939f 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl @@ -6646,6 +6646,12 @@ sub process { # } # } +# strlcpy uses that should likely be strscpy + if ($line =~ /\bstrlcpy\s*\(/) { + WARN("STRLCPY", + "Prefer strscpy over strlcpy - see: https://lore.kernel.org/r/CAHk-=wgfRnXz0W3D37d01q3JFkr_i_uTL=V6A6G1oUZcprmknw\@mail.gmail.com/\n" . $herecurr); + } + # typecasts on min/max could be min_t/max_t if ($perl_version_ok && defined $stat && |