0

我已经开始使用 ADOdb 和参数化查询（例如 $db->Execute("SELECT FROM users WHERE user_name=?;",array($get->id);) 来防止 SQL 注入。我已经读过这是假设在 MySQL 注入方面保护你，但显然不是 XSS。虽然可能是这种情况，但我仍然对此持怀疑态度。

尽管如此，我总是在我的包装代码（kernel.php）的开头使用shot弹枪方法过滤我的环境变量以确保安全。我注意到使用 ADOdb 和以下函数的组合会产生浏览器可见的回车符 (\r\n \" \')，这是我不想要的（尽管我确实想存储该信息！）。我也不想在显示之前过滤我的输出，因为我已经正确过滤了我的输入（除了 BBcode 和那种东西）。下面你会找到我所指的功能。

虽然通常我已将此问题隔离到 sanitize 函数的 mysql_real_escape_string 部分，但请注意我的服务器正在运行 PHP 5.2+，并且当我使用自己的简化数据库抽象类时不存在此问题。此外，该站点主要是在我自己的代码上运行的，而不是建立在某些预先存在的 CMS 的脚手架上）。因此，考虑到这些因素，我唯一的猜测是发生了一些双重转义。但是，当我查看 adodb.inc.php 文件时，我注意到 $rs->FetchNextObj() 没有使用 mysql_real_escape_string。执行此操作的唯一函数似乎是 qstr，它封装了整个字符串。这让我担心依赖参数化查询可能还不够，但我不知道！

// Sanitize all possible user inputs

if(keyring_access("am")) // XSS and HTML stripping exemption for administrators editing HTML content
{
$_POST =    sanitize($_POST,false,false);
$_GET =     sanitize($_GET,false,false);
$_COOKIE =  sanitize($_COOKIE,false,false);
$_SESSION = sanitize($_SESSION,false,false);    
}
else
{   
$_POST =    sanitize($_POST);
$_GET =     sanitize($_GET);
$_COOKIE =  sanitize($_COOKIE);
$_SESSION = sanitize($_SESSION);
}

// Setup $form object shortcuts (merely convenience)

if($_POST)
{
foreach($_POST as $key => $value)
{
        $form->$key = $value;
}
}

if($_GET)
{
foreach($_GET as $key => $value)
{
    $get->$key = $value;
}
}


function sanitize($val, $strip = true, $xss = true, $charset = 'UTF-8')
{
  if (is_array($val))
  {
    $output = array();
    foreach ($val as $key => $data)
    {
      $output[$key] = sanitize($data, $strip, $xss, $charset);
    }
    return $output;
  }
  else
  {
    if ($xss)
    {
      // code by nicolaspar
      $val = preg_replace('/([\x00-\x08][\x0b-\x0c][\x0e-\x20])/', '', $val);
      $search = 'abcdefghijklmnopqrstuvwxyz';
      $search .= 'ABCDEFGHIJKLMNOPQRSTUVWXYZ';
      $search .= '1234567890!@#$%^&*()';
      $search .= '~`";:?+/={}[]-_|\'\\';

      for ($i = 0; $i < strlen($search); $i++)
      {
        $val = preg_replace('/(&amp;#[x|X]0{0,8}'.dechex(ord($search[$i])).';?)/i', $search[$i], $val); // with a ;
        $val = preg_replace('/(&amp;#0{0,8}'.ord($search[$i]).';?)/', $search[$i], $val); // with a ;
      }

      $ra1 = Array('javascript', 'vbscript', 'expression', 'applet', 'meta', 'xml', 'blink', 'link', 'style', 'script', 'embed', 'object', 'iframe', 'frame', 'frameset', 'ilayer', 'layer', 'bgsound', 'title', 'base');
      $ra2 = Array('onabort', 'onactivate', 'onafterprint', 'onafterupdate', 'onbeforeactivate', 'onbeforecopy', 'onbeforecut', 'onbeforedeactivate', 'onbeforeeditfocus', 'onbeforepaste', 'onbeforeprint', 'onbeforeunload', 'onbeforeupdate', 'onblur', 'onbounce', 'oncellchange', 'onchange', 'onclick', 'oncontextmenu', 'oncontrolselect', 'oncopy', 'oncut', 'ondataavailable', 'ondatasetchanged', 'ondatasetcomplete', 'ondblclick', 'ondeactivate', 'ondrag', 'ondragend', 'ondragenter', 'ondragleave', 'ondragover', 'ondragstart', 'ondrop', 'onerror', 'onerrorupdate', 'onfilterchange', 'onfinish', 'onfocus', 'onfocusin', 'onfocusout', 'onhelp', 'onkeydown', 'onkeypress', 'onkeyup', 'onlayoutcomplete', 'onload', 'onlosecapture', 'onmousedown', 'onmouseenter', 'onmouseleave', 'onmousemove', 'onmouseout', 'onmouseover', 'onmouseup', 'onmousewheel', 'onmove', 'onmoveend', 'onmovestart', 'onpaste', 'onpropertychange', 'onreadystatechange', 'onreset', 'onresize', 'onresizeend', 'onresizestart', 'onrowenter', 'onrowexit', 'onrowsdelete', 'onrowsinserted', 'onscroll', 'onselect', 'onselectionchange', 'onselectstart', 'onstart', 'onstop', 'onsubmit', 'onunload');

      $ra = array_merge($ra1, $ra2);
      $found = true;

      while ($found == true)
      {
        $val_before = $val;
        for ($i = 0; $i < sizeof($ra); $i++)
        {
          $pattern = '/';
          for ($j = 0; $j < strlen($ra[$i]); $j++)
          {
            if ($j > 0)
            {
              $pattern .= '(';
              $pattern .= '(&amp;#[x|X]0{0,8}([9][a][b]);?)?';
              $pattern .= '|(&amp;#0{0,8}([9][10][13]);?)?';
              $pattern .= ')?';
            }
            $pattern .= $ra[$i][$j];
          }
          $pattern .= '/i';
          $replacement = substr($ra[$i], 0, 2).'<x>'.substr($ra[$i], 2);
          $val = preg_replace($pattern, $replacement, $val);
          if ($val_before == $val)
          {
            $found = false;
          }
        }
      }
    }

    // Strip HTML tags
    if ($strip)
    {
      $val = strip_tags($val);
      // Encode special chars
      $val = htmlentities($val, ENT_QUOTES, $charset);
    }

    // Cross your fingers that we don't get a MySQL injection with relying on ADOdb prepared statements alone… ? It works great otherwise by just returning $val... so it appears the code below is the culprit of the \r\n \" etc. escaping
    //return $val;

    if(function_exists('get_magic_quotes_gpc') or get_magic_quotes_gpc())
    {
        return mysql_real_escape_string(stripslashes($val));
    }
    else
    {
        return mysql_real_escape_string($val);
    }
  }
}

非常感谢您的帮助！如果您需要任何进一步的说明，请告诉我。

更新反斜杠仍然出现在“和'前面，是的，我删除了额外的mysql_real_escape_string ...现在我只能认为这可能是get_quotes_gpc，或者ADOdb添加它们...

~elix

4

1 回答 1

0

事实证明这是 ADOdb 中 qstr 的副作用，即使我没有引用该类的特定函数，但必须在其他地方调用。在我的特殊情况下，问题是启用了魔术引号，因此我将函数的默认参数设置为 $magic_quotes=disabled。至于不需要对此进行任何转义，我发现 ADOdb 本身并不通过仅绑定的基本 Execute() 使用 mysql_real_escape_string ！我如何认识到这一点是由于字符“”引发了错误（因此没有在我的服务器上显示 error_reporting 被禁用）。似乎将这些功能与使用 ADOdb 修复这个小问题相结合让我都得到了很好的保护，并以我希望的方式接受大多数/所有输入：

尽管如此，我很欣赏你的建议，但也觉得我的后续行动可能会帮助其他人。

于 2011-08-16T06:35:27.943 回答