summaryrefslogtreecommitdiffstats
path: root/scripts/headers_check.pl
diff options
context:
space:
mode:
authorakpm@linux-foundation.org <akpm@linux-foundation.org>2010-11-30 13:51:13 -0800
committerMichal Marek <mmarek@suse.cz>2010-12-14 17:02:45 +0100
commitde323f22a83b024b7432e813609c6efb74b1bbfc (patch)
tree69e8528d4b5446f1300885f54821e93dc349aa98 /scripts/headers_check.pl
parenta1087ef6abedf0bfd60e5e3fddf33192cb2c1325 (diff)
downloadop-kernel-dev-de323f22a83b024b7432e813609c6efb74b1bbfc.zip
op-kernel-dev-de323f22a83b024b7432e813609c6efb74b1bbfc.tar.gz
headers_check: better search for functions in headers
Some headers don't bother with "extern" in function prototypes, which results in said prototypes being unnoticed and exported to userland. This patch slightly improves detection of such cases by checking for C type names as well in the beginning of a line. Signed-off-by: Alexander Shishkin <virtuoso@slind.org> Cc: Stephen Hemminger <shemminger@vyatta.com> Cc: Michal Marek <mmarek@suse.cz> Cc: WANG Cong <amwang@redhat.com> Cc: Randy Dunlap <randy.dunlap@oracle.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Michal Marek <mmarek@suse.cz>
Diffstat (limited to 'scripts/headers_check.pl')
-rw-r--r--scripts/headers_check.pl2
1 files changed, 1 insertions, 1 deletions
diff --git a/scripts/headers_check.pl b/scripts/headers_check.pl
index 50d6cfd..e0e25a1 100644
--- a/scripts/headers_check.pl
+++ b/scripts/headers_check.pl
@@ -64,7 +64,7 @@ sub check_include
sub check_declarations
{
- if ($line =~m/^\s*extern\b/) {
+ if ($line =~m/^(\s*extern|unsigned|char|short|int|long|void)\b/) {
printf STDERR "$filename:$lineno: " .
"userspace cannot call function or variable " .
"defined in the kernel\n";
OpenPOWER on IntegriCloud