Integrate perlio:
[p5sagit/p5-mst-13.2.git] / pod / perlsec.pod
CommitLineData
a0d0e21e 1=head1 NAME
2
3perlsec - Perl security
4
5=head1 DESCRIPTION
6
425e5e39 7Perl is designed to make it easy to program securely even when running
8with extra privileges, like setuid or setgid programs. Unlike most
54310121 9command line shells, which are based on multiple substitution passes on
425e5e39 10each line of the script, Perl uses a more conventional evaluation scheme
11with fewer hidden snags. Additionally, because the language has more
54310121 12builtin functionality, it can rely less upon external (and possibly
425e5e39 13untrustworthy) programs to accomplish its purposes.
a0d0e21e 14
425e5e39 15Perl automatically enables a set of special security checks, called I<taint
16mode>, when it detects its program running with differing real and effective
17user or group IDs. The setuid bit in Unix permissions is mode 04000, the
18setgid bit mode 02000; either or both may be set. You can also enable taint
5f05dabc 19mode explicitly by using the B<-T> command line flag. This flag is
425e5e39 20I<strongly> suggested for server programs and any program run on behalf of
fb73857a 21someone else, such as a CGI script. Once taint mode is on, it's on for
22the remainder of your script.
a0d0e21e 23
1e422769 24While in this mode, Perl takes special precautions called I<taint
25checks> to prevent both obvious and subtle traps. Some of these checks
26are reasonably simple, such as verifying that path directories aren't
27writable by others; careful programmers have always used checks like
28these. Other checks, however, are best supported by the language itself,
fb73857a 29and it is these checks especially that contribute to making a set-id Perl
425e5e39 30program more secure than the corresponding C program.
31
fb73857a 32You may not use data derived from outside your program to affect
33something else outside your program--at least, not by accident. All
34command line arguments, environment variables, locale information (see
d929ce6f 35L<perllocale>), results of certain system calls (readdir(),
41d6edb2 36readlink(), the variable of shmread(), the messages returned by
37msgrcv(), the password, gcos and shell fields returned by the
38getpwxxx() calls), and all file input are marked as "tainted".
39Tainted data may not be used directly or indirectly in any command
40that invokes a sub-shell, nor in any command that modifies files,
b7ee89ce 41directories, or processes, B<with the following exceptions>:
42
43=over 4
44
45=item *
46
ee556d55 47If you pass more than one argument to either C<system> or C<exec>,
bbd7eb8a 48the arguments are checked for taintedness B<but> the operation will still
49be attempted, emitting an optional warning. This will be fatal in a
50future version of perl so do not rely on it to bypass the tainting
51mechanism.
b7ee89ce 52
53=item *
54
55Arguments to C<print> and C<syswrite> are B<not> checked for taintedness.
56
57=back
58
ee556d55 59The value of an expression containing tainted data will itself be
60tainted, even if it is logically impossible for the tainted data to
61affect the value.
62
d929ce6f 63Because taintedness is associated with each scalar value, some
64elements of an array can be tainted and others not.
a0d0e21e 65
a0d0e21e 66For example:
67
425e5e39 68 $arg = shift; # $arg is tainted
69 $hid = $arg, 'bar'; # $hid is also tainted
70 $line = <>; # Tainted
8ebc5c01 71 $line = <STDIN>; # Also tainted
72 open FOO, "/home/me/bar" or die $!;
73 $line = <FOO>; # Still tainted
a0d0e21e 74 $path = $ENV{'PATH'}; # Tainted, but see below
425e5e39 75 $data = 'abc'; # Not tainted
a0d0e21e 76
425e5e39 77 system "echo $arg"; # Insecure
bbd7eb8a 78 system "/bin/echo", $arg; # Allowed but considered insecure
79 # (Perl doesn't know about /bin/echo)
425e5e39 80 system "echo $hid"; # Insecure
81 system "echo $data"; # Insecure until PATH set
a0d0e21e 82
425e5e39 83 $path = $ENV{'PATH'}; # $path now tainted
a0d0e21e 84
54310121 85 $ENV{'PATH'} = '/bin:/usr/bin';
c90c0ff4 86 delete @ENV{'IFS', 'CDPATH', 'ENV', 'BASH_ENV'};
a0d0e21e 87
425e5e39 88 $path = $ENV{'PATH'}; # $path now NOT tainted
89 system "echo $data"; # Is secure now!
a0d0e21e 90
425e5e39 91 open(FOO, "< $arg"); # OK - read-only file
92 open(FOO, "> $arg"); # Not OK - trying to write
a0d0e21e 93
bbd7eb8a 94 open(FOO,"echo $arg|"); # Not OK
425e5e39 95 open(FOO,"-|")
bbd7eb8a 96 or exec 'echo', $arg; # Allowed but not really OK
a0d0e21e 97
425e5e39 98 $shout = `echo $arg`; # Insecure, $shout now tainted
a0d0e21e 99
425e5e39 100 unlink $data, $arg; # Insecure
101 umask $arg; # Insecure
a0d0e21e 102
bbd7eb8a 103 exec "echo $arg"; # Insecure
104 exec "echo", $arg; # Allowed but considered insecure
425e5e39 105 exec "sh", '-c', $arg; # Considered secure, alas!
a0d0e21e 106
3a4b19e4 107 @files = <*.c>; # insecure (uses readdir() or similar)
108 @files = glob('*.c'); # insecure (uses readdir() or similar)
7bac28a0 109
3f7d42d8 110 # In Perl releases older than 5.6.0 the <*.c> and glob('*.c') would
111 # have used an external program to do the filename expansion; but in
112 # either case the result is tainted since the list of filenames comes
113 # from outside of the program.
114
ee556d55 115 $bad = ($arg, 23); # $bad will be tainted
116 $arg, `true`; # Insecure (although it isn't really)
117
a0d0e21e 118If you try to do something insecure, you will get a fatal error saying
62f468fc 119something like "Insecure dependency" or "Insecure $ENV{PATH}". Note that you
425e5e39 120can still write an insecure B<system> or B<exec>, but only by explicitly
bbd7eb8a 121doing something like the "considered secure" example above. This will not
122be possible in a future version of Perl.
425e5e39 123
124=head2 Laundering and Detecting Tainted Data
125
3f7d42d8 126To test whether a variable contains tainted data, and whose use would
127thus trigger an "Insecure dependency" message, you can use the
128tainted() function of the Scalar::Util module, available in your
129nearby CPAN mirror, and included in Perl starting from the release 5.8.0.
130Or you may be able to use the following I<is_tainted()> function.
425e5e39 131
132 sub is_tainted {
61890e45 133 return ! eval { eval("#" . substr(join("", @_), 0, 0)); 1 };
425e5e39 134 }
135
136This function makes use of the fact that the presence of tainted data
137anywhere within an expression renders the entire expression tainted. It
138would be inefficient for every operator to test every argument for
139taintedness. Instead, the slightly more efficient and conservative
140approach is used that if any tainted value has been accessed within the
141same expression, the whole expression is considered tainted.
142
5f05dabc 143But testing for taintedness gets you only so far. Sometimes you have just
425e5e39 144to clear your data's taintedness. The only way to bypass the tainting
54310121 145mechanism is by referencing subpatterns from a regular expression match.
425e5e39 146Perl presumes that if you reference a substring using $1, $2, etc., that
147you knew what you were doing when you wrote the pattern. That means using
148a bit of thought--don't just blindly untaint anything, or you defeat the
a034a98d 149entire mechanism. It's better to verify that the variable has only good
150characters (for certain values of "good") rather than checking whether it
151has any bad characters. That's because it's far too easy to miss bad
152characters that you never thought of.
425e5e39 153
154Here's a test to make sure that the data contains nothing but "word"
155characters (alphabetics, numerics, and underscores), a hyphen, an at sign,
156or a dot.
157
54310121 158 if ($data =~ /^([-\@\w.]+)$/) {
425e5e39 159 $data = $1; # $data now untainted
160 } else {
161 die "Bad data in $data"; # log this somewhere
162 }
163
5f05dabc 164This is fairly secure because C</\w+/> doesn't normally match shell
425e5e39 165metacharacters, nor are dot, dash, or at going to mean something special
166to the shell. Use of C</.+/> would have been insecure in theory because
167it lets everything through, but Perl doesn't check for that. The lesson
168is that when untainting, you must be exceedingly careful with your patterns.
19799a22 169Laundering data using regular expression is the I<only> mechanism for
425e5e39 170untainting dirty data, unless you use the strategy detailed below to fork
171a child of lesser privilege.
172
a034a98d 173The example does not untaint $data if C<use locale> is in effect,
174because the characters matched by C<\w> are determined by the locale.
175Perl considers that locale definitions are untrustworthy because they
176contain data from outside the program. If you are writing a
177locale-aware program, and want to launder data with a regular expression
178containing C<\w>, put C<no locale> ahead of the expression in the same
179block. See L<perllocale/SECURITY> for further discussion and examples.
180
3a52c276 181=head2 Switches On the "#!" Line
182
183When you make a script executable, in order to make it usable as a
184command, the system will pass switches to perl from the script's #!
54310121 185line. Perl checks that any command line switches given to a setuid
3a52c276 186(or setgid) script actually match the ones set on the #! line. Some
54310121 187Unix and Unix-like environments impose a one-switch limit on the #!
3a52c276 188line, so you may need to use something like C<-wU> instead of C<-w -U>
54310121 189under such systems. (This issue should arise only in Unix or
190Unix-like environments that support #! and setuid or setgid scripts.)
3a52c276 191
425e5e39 192=head2 Cleaning Up Your Path
193
1fef88e7 194For "Insecure C<$ENV{PATH}>" messages, you need to set C<$ENV{'PATH'}> to a
1e422769 195known value, and each directory in the path must be non-writable by others
196than its owner and group. You may be surprised to get this message even
197if the pathname to your executable is fully qualified. This is I<not>
198generated because you didn't supply a full path to the program; instead,
199it's generated because you never set your PATH environment variable, or
200you didn't set it to something that was safe. Because Perl can't
201guarantee that the executable in question isn't itself going to turn
202around and execute some other program that is dependent on your PATH, it
54310121 203makes sure you set the PATH.
a0d0e21e 204
a3cb178b 205The PATH isn't the only environment variable which can cause problems.
206Because some shells may use the variables IFS, CDPATH, ENV, and
207BASH_ENV, Perl checks that those are either empty or untainted when
208starting subprocesses. You may wish to add something like this to your
209setid and taint-checking scripts.
210
211 delete @ENV{qw(IFS CDPATH ENV BASH_ENV)}; # Make %ENV safer
212
a0d0e21e 213It's also possible to get into trouble with other operations that don't
214care whether they use tainted values. Make judicious use of the file
215tests in dealing with any user-supplied filenames. When possible, do
fb73857a 216opens and such B<after> properly dropping any special user (or group!)
217privileges. Perl doesn't prevent you from opening tainted filenames for reading,
a0d0e21e 218so be careful what you print out. The tainting mechanism is intended to
219prevent stupid mistakes, not to remove the need for thought.
220
425e5e39 221Perl does not call the shell to expand wild cards when you pass B<system>
222and B<exec> explicit parameter lists instead of strings with possible shell
223wildcards in them. Unfortunately, the B<open>, B<glob>, and
54310121 224backtick functions provide no such alternate calling convention, so more
225subterfuge will be required.
425e5e39 226
227Perl provides a reasonably safe way to open a file or pipe from a setuid
228or setgid program: just create a child process with reduced privilege who
229does the dirty work for you. First, fork a child using the special
230B<open> syntax that connects the parent and child by a pipe. Now the
231child resets its ID set and any other per-process attributes, like
232environment variables, umasks, current working directories, back to the
233originals or known safe values. Then the child process, which no longer
234has any special permissions, does the B<open> or other system call.
235Finally, the child passes the data it managed to access back to the
5f05dabc 236parent. Because the file or pipe was opened in the child while running
425e5e39 237under less privilege than the parent, it's not apt to be tricked into
238doing something it shouldn't.
239
54310121 240Here's a way to do backticks reasonably safely. Notice how the B<exec> is
425e5e39 241not called with a string that the shell could expand. This is by far the
242best way to call something that might be subjected to shell escapes: just
fb73857a 243never call the shell at all.
cb1a09d0 244
a1ce9542 245 use English '-no_match_vars';
e093bcf0 246 die "Can't fork: $!" unless defined($pid = open(KID, "-|"));
247 if ($pid) { # parent
248 while (<KID>) {
249 # do something
250 }
251 close KID;
252 } else {
253 my @temp = ($EUID, $EGID);
254 my $orig_uid = $UID;
255 my $orig_gid = $GID;
256 $EUID = $UID;
257 $EGID = $GID;
258 # Drop privileges
259 $UID = $orig_uid;
260 $GID = $orig_gid;
261 # Make sure privs are really gone
262 ($EUID, $EGID) = @temp;
263 die "Can't drop privileges"
264 unless $UID == $EUID && $GID eq $EGID;
265 $ENV{PATH} = "/bin:/usr/bin"; # Minimal PATH.
266 # Consider sanitizing the environment even more.
267 exec 'myprog', 'arg1', 'arg2'
268 or die "can't exec myprog: $!";
269 }
425e5e39 270
fb73857a 271A similar strategy would work for wildcard expansion via C<glob>, although
272you can use C<readdir> instead.
425e5e39 273
274Taint checking is most useful when although you trust yourself not to have
275written a program to give away the farm, you don't necessarily trust those
276who end up using it not to try to trick it into doing something bad. This
fb73857a 277is the kind of security checking that's useful for set-id programs and
425e5e39 278programs launched on someone else's behalf, like CGI programs.
279
280This is quite different, however, from not even trusting the writer of the
281code not to try to do something evil. That's the kind of trust needed
282when someone hands you a program you've never seen before and says, "Here,
283run this." For that kind of safety, check out the Safe module,
284included standard in the Perl distribution. This module allows the
285programmer to set up special compartments in which all system operations
286are trapped and namespace access is carefully controlled.
287
288=head2 Security Bugs
289
290Beyond the obvious problems that stem from giving special privileges to
fb73857a 291systems as flexible as scripts, on many versions of Unix, set-id scripts
425e5e39 292are inherently insecure right from the start. The problem is a race
293condition in the kernel. Between the time the kernel opens the file to
fb73857a 294see which interpreter to run and when the (now-set-id) interpreter turns
425e5e39 295around and reopens the file to interpret it, the file in question may have
296changed, especially if you have symbolic links on your system.
297
298Fortunately, sometimes this kernel "feature" can be disabled.
299Unfortunately, there are two ways to disable it. The system can simply
fb73857a 300outlaw scripts with any set-id bit set, which doesn't help much.
301Alternately, it can simply ignore the set-id bits on scripts. If the
425e5e39 302latter is true, Perl can emulate the setuid and setgid mechanism when it
303notices the otherwise useless setuid/gid bits on Perl scripts. It does
304this via a special executable called B<suidperl> that is automatically
54310121 305invoked for you if it's needed.
425e5e39 306
fb73857a 307However, if the kernel set-id script feature isn't disabled, Perl will
308complain loudly that your set-id script is insecure. You'll need to
309either disable the kernel set-id script feature, or put a C wrapper around
425e5e39 310the script. A C wrapper is just a compiled program that does nothing
311except call your Perl program. Compiled programs are not subject to the
fb73857a 312kernel bug that plagues set-id scripts. Here's a simple wrapper, written
425e5e39 313in C:
314
315 #define REAL_PATH "/path/to/script"
54310121 316 main(ac, av)
425e5e39 317 char **av;
318 {
319 execv(REAL_PATH, av);
54310121 320 }
cb1a09d0 321
54310121 322Compile this wrapper into a binary executable and then make I<it> rather
323than your script setuid or setgid.
425e5e39 324
425e5e39 325In recent years, vendors have begun to supply systems free of this
326inherent security bug. On such systems, when the kernel passes the name
fb73857a 327of the set-id script to open to the interpreter, rather than using a
425e5e39 328pathname subject to meddling, it instead passes I</dev/fd/3>. This is a
329special file already opened on the script, so that there can be no race
330condition for evil scripts to exploit. On these systems, Perl should be
331compiled with C<-DSETUID_SCRIPTS_ARE_SECURE_NOW>. The B<Configure>
332program that builds Perl tries to figure this out for itself, so you
333should never have to specify this yourself. Most modern releases of
334SysVr4 and BSD 4.4 use this approach to avoid the kernel race condition.
335
0325b4c4 336Prior to release 5.6.1 of Perl, bugs in the code of B<suidperl> could
337introduce a security hole.
68dc0745 338
339=head2 Protecting Your Programs
340
341There are a number of ways to hide the source to your Perl programs,
342with varying levels of "security".
343
344First of all, however, you I<can't> take away read permission, because
345the source code has to be readable in order to be compiled and
346interpreted. (That doesn't mean that a CGI script's source is
347readable by people on the web, though.) So you have to leave the
5a964f20 348permissions at the socially friendly 0755 level. This lets
349people on your local system only see your source.
68dc0745 350
5a964f20 351Some people mistakenly regard this as a security problem. If your program does
68dc0745 352insecure things, and relies on people not knowing how to exploit those
353insecurities, it is not secure. It is often possible for someone to
354determine the insecure things and exploit them without viewing the
355source. Security through obscurity, the name for hiding your bugs
356instead of fixing them, is little security indeed.
357
83df6a1d 358You can try using encryption via source filters (Filter::* from CPAN,
359or Filter::Util::Call and Filter::Simple since Perl 5.8).
360But crackers might be able to decrypt it. You can try using the byte
361code compiler and interpreter described below, but crackers might be
362able to de-compile it. You can try using the native-code compiler
68dc0745 363described below, but crackers might be able to disassemble it. These
364pose varying degrees of difficulty to people wanting to get at your
365code, but none can definitively conceal it (this is true of every
366language, not just Perl).
367
368If you're concerned about people profiting from your code, then the
369bottom line is that nothing but a restrictive licence will give you
370legal security. License your software and pepper it with threatening
371statements like "This is unpublished proprietary software of XYZ Corp.
372Your access to it does not give you permission to use it blah blah
373blah." You should see a lawyer to be sure your licence's wording will
374stand up in court.
5a964f20 375
376=head1 SEE ALSO
377
378L<perlrun> for its description of cleaning up environment variables.