PHP 小心urldecode引发的SQL注入漏洞

ihipop 学校的 discuz x1.5 论坛被黑，在那里吵了一个下午。google 一下“discuz! x1-1.5 notify_credit.php blind sql injection exploit”，你就知道。

discuz 是国内很流行的论坛系统，被黑的网站应该会很多吧。不过我对入侵别人的网站不感兴趣，同时也鄙视那些代码都不会写只会使用别人放出的工具攻击的所谓的“黑客”。


粗略看了一下代码，这个 sql 注入漏洞是 urldecode 函数造成的。在 php 手册中，urldecode 函数下面有一个警告：

the superglobals $_get and $_request are already decoded. using urldecode() on an element in $_get or $_request could have unexpected and dangerous results.

而 discuz 的开发人员（估计是新手）画蛇添足，多加了一个 urldecode：

foreach($_post as $k => $v) {
$value = urldecode($v);
$this->setparameter($k, $value);
}

单引号被 urlencode 两次以后是 %2527，然后 post，php 内部在生成全局变量 $_post 的时候会先 urldecode，得到 %27，然后 php 会检查 magic quotes 的设置，但是无论是否开启 magic quotes，%27 都不会被 addslashes，因为这时根本没有单引号。但是这时如果你在 php 代码中画蛇添足的加上 urldecode，%27就变成单引号了，然后……你懂的。

在我初学 php 的时候，看的是学校图书馆的一本烂书，里面根本就没写 php 在处理表单的时候会自动 urldecode，所以自己用 urldecode 函数来解码（依稀记得书上好像也是这么写的，真是误人子弟啊）。

总结一下，就是：1、选择一本好书非常重要；2、慎用 urldecode 函数。3、注意 php 手册中的警告。
原文来自