blob: 64ac2380e4d52c91de87df2d2f222c978ad39bee [file] [log] [blame]
Jeremy Huntwork15a2ee72008-10-29 14:20:13 -07001#!/usr/bin/perl -w
Sam Ravnborg77124012008-06-15 21:41:09 +02002#
3# headers_check.pl execute a number of trivial consistency checks
4#
Amerigo Wang67b7ebe2009-06-04 22:12:01 -04005# Usage: headers_check.pl dir arch [files...]
Sam Ravnborg77124012008-06-15 21:41:09 +02006# dir: dir to look for included files
7# arch: architecture
8# files: list of files to check
9#
10# The script reads the supplied files line by line and:
11#
12# 1) for each include statement it checks if the
13# included file actually exists.
14# Only include files located in asm* and linux* are checked.
15# The rest are assumed to be system include files.
16#
Mike Frysinger46b8af52008-12-27 02:43:36 -050017# 2) It is checked that prototypes does not use "extern"
18#
Sam Ravnborg7e557a22008-12-27 19:52:20 +010019# 3) Check for leaked CONFIG_ symbols
Sam Ravnborg77124012008-06-15 21:41:09 +020020
21use strict;
Bobby Powersf75a8df2012-03-05 15:08:09 -080022use File::Basename;
Sam Ravnborg77124012008-06-15 21:41:09 +020023
24my ($dir, $arch, @files) = @ARGV;
25
26my $ret = 0;
27my $line;
28my $lineno = 0;
29my $filename;
30
31foreach my $file (@files) {
32 $filename = $file;
Stephen Hemmingerdbbe33e2010-02-22 15:17:24 -080033
34 open(my $fh, '<', $filename)
35 or die "$filename: $!\n";
Sam Ravnborg77124012008-06-15 21:41:09 +020036 $lineno = 0;
Stephen Hemmingerdbbe33e2010-02-22 15:17:24 -080037 while ($line = <$fh>) {
Sam Ravnborg77124012008-06-15 21:41:09 +020038 $lineno++;
Sam Ravnborg483b4122008-12-30 11:34:58 +010039 &check_include();
40 &check_asm_types();
41 &check_sizetypes();
Amerigo Wang67b7ebe2009-06-04 22:12:01 -040042 &check_declarations();
Sam Ravnborg7e3fa562009-01-30 23:56:42 +010043 # Dropped for now. Too much noise &check_config();
Sam Ravnborg77124012008-06-15 21:41:09 +020044 }
Stephen Hemmingerdbbe33e2010-02-22 15:17:24 -080045 close $fh;
Sam Ravnborg77124012008-06-15 21:41:09 +020046}
47exit $ret;
48
49sub check_include
50{
51 if ($line =~ m/^\s*#\s*include\s+<((asm|linux).*)>/) {
52 my $inc = $1;
53 my $found;
54 $found = stat($dir . "/" . $inc);
55 if (!$found) {
56 $inc =~ s#asm/#asm-$arch/#;
57 $found = stat($dir . "/" . $inc);
58 }
59 if (!$found) {
60 printf STDERR "$filename:$lineno: included file '$inc' is not exported\n";
61 $ret = 1;
62 }
63 }
64}
Mike Frysinger46b8af52008-12-27 02:43:36 -050065
Amerigo Wang67b7ebe2009-06-04 22:12:01 -040066sub check_declarations
Mike Frysinger46b8af52008-12-27 02:43:36 -050067{
akpm@linux-foundation.orgde323f22010-11-30 13:51:13 -080068 if ($line =~m/^(\s*extern|unsigned|char|short|int|long|void)\b/) {
Amerigo Wang67b7ebe2009-06-04 22:12:01 -040069 printf STDERR "$filename:$lineno: " .
akpm@linux-foundation.orgd52784e2010-11-30 13:52:14 -080070 "userspace cannot reference function or " .
71 "variable defined in the kernel\n";
Mike Frysinger46b8af52008-12-27 02:43:36 -050072 }
73}
Sam Ravnborg7e557a22008-12-27 19:52:20 +010074
75sub check_config
76{
Robert P. J. Day1581c1c2009-05-12 13:43:36 -070077 if ($line =~ m/[^a-zA-Z0-9_]+CONFIG_([a-zA-Z0-9_]+)[^a-zA-Z0-9_]/) {
Sam Ravnborg7e557a22008-12-27 19:52:20 +010078 printf STDERR "$filename:$lineno: leaks CONFIG_$1 to userspace where it is not valid\n";
79 }
80}
81
Sam Ravnborg483b4122008-12-30 11:34:58 +010082my $linux_asm_types;
Stephen Hemmingerdbbe33e2010-02-22 15:17:24 -080083sub check_asm_types
Sam Ravnborg483b4122008-12-30 11:34:58 +010084{
Sam Ravnborgb67ff8c2008-12-31 09:32:30 +010085 if ($filename =~ /types.h|int-l64.h|int-ll64.h/o) {
86 return;
87 }
Sam Ravnborg483b4122008-12-30 11:34:58 +010088 if ($lineno == 1) {
89 $linux_asm_types = 0;
90 } elsif ($linux_asm_types >= 1) {
91 return;
92 }
93 if ($line =~ m/^\s*#\s*include\s+<asm\/types.h>/) {
94 $linux_asm_types = 1;
95 printf STDERR "$filename:$lineno: " .
96 "include of <linux/types.h> is preferred over <asm/types.h>\n"
97 # Warn until headers are all fixed
98 #$ret = 1;
99 }
100}
101
102my $linux_types;
Bobby Powersf75a8df2012-03-05 15:08:09 -0800103my %import_stack = ();
104sub check_include_typesh
105{
106 my $path = $_[0];
107 my $import_path;
108
109 my $fh;
110 my @file_paths = ($path, $dir . "/" . $path, dirname($filename) . "/" . $path);
111 for my $possible ( @file_paths ) {
112 if (not $import_stack{$possible} and open($fh, '<', $possible)) {
113 $import_path = $possible;
114 $import_stack{$import_path} = 1;
115 last;
116 }
117 }
118 if (eof $fh) {
119 return;
120 }
121
122 my $line;
123 while ($line = <$fh>) {
124 if ($line =~ m/^\s*#\s*include\s+<linux\/types.h>/) {
125 $linux_types = 1;
126 last;
127 }
128 if (my $included = ($line =~ /^\s*#\s*include\s+[<"](\S+)[>"]/)[0]) {
129 check_include_typesh($included);
130 }
131 }
132 close $fh;
133 delete $import_stack{$import_path};
134}
135
Sam Ravnborg483b4122008-12-30 11:34:58 +0100136sub check_sizetypes
137{
Sam Ravnborgb67ff8c2008-12-31 09:32:30 +0100138 if ($filename =~ /types.h|int-l64.h|int-ll64.h/o) {
139 return;
140 }
Sam Ravnborg483b4122008-12-30 11:34:58 +0100141 if ($lineno == 1) {
142 $linux_types = 0;
143 } elsif ($linux_types >= 1) {
144 return;
145 }
146 if ($line =~ m/^\s*#\s*include\s+<linux\/types.h>/) {
147 $linux_types = 1;
148 return;
149 }
Bobby Powersf75a8df2012-03-05 15:08:09 -0800150 if (my $included = ($line =~ /^\s*#\s*include\s+[<"](\S+)[>"]/)[0]) {
151 check_include_typesh($included);
152 }
Sam Ravnborg483b4122008-12-30 11:34:58 +0100153 if ($line =~ m/__[us](8|16|32|64)\b/) {
154 printf STDERR "$filename:$lineno: " .
155 "found __[us]{8,16,32,64} type " .
156 "without #include <linux/types.h>\n";
157 $linux_types = 2;
158 # Warn until headers are all fixed
159 #$ret = 1;
160 }
161}