diff options
author | Alexey Dobriyan <adobriyan@gmail.com> | 2017-02-08 10:44:09 +1100 |
---|---|---|
committer | Stephen Rothwell <sfr@canb.auug.org.au> | 2017-02-09 17:01:37 +1100 |
commit | f7d14ff147e8f688b88a4f086c2d4c481c9513b0 (patch) | |
tree | 132e02dbd6a0e42ed30ad1f8d9b862a3fd44732d | |
parent | 5133eeedcfcc5a19fbceafab61d31f0f4c5d5899 (diff) |
checkpatch: warn when formats use %Z and suggest %z
vsnprintf extension %Z<foo> is non-standard C.
Suggest the use of %z instead.
Miscellanea:
o Correct the misuse of type string PRINTF_0xDECIMAL
type strings are supposed to be uppercase only.
Fix this and add tr/[a-z]/[A-Z] to the type check in case
I forget this again sometime in the future.
o Improve the mechanism to find these defects so all
3 current checks are done on the format string
Link: http://lkml.kernel.org/r/20170109235955.GA6787@avx2
Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
Cc: Joe Perches <joe@perches.com>
Signed-off-by: Andrew Morton <akpm@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 12db1483e6c2..289d61303cfc 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl @@ -5208,6 +5208,12 @@ sub process { "\%Ld/%Lu are not-standard C, use %lld/%llu\n" . $herecurr); last; } + # check for %Z + if ($string =~ /(?<!%)%[\*\d\.\$]*Z[diouxX]/) { + WARN("PRINTF_Z", + "%Z is non-standard C, use %z\n" . $herecurr); + last; + } if ($string =~ /0x%[\*\d\.\$\Llzth]*[udi]/) { ERROR("PRINTF_0xDECIMAL", "Prefixing 0x with decimal output is defective\n" . $herecurr); |