checkpatch-add-check-for-use-of-sizeof-without-parenthesis-v2

Add check that works for sizeof *foo as well as sizeof foo

Signed-off-by: Joe Perches <joe@perches.com>
Cc: Andy Whitcroft <apw@canonical.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>

diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
index a7436227bed9754feaad29c0ab126c0ae80f27b2..72c180323c56cf69e78c13578dece50b4c8ab9c8 100755 (executable)
--- a/scripts/checkpatch.pl
+++ b/scripts/checkpatch.pl
@@ -3266,7 +3266,7 @@ sub process {
                }
 
 # check for sizeof without parenthesis
-               if ($line =~ /\bsizeof\s+($Lval)/) {
+               if ($line =~ /\bsizeof\s+((?:\*\s*|)$Lval|$Type(?:\s+$Lval|))/) {
                        WARN("SIZEOF_PARENTHESIS",
                             "sizeof $1 should be sizeof($1)\n" . $herecurr);
                }