示例代码
搭建环境：
nginx（1.24.0）+php（5.6.40）
session_start();
function client_ip(){ return !empty($_SERVER['HTTP_X_FORWARDED_FOR']) ? $_SERVER['HTTP_X_FORWARDED_FOR'] : $_SERVER['REMOTE_ADDR'];}
$ip = client_ip();if(!filter_var($ip, FILTER_VALIDATE_IP) || !in_array($ip, array('localhost', '127.0.0.1'))){ die(htmlspecialchars("Not allowed!n"));}
if(!isset($_SESSION['auth'])){ header("Location: error.php");}
echo call_user_func($_GET['cmd'], $_GET['arg']);?>
请查看上述代码，找出其中的漏洞。
漏洞位置
if(!filter_var($ip, FILTER_VALIDATE_IP) || !in_array($ip, array('localhost', '127.0.0.1'))){ die(htmlspecialchars("Not allowed!n"));}echo call_user_func($_GET['cmd'], $_GET['arg']);
漏洞原理
代码验证X-Forwarded-For标头不够严格，攻击者可以伪造X-Forwarded-For：127.0.0.1进行绕过检测。攻击者在$_session[‘auth’]中没有有效的会话，这会自动导致重定向到另一个页面。重定向之后代码会继续向下运行，因而重定向没有任何效果，代码运行到call_user_func（），它允许执行任意的PHP代码或命令。
漏洞复现
Burpsuite抓包，更改请求包中的
X-Forwarded-For：127.0.0.1
提交参数：?cmd=exec&arg=pwd
修复方案
对于
`$_SERVER[‘HTTP_X_FORWARDED_FOR’]`伪造的问题，应该检查`$_SERVER[‘REMOTE_ADDR’]`和`$_SERVER[‘HTTP_CLIENT_IP’]`。然而，这仍然不能保证IP地址的完全准确，因为代理服务器可能会提供这些头部。最安全的做法可能是通过应用程序的登录过程来验证用户的身份，而不是依赖IP地址。
对于call_user_func函数造成的命令执行漏洞，需要对`$_GET[‘cmd’]`和`$_GET[‘arg’]`进行严格的过滤和验证。只允许执行白名单中的命令，并且只允许这些命令接受预定义的参数。对输出可以使用`htmlspecialchars()`函数来转义。
相关知识
call_user_func
call_user_func 是 PHP 中的一个函数，它用于调用由用户定义的回调函数。这个函数可以调用任何可调用的 PHP 对象，包括函数、方法、闭包等。
call_user_func 的语法如下：
call_user_func(callable $callback, ...$args)参数说明：
- $callback：要调用的回调函数。这是一个必需参数，可以是一个函数名、方法名、闭包等。
- …$args：要传递给回调函数的参数列表。这是一个可选参数，你可以根据需要传递任意数量的参数。
call_user_func 函数会调用 $callback 指定的回调函数，并将传递的参数列表作为回调函数的参数。它会返回回调函数执行的结果。
作者：Spider-Man
2023年12月8日
洞源实验室
