Question

我已经开始使用ADOdb和参数化查询（例如$ db-＆gt;执行（“SELECT FROM users WHERE user_name =？;”，array（$ get-＆gt; id）;）以防止SQL注入。我有阅读这个假设是为了保护你在MySQL注入方面的事情，但显然不是XSS。虽然可能是这种情况，但我仍然对此持怀疑态度。

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

虽然我一般将此问题隔离到sanitize函数的mysql_real_escape_string部分，请注意我的服务器运行的是PHP 5.2+，当我使用自己的简化数据库抽象类时，这个问题不存在。此外，该网站主要是我自己的代码运行，而不是建立在一些预先存在的CMS的支架上。因此，考虑到这些因素，我唯一的猜测是有一些双重逃避。但是，当我查看adodb.inc.php文件时，我注意到$ rs-＆gt; 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!@#$%^&amp;*()';
      $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

Answer 1

事实证明，这是qstr在ADOdb中的副作用，即使我没有引用该类的特定功能，但必须在别处调用。在我的特定情况下的问题是启用了魔术引号，因此我将该函数的默认参数设置为$ magic_quotes = disabled。至于不需要任何转义，我发现ADOdb本身并没有通过基本的Execute（）单独绑定使用mysql_real_escape_string！我是如何认识到这是因为字符“'引发了错误（因此在我的服务器上没有显示error_reporting被禁用）。看起来这些函数的组合与修复ADOdb的小问题让我得到了很好的保护，并以我想要的方式接受大部分/全部输入：在双引号的情况下，禁止将任何引号作为内容输入到数据库中，这至少意味着没有HTML

尽管如此，我感谢您的建议，但也感到我的后续行动可能会对其他人有所帮助。

\ r \ n \“打印出来

1 个答案: