| line |
stmt |
bran |
cond |
sub |
pod |
time |
code |
|
1
|
|
|
|
|
|
|
package Taint; |
|
2
|
|
|
|
|
|
|
|
|
3
|
|
|
|
|
|
|
# See docs at end for author and copyright info |
|
4
|
|
|
|
|
|
|
|
|
5
|
|
|
|
|
|
|
=head1 NAME |
|
6
|
|
|
|
|
|
|
|
|
7
|
|
|
|
|
|
|
Taint - Perl utility extensions for tainted data |
|
8
|
|
|
|
|
|
|
|
|
9
|
|
|
|
|
|
|
=head1 SYNOPSIS |
|
10
|
|
|
|
|
|
|
|
|
11
|
|
|
|
|
|
|
use Taint; |
|
12
|
|
|
|
|
|
|
warn "Oops" |
|
13
|
|
|
|
|
|
|
if tainted $num, @ids; # Test for tainted data |
|
14
|
|
|
|
|
|
|
kill $num, @ids; # before using it |
|
15
|
|
|
|
|
|
|
|
|
16
|
|
|
|
|
|
|
use Carp; |
|
17
|
|
|
|
|
|
|
use Taint; |
|
18
|
|
|
|
|
|
|
sub baz { croak "Insecure request" if tainted @_; ... } |
|
19
|
|
|
|
|
|
|
|
|
20
|
|
|
|
|
|
|
use Taint qw(taint); |
|
21
|
|
|
|
|
|
|
taint @list, $item; # Intentionally taint data |
|
22
|
|
|
|
|
|
|
|
|
23
|
|
|
|
|
|
|
use Taint qw(:ALL); |
|
24
|
|
|
|
|
|
|
$pi = 3.14159 + tainted_zero; # I don't trust irrational numbers |
|
25
|
|
|
|
|
|
|
|
|
26
|
|
|
|
|
|
|
=head1 DESCRIPTION |
|
27
|
|
|
|
|
|
|
|
|
28
|
|
|
|
|
|
|
Perl has the ability to mark data as 'tainted', as described in |
|
29
|
|
|
|
|
|
|
L. Perl will prevent tainted data from being used for |
|
30
|
|
|
|
|
|
|
some operations, and you may wish to add such caution to your own |
|
31
|
|
|
|
|
|
|
code. The routines in this module provide convenient ways to taint |
|
32
|
|
|
|
|
|
|
data and to check data for taint. To remove the taint from data, |
|
33
|
|
|
|
|
|
|
use the method described in L, or use the make_extractor |
|
34
|
|
|
|
|
|
|
routine. |
|
35
|
|
|
|
|
|
|
|
|
36
|
|
|
|
|
|
|
Please read L and L. |
|
37
|
|
|
|
|
|
|
|
|
38
|
|
|
|
|
|
|
=head1 ROUTINES |
|
39
|
|
|
|
|
|
|
|
|
40
|
|
|
|
|
|
|
=over 5 |
|
41
|
|
|
|
|
|
|
|
|
42
|
|
|
|
|
|
|
=cut |
|
43
|
|
|
|
|
|
|
|
|
44
|
|
|
|
|
|
|
require 5.004; |
|
45
|
4
|
|
|
4
|
|
2703
|
use strict; |
|
|
4
|
|
|
|
|
6
|
|
|
|
4
|
|
|
|
|
154
|
|
|
46
|
4
|
|
|
|
|
3801
|
use vars qw( |
|
47
|
|
|
|
|
|
|
$VERSION |
|
48
|
|
|
|
|
|
|
$DEBUGGING |
|
49
|
|
|
|
|
|
|
@ISA |
|
50
|
|
|
|
|
|
|
@EXPORT @EXPORT_OK %EXPORT_TAGS |
|
51
|
4
|
|
|
4
|
|
34
|
); |
|
|
4
|
|
|
|
|
6
|
|
|
52
|
|
|
|
|
|
|
my %insanity; |
|
53
|
|
|
|
|
|
|
my %no_taint_okay; |
|
54
|
|
|
|
|
|
|
|
|
55
|
|
|
|
|
|
|
require Exporter; |
|
56
|
|
|
|
|
|
|
|
|
57
|
|
|
|
|
|
|
@ISA = qw(Exporter); |
|
58
|
|
|
|
|
|
|
|
|
59
|
|
|
|
|
|
|
@EXPORT = qw(tainted); |
|
60
|
|
|
|
|
|
|
|
|
61
|
|
|
|
|
|
|
@EXPORT_OK = qw( |
|
62
|
|
|
|
|
|
|
taint |
|
63
|
|
|
|
|
|
|
is_tainted any_tainted all_tainted taintedness |
|
64
|
|
|
|
|
|
|
make_extractor |
|
65
|
|
|
|
|
|
|
tainted_null tainted_zero taint_checking |
|
66
|
|
|
|
|
|
|
); |
|
67
|
|
|
|
|
|
|
|
|
68
|
|
|
|
|
|
|
# Installer's option: |
|
69
|
|
|
|
|
|
|
# Use 1 for normal operation, 0 to disable the ability to |
|
70
|
|
|
|
|
|
|
# use the unconditional untainting code from this module. |
|
71
|
|
|
|
|
|
|
# Edit with care: This is a machine-editable line. |
|
72
|
|
|
|
|
|
|
sub allowing_insanity () { 1 } # Default is 1. |
|
73
|
|
|
|
|
|
|
# This constant sub is for internal (testing) use only. |
|
74
|
|
|
|
|
|
|
# It's not documented or intended for outside use. |
|
75
|
|
|
|
|
|
|
|
|
76
|
|
|
|
|
|
|
# The pseudo-tag ALL does not include unconditional_untaint. That |
|
77
|
|
|
|
|
|
|
# must be explicitly imported, in a special way. Don't bother. Use |
|
78
|
|
|
|
|
|
|
# the untainting methods described in the perlsec(1) manpage, or use |
|
79
|
|
|
|
|
|
|
# make_extractor. |
|
80
|
|
|
|
|
|
|
|
|
81
|
|
|
|
|
|
|
%EXPORT_TAGS = ( ALL => [ @EXPORT, @EXPORT_OK ] ); |
|
82
|
|
|
|
|
|
|
|
|
83
|
|
|
|
|
|
|
$VERSION = '0.09'; |
|
84
|
|
|
|
|
|
|
|
|
85
|
|
|
|
|
|
|
BEGIN { |
|
86
|
4
|
|
|
4
|
|
8
|
my $saved_warnings; |
|
87
|
|
|
|
|
|
|
BEGIN { |
|
88
|
4
|
|
|
4
|
|
14
|
$saved_warnings = $^W; |
|
89
|
4
|
|
|
|
|
324
|
$^W = 0; # No warnings while compiling this sub |
|
90
|
|
|
|
|
|
|
} |
|
91
|
4
|
|
|
|
|
3539
|
$^W = $saved_warnings; |
|
92
|
|
|
|
|
|
|
|
|
93
|
|
|
|
|
|
|
# A note to the worried, curious, or paranoid: |
|
94
|
|
|
|
|
|
|
# |
|
95
|
|
|
|
|
|
|
# This sub does _not_ actually kill anything. The signal |
|
96
|
|
|
|
|
|
|
# 0 is actually a fake signal which doesn't get sent, and |
|
97
|
|
|
|
|
|
|
# which wouldn't do anything if it were sent. And besides, |
|
98
|
|
|
|
|
|
|
# we never send it anywhere, since there are no process ids |
|
99
|
|
|
|
|
|
|
# being passed to kill. |
|
100
|
|
|
|
|
|
|
# |
|
101
|
|
|
|
|
|
|
# Here's how it works: |
|
102
|
|
|
|
|
|
|
# First, join unites the arguments, then they are silently |
|
103
|
|
|
|
|
|
|
# discarded by the comma operator. Next, Perl tries to do |
|
104
|
|
|
|
|
|
|
# a harmless kill 0. Kill refuses to work if there are any |
|
105
|
|
|
|
|
|
|
# tainted data being used in the same statement. So, either |
|
106
|
|
|
|
|
|
|
# the eval aborts (returning undef), or it succeeds, and |
|
107
|
|
|
|
|
|
|
# returns 1. That return value is inverted by the not |
|
108
|
|
|
|
|
|
|
# operator, thus making the function return value. Ta da! |
|
109
|
|
|
|
|
|
|
# |
|
110
|
|
|
|
|
|
|
sub any_tainted (@) { |
|
111
|
129
|
|
|
129
|
1
|
654
|
local(@_, $@, $^W) = @_; # Prevent errors, stringifying |
|
112
|
129
|
|
|
|
|
445
|
not eval { join("",@_), kill 0; 1 }; |
|
|
129
|
|
|
|
|
3573
|
|
|
|
40
|
|
|
|
|
802
|
|
|
113
|
|
|
|
|
|
|
} |
|
114
|
|
|
|
|
|
|
} |
|
115
|
|
|
|
|
|
|
|
|
116
|
|
|
|
|
|
|
# Just a different prototype |
|
117
|
|
|
|
|
|
|
sub is_tainted ($) { |
|
118
|
101
|
|
|
101
|
1
|
411
|
goto &any_tainted; |
|
119
|
|
|
|
|
|
|
} |
|
120
|
|
|
|
|
|
|
|
|
121
|
|
|
|
|
|
|
sub all_tainted (@) { |
|
122
|
10
|
100
|
|
10
|
1
|
25
|
for (@_) { return unless is_tainted $_ } |
|
|
35
|
|
|
|
|
66
|
|
|
123
|
5
|
|
|
|
|
20
|
1; |
|
124
|
|
|
|
|
|
|
} |
|
125
|
|
|
|
|
|
|
|
|
126
|
|
|
|
|
|
|
sub tainted (@) { |
|
127
|
11
|
|
|
11
|
1
|
745
|
goto &any_tainted; |
|
128
|
|
|
|
|
|
|
} |
|
129
|
|
|
|
|
|
|
|
|
130
|
|
|
|
|
|
|
=item tainted LIST |
|
131
|
|
|
|
|
|
|
|
|
132
|
|
|
|
|
|
|
=item is_tainted EXPR |
|
133
|
|
|
|
|
|
|
|
|
134
|
|
|
|
|
|
|
=item any_tainted LIST |
|
135
|
|
|
|
|
|
|
|
|
136
|
|
|
|
|
|
|
=item all_tainted LIST |
|
137
|
|
|
|
|
|
|
|
|
138
|
|
|
|
|
|
|
Test one or more items for taint. C is an alias for |
|
139
|
|
|
|
|
|
|
C, provided for convenience. (Also, C is |
|
140
|
|
|
|
|
|
|
exported by default.) C is prototyped to take a B
|
|
141
|
|
|
|
|
|
|
scalar> argument, the others take lists. (If you're not sure which |
|
142
|
|
|
|
|
|
|
one to use, use C.) When taint checks are off, these always |
|
143
|
|
|
|
|
|
|
return false. |
|
144
|
|
|
|
|
|
|
|
|
145
|
|
|
|
|
|
|
=cut |
|
146
|
|
|
|
|
|
|
|
|
147
|
|
|
|
|
|
|
sub taintedness (@) { |
|
148
|
|
|
|
|
|
|
# Could do this with C, |
|
149
|
|
|
|
|
|
|
# but that's buggy through 5.004_03. |
|
150
|
5
|
100
|
|
5
|
1
|
16
|
any_tainted(@_) ? tainted_null() : ''; |
|
151
|
|
|
|
|
|
|
} |
|
152
|
|
|
|
|
|
|
|
|
153
|
|
|
|
|
|
|
=item taintedness LIST |
|
154
|
|
|
|
|
|
|
|
|
155
|
|
|
|
|
|
|
This is a utility function, mostly useful for authors of subroutines |
|
156
|
|
|
|
|
|
|
in modules. It is possible that an algorithm, by its nature, doesn't |
|
157
|
|
|
|
|
|
|
propagate taintedness as it should. This routine returns the |
|
158
|
|
|
|
|
|
|
taintedness of its parameters in the form of a null string which |
|
159
|
|
|
|
|
|
|
is either tainted or not. (When taint checking is off, the return |
|
160
|
|
|
|
|
|
|
value is always an untainted null string.) That string may be (for |
|
161
|
|
|
|
|
|
|
example) appended to a return value to taint it if needed. |
|
162
|
|
|
|
|
|
|
|
|
163
|
|
|
|
|
|
|
sub frobnicate { |
|
164
|
|
|
|
|
|
|
my($taintedness) = taintedness @_; # save it |
|
165
|
|
|
|
|
|
|
# ...do some stuff which may or may not |
|
166
|
|
|
|
|
|
|
# properly propagate taint... |
|
167
|
|
|
|
|
|
|
return undef if $you_want_to; |
|
168
|
|
|
|
|
|
|
return $taintedness . $return_value; # restore it |
|
169
|
|
|
|
|
|
|
} |
|
170
|
|
|
|
|
|
|
|
|
171
|
|
|
|
|
|
|
=cut |
|
172
|
|
|
|
|
|
|
|
|
173
|
|
|
|
|
|
|
BEGIN { |
|
174
|
|
|
|
|
|
|
# Before anything else, we need to get a little |
|
175
|
|
|
|
|
|
|
# taint on our taintbrush. |
|
176
|
4
|
|
|
4
|
|
8
|
my $TAINT; |
|
177
|
|
|
|
|
|
|
{ |
|
178
|
|
|
|
|
|
|
# Let's try the easy way first. Either of these should be |
|
179
|
|
|
|
|
|
|
# tainted, unless somebody has untainted them, so this |
|
180
|
|
|
|
|
|
|
# will almost always work on the first try. |
|
181
|
|
|
|
|
|
|
# (Unless, of course, taint checking has been turned off!) |
|
182
|
4
|
|
|
|
|
14
|
$TAINT = substr("$0$^X", 0, 0); |
|
|
4
|
|
|
|
|
33
|
|
|
183
|
4
|
100
|
|
|
|
16
|
last if is_tainted $TAINT; |
|
184
|
|
|
|
|
|
|
|
|
185
|
|
|
|
|
|
|
# Let's try again. Maybe somebody cleaned those. |
|
186
|
1
|
|
|
|
|
15
|
$TAINT = substr(join("", @ARGV, %ENV), 0, 0); |
|
187
|
1
|
50
|
|
|
|
4
|
last if is_tainted $TAINT; |
|
188
|
|
|
|
|
|
|
|
|
189
|
|
|
|
|
|
|
# Oh, a wise guy, eh? |
|
190
|
1
|
|
|
|
|
2
|
local(*FOO); |
|
191
|
1
|
|
|
|
|
2
|
my $data = ''; |
|
192
|
1
|
|
|
|
|
22
|
for (qw(/dev/null / . ..), values %INC, $0, $^X) { |
|
193
|
|
|
|
|
|
|
# Why so many? Maybe a file was just deleted or moved; |
|
194
|
|
|
|
|
|
|
# you never know! :-) At this point, taint checks |
|
195
|
|
|
|
|
|
|
# are probably off anyway, but this is the ironclad |
|
196
|
|
|
|
|
|
|
# way to get tainted data if it's possible. |
|
197
|
|
|
|
|
|
|
# (Yes, even reading from /dev/null works!) |
|
198
|
|
|
|
|
|
|
# |
|
199
|
1
|
50
|
33
|
|
|
94
|
last if open FOO, $_ |
|
200
|
|
|
|
|
|
|
and defined sysread FOO, $data, 1 |
|
201
|
|
|
|
|
|
|
} |
|
202
|
|
|
|
|
|
|
# Assume one of them succeeded. We need only one! |
|
203
|
1
|
|
|
|
|
2
|
$TAINT = substr($data, 0, 0); |
|
204
|
1
|
|
|
|
|
11
|
close FOO; |
|
205
|
|
|
|
|
|
|
} |
|
206
|
|
|
|
|
|
|
|
|
207
|
|
|
|
|
|
|
# Sanity check |
|
208
|
4
|
50
|
|
|
|
37
|
die "Internal error. Oops!" if length $TAINT; |
|
209
|
|
|
|
|
|
|
|
|
210
|
|
|
|
|
|
|
# A tainted zero |
|
211
|
4
|
|
|
|
|
25
|
my $TAINT0 = 0+"0$TAINT"; |
|
212
|
|
|
|
|
|
|
|
|
213
|
|
|
|
|
|
|
sub taint (@) { |
|
214
|
11
|
50
|
|
11
|
1
|
320
|
return unless taint_checking(); |
|
215
|
11
|
|
|
|
|
30
|
for (@_) { |
|
216
|
34
|
100
|
|
|
|
68
|
next if not defined; |
|
217
|
33
|
50
|
|
|
|
134
|
next if ref; |
|
218
|
|
|
|
|
|
|
# Taint tied objects by method, if possible |
|
219
|
33
|
100
|
|
|
|
129
|
if (defined(my $thingy = tied $_)) { |
|
220
|
2
|
50
|
|
|
|
19
|
if ($thingy->can('TAINT')) { |
|
221
|
2
|
|
|
|
|
7
|
$thingy->TAINT(1); |
|
222
|
2
|
|
|
|
|
19
|
next; |
|
223
|
|
|
|
|
|
|
} |
|
224
|
|
|
|
|
|
|
} |
|
225
|
31
|
|
|
|
|
78
|
eval { |
|
226
|
31
|
100
|
|
|
|
72
|
if ( not $_ & '00' | '00' ) { |
|
227
|
|
|
|
|
|
|
# Must already be a number, |
|
228
|
|
|
|
|
|
|
# so don't stringify it now |
|
229
|
23
|
|
|
|
|
46
|
$_ += $TAINT0; |
|
230
|
|
|
|
|
|
|
} else { |
|
231
|
8
|
|
|
|
|
20
|
$_ .= $TAINT; |
|
232
|
|
|
|
|
|
|
} |
|
233
|
|
|
|
|
|
|
}; |
|
234
|
31
|
50
|
|
|
|
158
|
if ($@ =~ /read-only/) { |
|
|
|
50
|
|
|
|
|
|
|
235
|
0
|
|
|
|
|
0
|
require Carp; |
|
236
|
0
|
|
|
|
|
0
|
&Carp::carp("Attempt to taint read-only value"); |
|
237
|
|
|
|
|
|
|
} elsif ($@) { |
|
238
|
0
|
|
|
|
|
0
|
require Carp; |
|
239
|
0
|
|
|
|
|
0
|
&Carp::carp("Unexpected error: $@"); |
|
240
|
|
|
|
|
|
|
} |
|
241
|
|
|
|
|
|
|
} |
|
242
|
11
|
|
|
|
|
28
|
return; # explicitly, no return value |
|
243
|
|
|
|
|
|
|
} |
|
244
|
|
|
|
|
|
|
|
|
245
|
|
|
|
|
|
|
=item taint LIST |
|
246
|
|
|
|
|
|
|
|
|
247
|
|
|
|
|
|
|
If taint checks are turned on, marks each (apparently) taintable |
|
248
|
|
|
|
|
|
|
argument in LIST as being tainted. (References and C are |
|
249
|
|
|
|
|
|
|
never taintable and are left unchanged. Some Cd and magical |
|
250
|
|
|
|
|
|
|
variables may fail to be tainted by this routine, try as it may.) |
|
251
|
|
|
|
|
|
|
|
|
252
|
|
|
|
|
|
|
To taint (the values of) an entire hash, use this idiom. |
|
253
|
|
|
|
|
|
|
|
|
254
|
|
|
|
|
|
|
taint @hash{ keys %hash }; # taint values of %hash |
|
255
|
|
|
|
|
|
|
|
|
256
|
|
|
|
|
|
|
=cut |
|
257
|
|
|
|
|
|
|
|
|
258
|
|
|
|
|
|
|
# The following subs are inlineable constants |
|
259
|
|
|
|
|
|
|
# because their values have no outside refs |
|
260
|
|
|
|
|
|
|
# (That's why the extra scopes.) |
|
261
|
|
|
|
|
|
|
{ |
|
262
|
4
|
|
|
|
|
15
|
my $taint = $TAINT; |
|
|
4
|
|
|
|
|
16
|
|
|
263
|
16
|
|
|
16
|
1
|
346
|
sub tainted_null () { $taint } # a tainted null string |
|
264
|
|
|
|
|
|
|
} |
|
265
|
|
|
|
|
|
|
{ |
|
266
|
4
|
|
|
|
|
6
|
my $taint = $TAINT0; |
|
|
4
|
|
|
|
|
25
|
|
|
267
|
3
|
|
|
3
|
1
|
16
|
sub tainted_zero () { $taint } # a tainted zero |
|
268
|
|
|
|
|
|
|
} |
|
269
|
|
|
|
|
|
|
|
|
270
|
|
|
|
|
|
|
=item tainted_null |
|
271
|
|
|
|
|
|
|
|
|
272
|
|
|
|
|
|
|
=item tainted_zero |
|
273
|
|
|
|
|
|
|
|
|
274
|
|
|
|
|
|
|
If you'd rather taint your data yourself, these constants will let |
|
275
|
|
|
|
|
|
|
you do it. C is a tainted null string, which may be |
|
276
|
|
|
|
|
|
|
appended to any data to taint it. (Of course, that will also |
|
277
|
|
|
|
|
|
|
stringify the data, if needed.) C is (surprise) a |
|
278
|
|
|
|
|
|
|
tainted zero, which may be added to any number to taint it. Note |
|
279
|
|
|
|
|
|
|
that when taint checking is off, nothing can be tainted, so then |
|
280
|
|
|
|
|
|
|
these are merely mundane C<''> and C<0> values. |
|
281
|
|
|
|
|
|
|
|
|
282
|
|
|
|
|
|
|
=cut |
|
283
|
|
|
|
|
|
|
|
|
284
|
|
|
|
|
|
|
# This one is inlineable as well |
|
285
|
|
|
|
|
|
|
{ |
|
286
|
4
|
|
|
|
|
9
|
my $taint_checking = is_tainted $TAINT; |
|
|
4
|
|
|
|
|
10
|
|
|
287
|
13
|
|
|
13
|
1
|
91
|
sub taint_checking () { $taint_checking } |
|
288
|
|
|
|
|
|
|
} |
|
289
|
|
|
|
|
|
|
|
|
290
|
|
|
|
|
|
|
=item taint_checking |
|
291
|
|
|
|
|
|
|
|
|
292
|
|
|
|
|
|
|
This constant tells whether taint checks are in use. This is |
|
293
|
|
|
|
|
|
|
usually only useful in connection with the allow_no_taint option |
|
294
|
|
|
|
|
|
|
(see L). |
|
295
|
|
|
|
|
|
|
|
|
296
|
|
|
|
|
|
|
print LOG "Warning: Taint checks not enabled\n" |
|
297
|
|
|
|
|
|
|
unless taint_checking; |
|
298
|
|
|
|
|
|
|
|
|
299
|
|
|
|
|
|
|
=cut |
|
300
|
|
|
|
|
|
|
|
|
301
|
|
|
|
|
|
|
} |
|
302
|
|
|
|
|
|
|
|
|
303
|
|
|
|
|
|
|
# Private stuff for _display_pattern |
|
304
|
|
|
|
|
|
|
{ |
|
305
|
|
|
|
|
|
|
my @map; # for converting a pattern to |
|
306
|
|
|
|
|
|
|
# the usual form, more or less. |
|
307
|
|
|
|
|
|
|
sub _display_pattern ($) { |
|
308
|
1
|
|
|
1
|
|
2
|
my $pattern = shift; |
|
309
|
|
|
|
|
|
|
# Make the map, if we have to |
|
310
|
1
|
50
|
|
|
|
4
|
unless (@map) { |
|
311
|
1
|
|
|
|
|
5
|
for (0..0x1f, 0x7f..0xff) { # defaults |
|
312
|
161
|
|
|
|
|
512
|
$map[$_] = '\\x' . sprintf '%02x', $_; |
|
313
|
|
|
|
|
|
|
} |
|
314
|
1
|
|
|
|
|
5
|
for (0x20..0x7e) { # printables |
|
315
|
95
|
|
|
|
|
226
|
$map[$_] = chr; |
|
316
|
|
|
|
|
|
|
} |
|
317
|
1
|
|
|
|
|
3
|
$map[ord("\n")] = '\\n'; |
|
318
|
1
|
|
|
|
|
2
|
$map[ord("\t")] = '\\t'; |
|
319
|
1
|
|
|
|
|
3
|
for (qw( / $ @ )) { # backwhackables |
|
320
|
3
|
|
|
|
|
11
|
$map[ord] = '\\' . $_; |
|
321
|
|
|
|
|
|
|
} |
|
322
|
|
|
|
|
|
|
} |
|
323
|
|
|
|
|
|
|
# We want to display the poor user's pattern in the way |
|
324
|
|
|
|
|
|
|
# they're used to seeing it... |
|
325
|
|
|
|
|
|
|
# ...more or less. If this prints out '\-', that might |
|
326
|
|
|
|
|
|
|
# not do what a real \- would. But there's no way to be |
|
327
|
|
|
|
|
|
|
# sure to get it right, really, without parsing the |
|
328
|
|
|
|
|
|
|
# (possibly invalid) regexp. :-( |
|
329
|
1
|
|
|
|
|
22
|
my $copy = |
|
330
|
|
|
|
|
|
|
join '', # Glue together |
|
331
|
|
|
|
|
|
|
map $map[ord], # a string representing |
|
332
|
|
|
|
|
|
|
split //, # each character |
|
333
|
|
|
|
|
|
|
$pattern; # in the pattern |
|
334
|
1
|
|
|
|
|
11
|
require Carp; |
|
335
|
1
|
|
|
|
|
268
|
&Carp::carp("Pattern was /$copy/o"); |
|
336
|
|
|
|
|
|
|
} |
|
337
|
|
|
|
|
|
|
} |
|
338
|
|
|
|
|
|
|
|
|
339
|
|
|
|
|
|
|
sub make_extractor ($) { |
|
340
|
4
|
|
|
4
|
1
|
85
|
my $pattern = shift; |
|
341
|
|
|
|
|
|
|
# We could allow $pattern to be tainted, but we shouldn't. |
|
342
|
|
|
|
|
|
|
# (The contents of $pattern can't break anything, even |
|
343
|
|
|
|
|
|
|
# if it's not a valid regexp. It may die, but not break.) |
|
344
|
4
|
50
|
|
|
|
10
|
if (is_tainted $pattern) { |
|
345
|
0
|
|
|
|
|
0
|
require Carp; |
|
346
|
0
|
|
|
|
|
0
|
&Carp::croak("Can't make code from tainted string '$pattern'"); |
|
347
|
|
|
|
|
|
|
} |
|
348
|
4
|
100
|
|
|
|
16
|
_display_pattern $pattern if $DEBUGGING; |
|
349
|
4
|
|
|
|
|
662
|
my $sub = eval q{ # Yes, a single-quote eval! |
|
350
|
|
|
|
|
|
|
my $sub = sub { |
|
351
|
|
|
|
|
|
|
my @list; |
|
352
|
|
|
|
|
|
|
for (@_) { |
|
353
|
|
|
|
|
|
|
push @list, ($_ =~ /$pattern/o); |
|
354
|
|
|
|
|
|
|
} |
|
355
|
|
|
|
|
|
|
wantarray ? @list : $list[0]; # return value |
|
356
|
|
|
|
|
|
|
}; |
|
357
|
|
|
|
|
|
|
&$sub('dummy parameter'); # catch bad patterns |
|
358
|
|
|
|
|
|
|
$sub; # return value from eval |
|
359
|
|
|
|
|
|
|
}; |
|
360
|
4
|
100
|
|
|
|
21
|
if ($@) { |
|
361
|
1
|
|
|
|
|
5
|
$@ =~ s/ at \(eval \d+\) line \d+\.\n?$//; |
|
362
|
1
|
|
|
|
|
7
|
require Carp; |
|
363
|
1
|
|
|
|
|
176
|
&Carp::croak($@); |
|
364
|
|
|
|
|
|
|
} |
|
365
|
3
|
|
|
|
|
13
|
$sub; # return value |
|
366
|
|
|
|
|
|
|
} |
|
367
|
|
|
|
|
|
|
|
|
368
|
|
|
|
|
|
|
=item make_extractor EXPR |
|
369
|
|
|
|
|
|
|
|
|
370
|
|
|
|
|
|
|
This routine returns a coderef for a subroutine which untaints its |
|
371
|
|
|
|
|
|
|
arguments according to the pattern passed in the string EXPR. |
|
372
|
|
|
|
|
|
|
Although the argument to this routine must be untainted, the |
|
373
|
|
|
|
|
|
|
arguments to the generated code may be tainted or not. When taint |
|
374
|
|
|
|
|
|
|
checking is off, this routine and its generated code behave in |
|
375
|
|
|
|
|
|
|
essentially the same way, even though neither their parameters nor |
|
376
|
|
|
|
|
|
|
return values are tainted. |
|
377
|
|
|
|
|
|
|
|
|
378
|
|
|
|
|
|
|
B: When untainting data, it's often easier to use the method |
|
379
|
|
|
|
|
|
|
described in L, especially if you're unfamiliar with |
|
380
|
|
|
|
|
|
|
constructing strings to be used as regular expressions. |
|
381
|
|
|
|
|
|
|
|
|
382
|
|
|
|
|
|
|
Here's one way this routine might be used. This example is part of |
|
383
|
|
|
|
|
|
|
a server (similar in some ways to B; see L) |
|
384
|
|
|
|
|
|
|
which, when given a username, runs the Unix C command, extracts |
|
385
|
|
|
|
|
|
|
and untaints some information about that user, and reports it. Note |
|
386
|
|
|
|
|
|
|
that the regular expression is compiled just once, (within the |
|
387
|
|
|
|
|
|
|
C routine) even though the username may change |
|
388
|
|
|
|
|
|
|
every time through the main loop. |
|
389
|
|
|
|
|
|
|
|
|
390
|
|
|
|
|
|
|
while () { # The server runs in an infinite loop |
|
391
|
|
|
|
|
|
|
my $username = &get_next_request; |
|
392
|
|
|
|
|
|
|
# $username must already be untainted! (But let's not |
|
393
|
|
|
|
|
|
|
# assume it doesn't have metacharacters, even though |
|
394
|
|
|
|
|
|
|
# Unix usernames can't have any.) |
|
395
|
|
|
|
|
|
|
my $pattern = |
|
396
|
|
|
|
|
|
|
'^' . |
|
397
|
|
|
|
|
|
|
quotemeta($username) . |
|
398
|
|
|
|
|
|
|
'\s+(\S+)\s+(.+)$'; |
|
399
|
|
|
|
|
|
|
my $get_who = make_extractor $pattern; |
|
400
|
|
|
|
|
|
|
|
|
401
|
|
|
|
|
|
|
my %info = (); |
|
402
|
|
|
|
|
|
|
for (`who`) { |
|
403
|
|
|
|
|
|
|
# $_ has lines of tainted information |
|
404
|
|
|
|
|
|
|
my($tty, $date) = &$get_who($_); |
|
405
|
|
|
|
|
|
|
# but $tty and $date are untainted |
|
406
|
|
|
|
|
|
|
$info{$tty} = $date; |
|
407
|
|
|
|
|
|
|
} |
|
408
|
|
|
|
|
|
|
# %info now has untainted information |
|
409
|
|
|
|
|
|
|
... |
|
410
|
|
|
|
|
|
|
} |
|
411
|
|
|
|
|
|
|
|
|
412
|
|
|
|
|
|
|
Any items which need to be extracted should be within memory parens. |
|
413
|
|
|
|
|
|
|
Because of that, the string should normally have at least one set |
|
414
|
|
|
|
|
|
|
of memory parens. The pattern will be applied to each of the |
|
415
|
|
|
|
|
|
|
arguments in turn, returning a list of all matched items in memory |
|
416
|
|
|
|
|
|
|
parens. Any arguments which fail to match will add no items to the |
|
417
|
|
|
|
|
|
|
list. If called in a scalar context, the generated sub will return |
|
418
|
|
|
|
|
|
|
just the first untainted item in the list. No locale is used; see |
|
419
|
|
|
|
|
|
|
L. |
|
420
|
|
|
|
|
|
|
|
|
421
|
|
|
|
|
|
|
Note that the pattern may need to be written a little differently |
|
422
|
|
|
|
|
|
|
than usual, since it's going to be passed as a string. For example, |
|
423
|
|
|
|
|
|
|
it's not necessary to backwhack forward slashes in the pattern, |
|
424
|
|
|
|
|
|
|
since those aren't regexp metacharacters. Also, if the pattern is |
|
425
|
|
|
|
|
|
|
built up in an expression, it's important that the components all |
|
426
|
|
|
|
|
|
|
be untainted! And, of course, it needs to be a valid regular |
|
427
|
|
|
|
|
|
|
expression; otherwise, it causes an immediate error which may |
|
428
|
|
|
|
|
|
|
be trapped with C. |
|
429
|
|
|
|
|
|
|
|
|
430
|
|
|
|
|
|
|
For a case-insensitive match, which would usually be indicated with |
|
431
|
|
|
|
|
|
|
the C modifier, use the embedded C<(?i)> modifier, as described |
|
432
|
|
|
|
|
|
|
in L. The other embeddable modifiers also work. |
|
433
|
|
|
|
|
|
|
|
|
434
|
|
|
|
|
|
|
If the pattern contains backslashes, as many do, it is especially |
|
435
|
|
|
|
|
|
|
problematic. For example, these attempts to make a pattern aren't |
|
436
|
|
|
|
|
|
|
doing what they might look like. |
|
437
|
|
|
|
|
|
|
|
|
438
|
|
|
|
|
|
|
$pattern1 = "(\w+)"; # effectively /(w+)/ |
|
439
|
|
|
|
|
|
|
|
|
440
|
|
|
|
|
|
|
$pattern2 = '\Q' . $foo; # doesn't use quotemeta |
|
441
|
|
|
|
|
|
|
|
|
442
|
|
|
|
|
|
|
Usually, though, single quotes will do what you expect (and double |
|
443
|
|
|
|
|
|
|
quotes will confuse you). To help in debugging, you may set |
|
444
|
|
|
|
|
|
|
C<$Taint::DEBUGGING = 1> before calling make_extractor, which will |
|
445
|
|
|
|
|
|
|
produce an allegedly-helpful debugging message as a warning. This |
|
446
|
|
|
|
|
|
|
message will have a form of the regular expression passed, like |
|
447
|
|
|
|
|
|
|
C(w+)/> for C<$pattern1> above. |
|
448
|
|
|
|
|
|
|
|
|
449
|
|
|
|
|
|
|
=cut |
|
450
|
|
|
|
|
|
|
|
|
451
|
|
|
|
|
|
|
sub import { |
|
452
|
6
|
|
|
6
|
|
40
|
my $class = shift; |
|
453
|
6
|
|
|
|
|
7
|
my @importables; |
|
454
|
6
|
|
|
|
|
221
|
my $pkg = caller; |
|
455
|
6
|
|
|
|
|
14
|
for (@_) { |
|
456
|
7
|
100
|
|
|
|
368
|
if ($_ eq 'unconditional_untaint') { |
|
|
|
100
|
|
|
|
|
|
|
457
|
1
|
50
|
|
|
|
6
|
unless ($insanity{$pkg}) { |
|
458
|
0
|
|
|
|
|
0
|
require Carp; |
|
459
|
0
|
|
|
|
|
0
|
&Carp::croak("Wrong way to import unconditional_untaint()"); |
|
460
|
|
|
|
|
|
|
} |
|
461
|
1
|
|
|
|
|
191
|
my $name = "${pkg}::unconditional_untaint"; |
|
462
|
4
|
|
|
4
|
|
31
|
no strict 'refs'; |
|
|
4
|
|
|
|
|
22
|
|
|
|
4
|
|
|
|
|
1691
|
|
|
463
|
1
|
50
|
|
|
|
8
|
if (defined &$name) { |
|
464
|
0
|
|
|
|
|
0
|
require Carp; |
|
465
|
0
|
|
|
|
|
0
|
&Carp::croak("Can't redefine &$name"); |
|
466
|
|
|
|
|
|
|
} |
|
467
|
|
|
|
|
|
|
# Okay, you want it, you got it. |
|
468
|
1
|
|
|
|
|
7
|
*{$name} = sub { |
|
469
|
|
|
|
|
|
|
# |
|
470
|
|
|
|
|
|
|
# This routine is provided on the long-established |
|
471
|
|
|
|
|
|
|
# Perlian principle that, if you really want it, you |
|
472
|
|
|
|
|
|
|
# should always be given enough rope to shoot yourself |
|
473
|
|
|
|
|
|
|
# in the foot. |
|
474
|
|
|
|
|
|
|
# |
|
475
|
|
|
|
|
|
|
# Besides, if this routine wasn't here, some fool would |
|
476
|
|
|
|
|
|
|
# write it up, do it badly, document it worsely, and |
|
477
|
|
|
|
|
|
|
# then print it in a book which would continue to |
|
478
|
|
|
|
|
|
|
# haunt us for the next decade. (It's happened |
|
479
|
|
|
|
|
|
|
# before. Remember 'getgrid'? And the bad methods |
|
480
|
|
|
|
|
|
|
# some books still use instead of using CGI.pm?) |
|
481
|
|
|
|
|
|
|
# |
|
482
|
|
|
|
|
|
|
# If you really want to use this, you lunatic, first put |
|
483
|
|
|
|
|
|
|
# "no Taint 'sanity';" into your code. This will show |
|
484
|
|
|
|
|
|
|
# other programmers that you have an odd number of bits |
|
485
|
|
|
|
|
|
|
# per byte, and they will shun you. |
|
486
|
|
|
|
|
|
|
# |
|
487
|
|
|
|
|
|
|
# You have been warned. |
|
488
|
|
|
|
|
|
|
# |
|
489
|
|
|
|
|
|
|
# (If you haven't heard by now, the real way to untaint |
|
490
|
|
|
|
|
|
|
# is described in the perlsec man pages. Doing it this |
|
491
|
|
|
|
|
|
|
# way is foolish. There's no point in using taint |
|
492
|
|
|
|
|
|
|
# checking at all if you'll do things like this. But, |
|
493
|
|
|
|
|
|
|
# hey, it's your funeral.) |
|
494
|
|
|
|
|
|
|
# |
|
495
|
|
|
|
|
|
|
# On the other hand, if you've gotten this far, maybe |
|
496
|
|
|
|
|
|
|
# you should consider a different line of work, such |
|
497
|
|
|
|
|
|
|
# as a opening a turnip-polishing franchise |
|
498
|
|
|
|
|
|
|
# or becoming a galley slave. |
|
499
|
|
|
|
|
|
|
# |
|
500
|
|
|
|
|
|
|
# You should know that whoever installed this module |
|
501
|
|
|
|
|
|
|
# may have disabled this routine. That person may |
|
502
|
|
|
|
|
|
|
# be smarter than you, and secretly laughing at |
|
503
|
|
|
|
|
|
|
# you now. If I were you, I'd go read the perlsec |
|
504
|
|
|
|
|
|
|
# manpage. Or at least a good Dilbert book. |
|
505
|
|
|
|
|
|
|
# |
|
506
|
|
|
|
|
|
|
# I can't put this off any longer, no matter how |
|
507
|
|
|
|
|
|
|
# hard I try... |
|
508
|
|
|
|
|
|
|
# |
|
509
|
2
|
|
|
2
|
|
4
|
for (@_) { |
|
510
|
11
|
100
|
66
|
|
|
24
|
$_ = $1 if is_tainted $_ and /^(.*)$/s |
|
511
|
|
|
|
|
|
|
} |
|
512
|
2
|
|
|
|
|
5
|
return; # explicitly returns nothing |
|
513
|
1
|
|
|
|
|
199
|
}; |
|
514
|
|
|
|
|
|
|
|
|
515
|
|
|
|
|
|
|
=item unconditional_untaint LIST |
|
516
|
|
|
|
|
|
|
|
|
517
|
|
|
|
|
|
|
By unpopular request, this routine is included. Don't use it. Use |
|
518
|
|
|
|
|
|
|
the method described in L instead. You'd have to be crazy |
|
519
|
|
|
|
|
|
|
to use this routine. (If you are, read the module itself to see |
|
520
|
|
|
|
|
|
|
how to enable it. I'm not gonna tell you here.) |
|
521
|
|
|
|
|
|
|
|
|
522
|
|
|
|
|
|
|
Given a list of possibly tainted B, this untaints each of them |
|
523
|
|
|
|
|
|
|
without any regard for whether they should be untainted or not. |
|
524
|
|
|
|
|
|
|
|
|
525
|
|
|
|
|
|
|
=cut |
|
526
|
|
|
|
|
|
|
|
|
527
|
|
|
|
|
|
|
} elsif ($_ eq 'allow_no_taint') { |
|
528
|
1
|
|
|
|
|
3
|
$no_taint_okay{$pkg} = 1; |
|
529
|
|
|
|
|
|
|
} else { |
|
530
|
5
|
|
|
|
|
16
|
push @importables, $_; |
|
531
|
|
|
|
|
|
|
} |
|
532
|
|
|
|
|
|
|
} |
|
533
|
6
|
100
|
|
|
|
53
|
return unless @importables; |
|
534
|
5
|
50
|
66
|
|
|
63
|
unless ($no_taint_okay{$pkg} or is_tainted tainted_null) { |
|
535
|
|
|
|
|
|
|
# What happened? Probably somebody forgot to use -T, |
|
536
|
|
|
|
|
|
|
# or they thought their script would be setuid/setgid. |
|
537
|
0
|
|
|
|
|
0
|
warn "Hmmm... Tainting doesn't seem to be turned on.\n"; |
|
538
|
0
|
|
|
|
|
0
|
warn "Did you forget to use the -T invocation option?\n"; |
|
539
|
0
|
|
|
|
|
0
|
require Carp; |
|
540
|
0
|
|
|
|
|
0
|
&Carp::croak("Taint checks not enabled"); |
|
541
|
|
|
|
|
|
|
} |
|
542
|
5
|
|
|
|
|
14
|
local($Exporter::ExportLevel) = 1; |
|
543
|
5
|
|
|
|
|
6066
|
SUPER::import $class @importables; |
|
544
|
|
|
|
|
|
|
} |
|
545
|
|
|
|
|
|
|
|
|
546
|
|
|
|
|
|
|
=item allow_no_taint |
|
547
|
|
|
|
|
|
|
|
|
548
|
|
|
|
|
|
|
By default, importing symbols from this module requires taint checks |
|
549
|
|
|
|
|
|
|
to be turned on. If you wish to use this module without requiring |
|
550
|
|
|
|
|
|
|
taint checking (for example, if writing a module which may or may |
|
551
|
|
|
|
|
|
|
not be run under C<-T>) either import this pseudo-item... |
|
552
|
|
|
|
|
|
|
|
|
553
|
|
|
|
|
|
|
use Taint qw(allow_no_taint); # allow to run without -T |
|
554
|
|
|
|
|
|
|
use Taint; # default import list |
|
555
|
|
|
|
|
|
|
|
|
556
|
|
|
|
|
|
|
or avoid importing any symbols by explicitly passing an empty import |
|
557
|
|
|
|
|
|
|
list. |
|
558
|
|
|
|
|
|
|
|
|
559
|
|
|
|
|
|
|
use Taint (); # importing no symbols |
|
560
|
|
|
|
|
|
|
|
|
561
|
|
|
|
|
|
|
If you use either of these methods to allow taint checks not to be |
|
562
|
|
|
|
|
|
|
required, you may want to use the constant C (see |
|
563
|
|
|
|
|
|
|
L) to determine whether checks are on. |
|
564
|
|
|
|
|
|
|
|
|
565
|
|
|
|
|
|
|
It may be helpful to allow checks to be off during development, |
|
566
|
|
|
|
|
|
|
but be sure to require them after release! |
|
567
|
|
|
|
|
|
|
|
|
568
|
|
|
|
|
|
|
=cut |
|
569
|
|
|
|
|
|
|
|
|
570
|
|
|
|
|
|
|
# This is the fake sub! (But you would have figured that |
|
571
|
|
|
|
|
|
|
# out for yourself.) |
|
572
|
|
|
|
|
|
|
sub unconditional_untaint (@) { |
|
573
|
1
|
|
|
1
|
1
|
13
|
require Carp; |
|
574
|
1
|
|
|
|
|
248
|
&Carp::carp("sub unconditional_untaint() not properly imported"); |
|
575
|
|
|
|
|
|
|
} |
|
576
|
|
|
|
|
|
|
|
|
577
|
|
|
|
|
|
|
sub unimport { |
|
578
|
1
|
|
|
1
|
|
4098
|
my $class = shift; |
|
579
|
1
|
|
|
|
|
894
|
my $pkg = caller; |
|
580
|
1
|
|
|
|
|
6
|
for (@_) { |
|
581
|
1
|
50
|
|
|
|
173
|
if ($_ eq 'sanity') { |
|
582
|
1
|
|
|
|
|
2
|
if (allowing_insanity) { |
|
583
|
1
|
|
|
|
|
171
|
$insanity{$pkg} = 1; |
|
584
|
|
|
|
|
|
|
} else { |
|
585
|
|
|
|
|
|
|
require Carp; |
|
586
|
|
|
|
|
|
|
&Carp::croak("Disabled option requested"); |
|
587
|
|
|
|
|
|
|
} |
|
588
|
|
|
|
|
|
|
} else { |
|
589
|
|
|
|
|
|
|
# Simply ignore other unimports |
|
590
|
|
|
|
|
|
|
} |
|
591
|
|
|
|
|
|
|
} |
|
592
|
|
|
|
|
|
|
} |
|
593
|
|
|
|
|
|
|
|
|
594
|
|
|
|
|
|
|
1; |
|
595
|
|
|
|
|
|
|
__END__ |