PHP 简单的标头注入预防

Question

我试图通过循环遍历所有 _POST 数据来防止邮件标头注入，但变量未通过我的验证。我有一些包含斜杠的“日期”字段，其中一些字段可能为空白。不确定这是否与它有关。谁能看出我的逻辑有问题？我不断收到我的“失败”信息。谢谢。

if(isset($_POST['submit'])) {

    $boolValidateOK = 1;

    function safe( $name ) {
       return( str_ireplace(array( "\r", "\n", "%0a", "%0d", "Content-Type:", "bcc:","to:","cc:" ), "", $name ) );
    }

    foreach($_POST as $value){
        if(!safe($value)){
            $boolValidateOK = 0;
        }else{
            $boolValidateOK = 1;
        }
    }

    if($boolValidateOK == 1){

        $headers  = 'MIME-Version: 1.0' . "\r\n";
        $headers .= 'Content-type: text/html; charset=iso-8859-1' . "\r\n";
        $to...etc
        $subject...etc
        $message...etc

        mail($to, $subject, $message, $headers);

        $success_message = "win";
    }else{
        $error_message = "fail";
    }
}

Answer 1

你正在做的事情是不必要的。注意PHP的mail()函数签名：

bool mail ( string $to , string $subject , string $message 
              [, string $additional_headers [, string $additional_parameters ]] )

收件人是第二个参数。标题都在第四个参数中。 $message 或 $subject 的内容不会神奇地“溢出”到其他参数。

由于您的$headers 不依赖于用户输入，因此用户是否输入并不重要。不可能有这样的“注入”。你所做的只是扼杀信息。

PS：这段代码是错误的：

foreach($_POST as $value){
    if(!safe($value)){
        $boolValidateOK = 0;
    }else{
        $boolValidateOK = 1;
    }
}

if($boolValidateOK == 1){

考虑一下。假设您所有的帖子值都不是“安全的”，但最后一个迭代的是。然后它将$boolValidateOK 覆盖为1，这就是循环结束时它将保留的值。

另外，PHP 也有布尔类型。如果您有布尔值，请将它们设置为 true 或 false 而不是 1 和 0。这样更清楚。

---

无论如何，我建议使用PHPMailer 之类的东西。原生的mail()函数用起来很别扭。