不要轻信 PHP_SELF

作者:手气不错 发布时间:November 11, 2008 分类:PHP

开门见山,考虑下面的代码(原文连接有详细的解释)

<html>
    <body>
        <?php
            if (isset($_REQUEST['submitted']) && $_REQUEST['submitted'] == '1') {
                echo "Form submitted!";
            }
        ?>
        <form action="<?php echo $_SERVER['PHP_SELF']; ?>">
            <input type="hidden" name="submitted" value="1" />
            <input type="submit" value="Submit!" />
        </form>
    </body>
</html>

看似准确无误的代码,但是暗藏着危险。让我们将其保存为 foo.php ,然后放到 PHP 环境中使用

foo.php/%22%3E%3Cscript%3Ealert('xss')%3C/script%3E%3Cfoo

访问,会发现弹出个 Javascript 的 alert -- 这很明显又是个 XSS 的注入漏洞。究其原因,发现是在

echo $_SERVER['PHP_SELF'];

这条语句上直接输出了未过滤的值。追根数源,我们看下 PHP 手册的描述

'PHP_SELF'

The filename of the currently executing script, relative to the document root. 
For instance, $_SERVER['PHP_SELF'] in a script at the address 
http://example.com/test.php/foo.bar would be /test.php/foo.bar. The __FILE__ 
constant contains the full path and filename of the current (i.e. included) file.
If PHP is running as a command-line processor this variable contains the script 
name since PHP 4.3.0. Previously it was not available. 

原因很明确了,原来是 $_SERVER['PHP_SELF'] 虽然“看起来”是服务器提供的环境变量,但这的确和 $_POST 与 $_GET 一样,是可以被用户更改的。

其它类似的变量有很多,比如 $_COOKIE 等(如果用户想“把玩”他们的 cookie,那我们也是没有办法)。解决方案很简单,使用 strip_tagshtmlentities 等此类函数过滤或者转义。

echo htmlentities($_SERVER['PHP_SELF']); 

-- Split --

上述的例子让我们需要时刻保持谨慎 coding 的心态。Chris Shiflett 在他的 Blog 总结的相当直白,防止 XSS 的两个基本的安全思想就是

Filter input
Escape output

我将上面翻译成 “过滤输入,转义输出”。详细的内容,可以参考他 Blog 的这篇文章,此处略。

已有 4 条回复

  1. 手气不错 November 11th, 2008 at 11:27 pm #1
    手气不错

    独行: 表单当要提交给自身的时候用: action="" 用 $_SERVER['PHP_SELF'] 会比较多余



    @独行 这只是个例子,用来说明 $_SERVER 这类的变量有部分是用户可以修改的

  2. blankyao November 12th, 2008 at 12:06 am #2
    blankyao

    Chris Shiflett 的blog不错,谢谢 :)

  3. 剑网3论坛 November 12th, 2008 at 03:34 pm #3
    剑网3论坛

    俺是新人

  4. Platinum November 12th, 2008 at 07:14 pm #4
    Platinum

    据说 $_SERVER["HTTP_X_FORWARDED_FOR"] 之类的也会被伪造

添加新回复

Yahoo 统计