Source of: /manual/en/security.variables.php
<?php
include_once $_SERVER['DOCUMENT_ROOT'] . '/include/shared-manual.inc';
$TOC = array();
$PARENTS = array();
include_once dirname(__FILE__) ."/toc/security.inc";
$setup = array (
'home' =>
array (
0 => 'index.php',
1 => 'PHP Manual',
),
'head' =>
array (
0 => 'UTF-8',
1 => 'en',
),
'this' =>
array (
0 => 'security.variables.php',
1 => 'User Submitted Data',
),
'up' =>
array (
0 => 'security.php',
1 => 'Security',
),
'prev' =>
array (
0 => 'security.globals.php',
1 => 'Using Register Globals',
),
'next' =>
array (
0 => 'security.magicquotes.php',
1 => 'Magic Quotes',
),
'alternatives' =>
array (
),
);
$setup["toc"] = $TOC;
$setup["parents"] = $PARENTS;
manual_setup($setup);
manual_header();
?>
<div id="security.variables" class="chapter">
<h1>User Submitted Data</h1>
<p class="para">
The greatest weakness in many <acronym title="PHP: Hypertext Preprocessor">PHP</acronym> programs is not inherent in the
language itself, but merely an issue of code not being written with
security in mind. For this reason, you should always take the time
to consider the implications of a given piece of code, to ascertain
the possible damage if an unexpected variable is submitted to it.
<div class="example" id="example-343">
<p><strong>Example #1 Dangerous Variable Usage</strong></p>
<div class="example-contents">
<div class="phpcode"><code><span style="color: #000000">
<span style="color: #0000BB"><?php<br /></span><span style="color: #FF8000">// remove a file from the user's home directory... or maybe<br />// somebody else's?<br /></span><span style="color: #0000BB">unlink </span><span style="color: #007700">(</span><span style="color: #0000BB">$evil_var</span><span style="color: #007700">);<br /><br /></span><span style="color: #FF8000">// Write logging of their access... or maybe an /etc/passwd entry?<br /></span><span style="color: #0000BB">fwrite </span><span style="color: #007700">(</span><span style="color: #0000BB">$fp</span><span style="color: #007700">, </span><span style="color: #0000BB">$evil_var</span><span style="color: #007700">);<br /><br /></span><span style="color: #FF8000">// Execute something trivial.. or rm -rf *?<br /></span><span style="color: #0000BB">system </span><span style="color: #007700">(</span><span style="color: #0000BB">$evil_var</span><span style="color: #007700">);<br /></span><span style="color: #0000BB">exec </span><span style="color: #007700">(</span><span style="color: #0000BB">$evil_var</span><span style="color: #007700">);<br /><br /></span><span style="color: #0000BB">?></span>
</span>
</code></div>
</div>
</div>
</p>
<p class="para">
You should always carefully examine your code to make sure that any
variables being submitted from a web browser are being properly
checked, and ask yourself the following questions:
<ul class="itemizedlist">
<li class="listitem">
<span class="simpara">
Will this script only affect the intended files?
</span>
</li>
<li class="listitem">
<span class="simpara">
Can unusual or undesirable data be acted upon?
</span>
</li>
<li class="listitem">
<span class="simpara">
Can this script be used in unintended ways?
</span>
</li>
<li class="listitem">
<span class="simpara">
Can this be used in conjunction with other scripts in a negative
manner?
</span>
</li>
<li class="listitem">
<span class="simpara">
Will any transactions be adequately logged?
</span>
</li>
</ul>
</p>
<p class="para">
By adequately asking these questions while writing the script,
rather than later, you prevent an unfortunate re-write when you
need to increase your security. By starting out with this mindset,
you won't guarantee the security of your system, but you can help
improve it.
</p>
<p class="para">
You may also want to consider turning off register_globals,
magic_quotes, or other convenience settings which may confuse
you as to the validity, source, or value of a given variable.
Working with <acronym title="PHP: Hypertext Preprocessor">PHP</acronym> in error_reporting(E_ALL) mode can also help warn
you about variables being used before they are checked or
initialized (so you can prevent unusual data from being
operated upon).
</p>
</div>
<?php manual_footer(); ?>
