projects
/
linux.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
7b84434
)
checkpatch: improve ALLOC_ARRAY_ARGS test
author
Christophe JAILLET
<christophe.jaillet@wanadoo.fr>
Fri, 7 May 2021 01:04:01 +0000
(18:04 -0700)
committer
Linus Torvalds
<torvalds@linux-foundation.org>
Fri, 7 May 2021 02:24:13 +0000
(19:24 -0700)
The devm_ variant of 'kcalloc()' and 'kmalloc_array()' are not tested
Add the corresponding check.
Link:
https://lkml.kernel.org/r/205fc4847972fb6779abcc8818f39c14d1b45af1.1618595794.git.christophe.jaillet@wanadoo.fr
Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Acked-by: Joe Perches <joe@perches.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
scripts/checkpatch.pl
patch
|
blob
|
history
diff --git
a/scripts/checkpatch.pl
b/scripts/checkpatch.pl
index 44b9dc330ac6dfcffc2ffddcdd24b5b4a3d792c9..23697a6b1eaada0ca51ec27e5e073638064a3387 100755
(executable)
--- a/
scripts/checkpatch.pl
+++ b/
scripts/checkpatch.pl
@@
-7006,7
+7006,7
@@
sub process {
}
# check for alloc argument mismatch
- if ($line =~ /\b(
kcalloc|kmalloc_array
)\s*\(\s*sizeof\b/) {
+ if ($line =~ /\b(
(?:devm_)?(?:kcalloc|kmalloc_array)
)\s*\(\s*sizeof\b/) {
WARN("ALLOC_ARRAY_ARGS",
"$1 uses number as first arg, sizeof is generally wrong\n" . $herecurr);
}