Add missing syms to global.sym; update magic doc
[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
9command-line shells, which are based on multiple substitution passes on
10each line of the script, Perl uses a more conventional evaluation scheme
11with fewer hidden snags. Additionally, because the language has more
12built-in functionality, it can rely less upon external (and possibly
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
21someone else, such as a CGI script.
a0d0e21e 22
425e5e39 23While in this mode, Perl takes special precautions called I<taint
24checks> to prevent both obvious and subtle traps. Some of these checks
25are reasonably simple, such as verifying that path directories aren't
26writable by others; careful programmers have always used checks like
27these. Other checks, however, are best supported by the language itself,
28and it is these checks especially that contribute to making a setuid Perl
29program more secure than the corresponding C program.
30
31You may not use data derived from outside your program to affect something
32else outside your program--at least, not by accident. All command-line
33arguments, environment variables, and file input are marked as "tainted".
34Tainted data may not be used directly or indirectly in any command that
5f05dabc 35invokes a sub-shell, nor in any command that modifies files, directories,
425e5e39 36or processes. Any variable set within an expression that has previously
37referenced a tainted value itself becomes tainted, even if it is logically
38impossible for the tainted value to influence the variable. Because
39taintedness is associated with each scalar value, some elements of an
40array can be tainted and others not.
a0d0e21e 41
a0d0e21e 42For example:
43
425e5e39 44 $arg = shift; # $arg is tainted
45 $hid = $arg, 'bar'; # $hid is also tainted
46 $line = <>; # Tainted
a0d0e21e 47 $path = $ENV{'PATH'}; # Tainted, but see below
425e5e39 48 $data = 'abc'; # Not tainted
a0d0e21e 49
425e5e39 50 system "echo $arg"; # Insecure
51 system "/bin/echo", $arg; # Secure (doesn't use sh)
52 system "echo $hid"; # Insecure
53 system "echo $data"; # Insecure until PATH set
a0d0e21e 54
425e5e39 55 $path = $ENV{'PATH'}; # $path now tainted
a0d0e21e 56
425e5e39 57 $ENV{'PATH'} = '/bin:/usr/bin';
58 $ENV{'IFS'} = '' if $ENV{'IFS'} ne '';
a0d0e21e 59
425e5e39 60 $path = $ENV{'PATH'}; # $path now NOT tainted
61 system "echo $data"; # Is secure now!
a0d0e21e 62
425e5e39 63 open(FOO, "< $arg"); # OK - read-only file
64 open(FOO, "> $arg"); # Not OK - trying to write
a0d0e21e 65
425e5e39 66 open(FOO,"echo $arg|"); # Not OK, but...
67 open(FOO,"-|")
68 or exec 'echo', $arg; # OK
a0d0e21e 69
425e5e39 70 $shout = `echo $arg`; # Insecure, $shout now tainted
a0d0e21e 71
425e5e39 72 unlink $data, $arg; # Insecure
73 umask $arg; # Insecure
a0d0e21e 74
425e5e39 75 exec "echo $arg"; # Insecure
76 exec "echo", $arg; # Secure (doesn't use the shell)
77 exec "sh", '-c', $arg; # Considered secure, alas!
a0d0e21e 78
79If you try to do something insecure, you will get a fatal error saying
80something like "Insecure dependency" or "Insecure PATH". Note that you
425e5e39 81can still write an insecure B<system> or B<exec>, but only by explicitly
82doing something like the last example above.
83
84=head2 Laundering and Detecting Tainted Data
85
86To test whether a variable contains tainted data, and whose use would thus
87trigger an "Insecure dependency" message, you can use the following
88I<is_tainted()> function.
89
90 sub is_tainted {
91 return ! eval {
92 join('',@_), kill 0;
93 1;
94 };
95 }
96
97This function makes use of the fact that the presence of tainted data
98anywhere within an expression renders the entire expression tainted. It
99would be inefficient for every operator to test every argument for
100taintedness. Instead, the slightly more efficient and conservative
101approach is used that if any tainted value has been accessed within the
102same expression, the whole expression is considered tainted.
103
5f05dabc 104But testing for taintedness gets you only so far. Sometimes you have just
425e5e39 105to clear your data's taintedness. The only way to bypass the tainting
5f05dabc 106mechanism is by referencing sub-patterns from a regular expression match.
425e5e39 107Perl presumes that if you reference a substring using $1, $2, etc., that
108you knew what you were doing when you wrote the pattern. That means using
109a bit of thought--don't just blindly untaint anything, or you defeat the
110entire mechanism. It's better to verify that the variable has only
111good characters (for certain values of "good") rather than checking
112whether it has any bad characters. That's because it's far too easy to
113miss bad characters that you never thought of.
114
115Here's a test to make sure that the data contains nothing but "word"
116characters (alphabetics, numerics, and underscores), a hyphen, an at sign,
117or a dot.
118
119 if ($data =~ /^([-\@\w.]+)$/) {
120 $data = $1; # $data now untainted
121 } else {
122 die "Bad data in $data"; # log this somewhere
123 }
124
5f05dabc 125This is fairly secure because C</\w+/> doesn't normally match shell
425e5e39 126metacharacters, nor are dot, dash, or at going to mean something special
127to the shell. Use of C</.+/> would have been insecure in theory because
128it lets everything through, but Perl doesn't check for that. The lesson
129is that when untainting, you must be exceedingly careful with your patterns.
130Laundering data using regular expression is the I<ONLY> mechanism for
131untainting dirty data, unless you use the strategy detailed below to fork
132a child of lesser privilege.
133
134=head2 Cleaning Up Your Path
135
1fef88e7 136For "Insecure C<$ENV{PATH}>" messages, you need to set C<$ENV{'PATH'}> to a
425e5e39 137known value, and each directory in the path must be non-writable by others
138than its owner and group. You may be surprised to get this message even
139if the pathname to your executable is fully qualified. This is I<not>
140generated because you didn't supply a full path to the program; instead,
141it's generated because you never set your PATH environment variable, or
142you didn't set it to something that was safe. Because Perl can't
143guarantee that the executable in question isn't itself going to turn
144around and execute some other program that is dependent on your PATH, it
145makes sure you set the PATH.
a0d0e21e 146
147It's also possible to get into trouble with other operations that don't
148care whether they use tainted values. Make judicious use of the file
149tests in dealing with any user-supplied filenames. When possible, do
150opens and such after setting C<$E<gt> = $E<lt>>. (Remember group IDs,
425e5e39 151too!) Perl doesn't prevent you from opening tainted filenames for reading,
a0d0e21e 152so be careful what you print out. The tainting mechanism is intended to
153prevent stupid mistakes, not to remove the need for thought.
154
425e5e39 155Perl does not call the shell to expand wild cards when you pass B<system>
156and B<exec> explicit parameter lists instead of strings with possible shell
157wildcards in them. Unfortunately, the B<open>, B<glob>, and
5f05dabc 158back-tick functions provide no such alternate calling convention, so more
425e5e39 159subterfuge will be required.
160
161Perl provides a reasonably safe way to open a file or pipe from a setuid
162or setgid program: just create a child process with reduced privilege who
163does the dirty work for you. First, fork a child using the special
164B<open> syntax that connects the parent and child by a pipe. Now the
165child resets its ID set and any other per-process attributes, like
166environment variables, umasks, current working directories, back to the
167originals or known safe values. Then the child process, which no longer
168has any special permissions, does the B<open> or other system call.
169Finally, the child passes the data it managed to access back to the
5f05dabc 170parent. Because the file or pipe was opened in the child while running
425e5e39 171under less privilege than the parent, it's not apt to be tricked into
172doing something it shouldn't.
173
5f05dabc 174Here's a way to do back-ticks reasonably safely. Notice how the B<exec> is
425e5e39 175not called with a string that the shell could expand. This is by far the
176best way to call something that might be subjected to shell escapes: just
177never call the shell at all. By the time we get to the B<exec>, tainting
178is turned off, however, so be careful what you call and what you pass it.
cb1a09d0 179
425e5e39 180 use English;
cb1a09d0 181 die unless defined $pid = open(KID, "-|");
182 if ($pid) { # parent
183 while (<KID>) {
184 # do something
425e5e39 185 }
cb1a09d0 186 close KID;
187 } else {
425e5e39 188 $EUID = $UID;
189 $EGID = $GID; # XXX: initgroups() not called
190 $ENV{PATH} = "/bin:/usr/bin";
191 exec 'myprog', 'arg1', 'arg2';
192 die "can't exec myprog: $!";
193 }
194
195A similar strategy would work for wildcard expansion via C<glob>.
196
197Taint checking is most useful when although you trust yourself not to have
198written a program to give away the farm, you don't necessarily trust those
199who end up using it not to try to trick it into doing something bad. This
200is the kind of security checking that's useful for setuid programs and
201programs launched on someone else's behalf, like CGI programs.
202
203This is quite different, however, from not even trusting the writer of the
204code not to try to do something evil. That's the kind of trust needed
205when someone hands you a program you've never seen before and says, "Here,
206run this." For that kind of safety, check out the Safe module,
207included standard in the Perl distribution. This module allows the
208programmer to set up special compartments in which all system operations
209are trapped and namespace access is carefully controlled.
210
211=head2 Security Bugs
212
213Beyond the obvious problems that stem from giving special privileges to
214systems as flexible as scripts, on many versions of Unix, setuid scripts
215are inherently insecure right from the start. The problem is a race
216condition in the kernel. Between the time the kernel opens the file to
217see which interpreter to run and when the (now-setuid) interpreter turns
218around and reopens the file to interpret it, the file in question may have
219changed, especially if you have symbolic links on your system.
220
221Fortunately, sometimes this kernel "feature" can be disabled.
222Unfortunately, there are two ways to disable it. The system can simply
223outlaw scripts with the setuid bit set, which doesn't help much.
224Alternately, it can simply ignore the setuid bit on scripts. If the
225latter is true, Perl can emulate the setuid and setgid mechanism when it
226notices the otherwise useless setuid/gid bits on Perl scripts. It does
227this via a special executable called B<suidperl> that is automatically
228invoked for you if it's needed.
229
230However, if the kernel setuid script feature isn't disabled, Perl will
231complain loudly that your setuid script is insecure. You'll need to
232either disable the kernel setuid script feature, or put a C wrapper around
233the script. A C wrapper is just a compiled program that does nothing
234except call your Perl program. Compiled programs are not subject to the
235kernel bug that plagues setuid scripts. Here's a simple wrapper, written
236in C:
237
238 #define REAL_PATH "/path/to/script"
239 main(ac, av)
240 char **av;
241 {
242 execv(REAL_PATH, av);
cb1a09d0 243 }
244
425e5e39 245Compile this wrapper into a binary executable and then make I<it> rather
246than your script setuid or setgid.
247
248See the program B<wrapsuid> in the F<eg> directory of your Perl
249distribution for a convenient way to do this automatically for all your
250setuid Perl programs. It moves setuid scripts into files with the same
251name plus a leading dot, and then compiles a wrapper like the one above
252for each of them.
253
254In recent years, vendors have begun to supply systems free of this
255inherent security bug. On such systems, when the kernel passes the name
256of the setuid script to open to the interpreter, rather than using a
257pathname subject to meddling, it instead passes I</dev/fd/3>. This is a
258special file already opened on the script, so that there can be no race
259condition for evil scripts to exploit. On these systems, Perl should be
260compiled with C<-DSETUID_SCRIPTS_ARE_SECURE_NOW>. The B<Configure>
261program that builds Perl tries to figure this out for itself, so you
262should never have to specify this yourself. Most modern releases of
263SysVr4 and BSD 4.4 use this approach to avoid the kernel race condition.
264
265Prior to release 5.003 of Perl, a bug in the code of B<suidperl> could
266introduce a security hole in systems compiled with strict POSIX
267compliance.