2012-03-14 51 views
1

我正在对使用简单的mysql_query()函数进行所有数据库查询的客户端网站进行一些维护。在他们的一个页面上,完成查询以基于他们的昵称来提取用户信息。该文件非常强大,并且通过更改每个实例以从用户标识中取出昵称并非真正可行。Mysql/PHP查询包含保留字的表值

他们遇到了一些昵称的问题,特别是“链接”,“回声”,“杀手”。我明白了为什么链接和回显可能会导致查询出现问题,但不是那么多Slayer。有什么我可以做的(除了防止将来创建这些名称)来帮助查询完成并提取我需要的信息?

编辑: 全功能:

function userInfo($username){ 
    global $username_array; 
    $username = prepare($username); 
    $username_array = mysql_fetch_array(mysql_query("SELECT * FROM `users` WHERE `name` = '$username' LIMIT 1")); 
} 

应该返回数组$ username_array回到原来的脚本。有99%的用户,这工作正常。出于某种原因,上面的用户,这完成不了。

function prepare($val,$type=0){ 
$val = XSS($val); 
$val = sqlInjection($val); 
return $val; 

}

function XSS($val) { 
    // remove all non-printable characters. CR(0a) and LF(0b) and TAB(9) are allowed 
    $val = preg_replace('/([\x00-\x08][\x0b-\x0c][\x0e-\x20])/', '', $val); 

    // straight replacements, the user should never need these since they're normal characters 
    // this prevents like <IMG SRC=&#X40&#X61&#X76&#X61&#X73&#X63&#X72&#X69&#X70&#X74&#X3A&#X61&#X6C&#X65&#X72&#X74&#X28&#X27&#X58&#X53&#X53&#X27&#X29> 
    $search = 'abcdefghijklmnopqrstuvwxyz'; 
    $search .= 'ABCDEFGHIJKLMNOPQRSTUVWXYZ'; 
    $search .= '[email protected]#$%^&*()'; 
    $search .= '~`";:?+/={}[]-_|\'\\'; 
    for ($i = 0; $i < strlen($search); $i++) { 
     // ;? matches the ;, which is optional 
     // 0{0,7} matches any padded zeros, which are optional and go up to 8 chars 
     // &#x0040 @ search for the hex values 
     $val = preg_replace('/(&#[x|X]0{0,8}'.dechex(ord($search[$i])).';?)/i', $search[$i], $val); // with a ; 
     // &#00064 @ 0{0,7} matches '0' zero to seven times 
     $val = preg_replace('/(&#0{0,8}'.ord($search[$i]).';?)/', $search[$i], $val); // with a ; 
    } 

    // now the only remaining whitespace attacks are \t, \n, and \r 
    $ra1 = Array('javascript', 'vbscript', 'expression', 'applet', 'blink', 'script', 'iframe', 'frameset', 'ilayer', 'bgsound'); 
    $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; // keep replacing as long as the previous round replaced something 
    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 .= '(&#[x|X]0{0,8}([9][a][b]);?)?'; 
       $pattern .= '|(&#0{0,8}([9][10][13]);?)?'; 
       $pattern .= ')?'; 
      } 
      $pattern .= $ra[$i][$j]; 
     } 
     $pattern .= '/i'; 
     $replacement = substr($ra[$i], 0, 2).'<x>'.substr($ra[$i], 2); // add in <> to nerf the tag 
     $val = preg_replace($pattern, $replacement, $val); // filter out the hex tags 
     if ($val_before == $val) { 
      // no replacements were made, so exit the loop 
      $found = false; 
     } 
     } 
    } 
    return $val; 
} 

function sqlInjection($val){ 
    if (get_magic_quotes_gpc()){ 
    $val = stripslashes($val); 
    } 
    if(version_compare(phpversion(),"4.3.0") == "-1"){ 
    return mysql_escape_string($val); 
    }else{ 
    return mysql_real_escape_string($val); 
    } 
} 
+2

什么是查询? – rid 2012-03-14 18:34:11

+3

永远不应该有这样一种情况,即用户名是一个保留字可能会导致您的查询失败,除非您做了非常*错误的事情。 – 2012-03-14 18:36:58

+0

查询:$ username_array = mysql_fetch_array(mysql_query(“SELECT * FROM'users' WHERE'name' ='$ username'LIMIT 1”));当你回显查询字符串时,它是SELECT * FROM'用户'WHERE'name' ='Slayer'LIMIT 1“ – Eric 2012-03-14 18:39:16

回答

1

应该有,因为没有用户输入都不应直接执行没有问题。

请确保你正确地逃避了字符串,并考虑使用准备好的语句,因为有人有一个令人讨厌的惊喜等待,如this

+0

哈哈,我喜欢那个特别的xkcd。遗憾的是,该网站并未使用准备好的语句,但在执行任何查询之前都有一个彻底的字符串清理功能。 – Eric 2012-03-14 19:11:32

+0

@Eric,如果你不使用预处理语句,你应该使用['mysql_real_escape_string()'](http://php.net/mysql_real_escape_string)。任何其他清洁都是可选的,但这是强制性的。 – rid 2012-03-14 19:45:32

+0

这已经是清理的一部分,并且已经在上面的$ username变量上执行过。 – Eric 2012-03-14 19:46:26