summaryrefslogtreecommitdiffstats
path: root/t/002goodperl.t
blob: b2a9d5751f7cfa3199f1c43329366a665cdc63db (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
# This Source Code Form is subject to the terms of the Mozilla Public
# License, v. 2.0. If a copy of the MPL was not distributed with this
# file, You can obtain one at http://mozilla.org/MPL/2.0/.
#
# This Source Code Form is "Incompatible With Secondary Licenses", as
# defined by the Mozilla Public License, v. 2.0.


#################
#Bugzilla Test 2#
####GoodPerl#####

use 5.10.1;
use strict;
use warnings;

use lib qw(. lib local/lib/perl5 t);

use Support::Files;

use Test::More tests =>
  (scalar(@Support::Files::testitems) + scalar(@Support::Files::test_files))
  * 6;

my @testitems = (@Support::Files::test_files, @Support::Files::testitems);
my @require_taint = qw(email_in.pl importxml.pl mod_perl.pl whine.pl);

foreach my $file (@testitems) {
  $file =~ s/\s.*$//;    # nuke everything after the first space (#comment)
  next if (!$file);      # skip null entries
  if (!open(FILE, $file)) {
    ok(0, "could not open $file --WARNING");
  }
  my $file_line1 = <FILE>;
  close(FILE);

  $file =~ m/.*\.(.*)/;
  my $ext = $1;

  if ($file_line1 !~ m/^#\!/) {
    ok(1, "$file does not have a shebang");
  }
  else {
    my $flags;
    if (!defined $ext || $ext eq 'pl' || $ext eq 'psgi') {

      # standalone programs aren't taint checked yet
      if (grep { $file eq $_ } @require_taint) {
        $flags = 'T';
      }
      else {
        $flags = '';
      }
    }
    elsif ($ext eq "pm") {
      ok(0, "$file is a module, but has a shebang");
      next;
    }
    elsif ($ext eq "cgi") {

      # cgi files must be taint checked
      $flags = 'T';
    }
    elsif ($ext eq 't') {
      $flags = '';
    }
    else {
      ok(0, "$file has shebang but unknown extension");
      next;
    }

    if ($file_line1 =~ m#^\#\!/usr/bin/perl(?:\s-(\w+))?$#) {
      my $file_flags = $1 || '';
      if ($flags eq $file_flags) {
        ok(1,
          "$file uses standard perl location" . ($flags ? " and -$flags flag" : ""));
      }
      elsif ($flags) {
        ok(0, "$file is MISSING -$flags flag --WARNING");
      }
      else {
        ok(0, "$file has unexpected -$file_flags flag --WARNING");
      }
    }
    else {
      ok(0, "$file uses non-standard perl location");
    }
  }
}

foreach my $file (@testitems) {
  my $found_use_perl     = 0;
  my $found_use_strict   = 0;
  my $found_use_warnings = 0;
  my $found_modern_perl  = 0;
  my $found_mojo         = 0;

  $file =~ s/\s.*$//;    # nuke everything after the first space (#comment)
  next if (!$file);      # skip null entries
  if (!open(FILE, $file)) {
    ok(0, "could not open $file --WARNING");
    next;
  }
  while (my $file_line = <FILE>) {
    $found_modern_perl = 1 if $file_line =~ m/^use\s*(?:Moo|Role::Tiny)/;
    $found_mojo = 1 if $file_line =~ m/^use\s(?:Mojo(?:licious::Lite|::Base)\b)/;
    $found_use_perl     = 1 if $file_line =~ m/^\s*use 5.10.1/;
    $found_use_strict   = 1 if $file_line =~ m/^\s*use strict/;
    $found_use_warnings = 1 if $file_line =~ m/^\s*use warnings/;
    if ($found_modern_perl || $found_mojo) {
      $found_use_strict   = 1;
      $found_use_warnings = 1;
    }
    if ($found_mojo) {
      $found_use_perl = 1;
    }
    last if ($found_use_perl && $found_use_strict && $found_use_warnings);
  }
  close(FILE);
  if ($found_use_perl) {
    ok(1, "$file requires Perl 5.10.1");
  }
  else {
    ok(0, "$file DOES NOT require Perl 5.10.1 --WARNING");
  }

  if ($found_use_strict) {
    ok(1, "$file uses strict");
  }
  else {
    ok(0, "$file DOES NOT use strict --WARNING");
  }

  if ($found_use_warnings) {
    ok(1, "$file uses warnings");
  }
  else {
    ok(0, "$file DOES NOT use warnings --WARNING");
  }
}

# Check to see that all error messages use tags (for l10n reasons.)
foreach my $file (@testitems) {
  $file =~ s/\s.*$//;    # nuke everything after the first space (#comment)
  next if (!$file);      # skip null entries
  if (!open(FILE, $file)) {
    ok(0, "could not open $file --WARNING");
    next;
  }
  my $lineno = 0;
  my $error  = 0;

  while (!$error && (my $file_line = <FILE>)) {
    $lineno++;
    if ($file_line =~ /Throw.*Error\("(.*?)"/) {
      if ($1 =~ /\s/) {
        ok(
          0, "$file has a Throw*Error call on line $lineno
                      which doesn't use a tag --ERROR"
        );
        $error = 1;
      }
    }
  }

  ok(1, "$file uses Throw*Error calls correctly") if !$error;

  close(FILE);
}

# Forbird the { foo => $cgi->param() } syntax, for security reasons.
foreach my $file (@testitems) {
  $file =~ s/\s.*$//;    # nuke everything after the first space (#comment)
  next unless $file;     # skip null entries
  if (!open(FILE, $file)) {
    ok(0, "could not open $file --WARNING");
    next;
  }
  my $lineno = 0;
  my @unsafe_args;

  while (my $file_line = <FILE>) {
    $lineno++;
    $file_line =~ s/^\s*(.+)\s*$/$1/;    # Remove leading and trailing whitespaces.
    if ($file_line =~ /^[^#]+=> \$cgi\->param/) {
      push(@unsafe_args, "$file_line on line $lineno");
    }
  }

  if (@unsafe_args) {
    ok(0,
      "$file incorrectly passes a CGI argument to a hash --ERROR\n"
        . join("\n", @unsafe_args));
  }
  else {
    ok(1, "$file has no vulnerable hash syntax");
  }

  close(FILE);
}

exit 0;