逝去的青春
PHP SQL 注入攻击的技术实现以及预防办法
2012-12-18 jao

1. php 配置文件 php.ini 中的 magic_quotes_gpc 选项没有打开，被置为 off


　　2. 开发者没有对数据类型进行检查和转义


　　不过事实上，第二点最为重要。我认为, 对用户输入的数据类型进行检查，向 MYSQL 提交正确的数据类型，这应该是一个 web 程序员最最基本的素质。但现实中，常常有许多小白式的 Web 开发者忘了这点, 从而导致后门大开。


　　为什么说第二点最为重要？因为如果没有第二点的保证，magic_quotes_gpc 选项，不论为 on，还是为 off，都有可能引发 SQL 注入攻击。下面来看一下技术实现：


　一. magic_quotes_gpc = Off 时的注入攻击


　　magic_quotes_gpc = Off 是 php 中一种非常不安全的选项。新版本的 php 已经将默认的值改为了 On。但仍有相当多的服务器的选项为 off。毕竟，再古董的服务器也是有人用的。


　　当magic_quotes_gpc = On　时，它会将提交的变量中所有的 '(单引号)、"(双号号)、\(反斜线)、空白字符，都为在前面自动加上 \。下面是 php 的官方说明：







1 magic_quotes_gpc boolean






2  






3 Sets the magic_quotes state for GPC (Get/Post/Cookie) operations. When magic_quotes are on, all ' (single-quote), \" (double quote), \ (backslash) and NUL's are escaped with a backslash automatically




如果没有转义，即 off 情况下，就会让攻击者有机可乘。以下列测试脚本为例：


http://www.xiaohui.com/weekly/20070314.htm







01 if ( isset($_POST[\"f_login\"] ) )






02 {






03   // 连接数据库...






04   // ...代码略...






05   






06   // 检查用户是否存在






07   $t_strUname = $_POST[\"f_uname\"];






08   $t_strPwd = $_POST[\"f_pwd\"];






09   $t_strSQL = \"SELECT * FROM tbl_users WHERE username='$t_strUname' AND password = '$t_strPwd'LIMIT 0,1\";






10  






11   if ( $t_hRes = mysql_query($t_strSQL) )






12   {






13     // 成功查询之后的处理. 略...






14   }






15 }








01 if ( isset($_POST[\"f_login\"] ) )






02 {






03   // 连接数据库...






04   // ...代码略...






05   






06   // 检查用户是否存在






07   $t_strUid = $_POST[\"f_uid\"];






08   $t_strPwd = $_POST[\"f_pwd\"];






09   $t_strSQL = \"SELECT * FROM tbl_users WHERE uid=$t_strUid AND password = '$t_strPwd' LIMIT 0,1\";






10   if ( $t_hRes = mysql_query($t_strSQL) )






11   {






12     // 成功查询之后的处理. 略...






13   }






14  






15 }








01 $t_strSQL = \"SELECT a from b....\";






02 if ( mysql_query($t_strSQL) )






03 {






04   // 正确的处理






05 }






06 else






07 {






08   echo \"错误! SQL 语句：$t_strSQL \r\n错误信息\".mysql_query();






09   exit;






10 }




　　这种做法是相当危险和愚蠢的。如果一定要这么做，最好在网站的配置文件中，设一个全局变量或定义一个宏，设一下 debug 标志：


全局配置文件中：




01 <tt>$t_strSQL = \"SELECT a from b....\";






02 if ( mysql_query($t_strSQL) )






03 {






04   // 正确的处理






05 }






06 else






07 {






08   if (DEBUG_MODE)






09     echo \"错误! SQL 语句：$t_strSQL \r\n错误信息\".mysql_query();






10   exit;






11 }</tt>




  1. 对提交的 sql 语句，进行转义和类型检查。


　四. 我写的一个安全参数获取函数


　　为了防止用户的错误数据和 php + mysql 注入 ，我写了一个函数 PAPI_GetSafeParam()，用来获取安全的参数值：






01 define(\"XH_PARAM_INT\",0);






02 define(\"XH_PARAM_TXT\",1);






03 function PAPI_GetSafeParam($pi_strName, $pi_Def = \"\", $pi_iType = XH_PARAM_TXT)






04 {






05   if ( isset($_GET[$pi_strName]) )






06     $t_Val = trim($_GET[$pi_strName]);






07   else if ( isset($_POST[$pi_strName]))






08     $t_Val = trim($_POST[$pi_strName]);






09   else






10     return $pi_Def;






11  






12   // INT






13   if ( XH_PARAM_INT == $pi_iType)






14   {






15     if (is_numeric($t_Val))






16       return $t_Val;






17     else






18       return $pi_Def;






19   }






20    






21   // String






22   $t_Val = str_replace(\"&\", \"&\",$t_Val);






23   $t_Val = str_replace(\"<\", \"<\",$t_Val);






24   $t_Val = str_replace(\">\", \">\",$t_Val);






25   if ( get_magic_quotes_gpc() )






26   {






27     $t_Val = str_replace(\"\\\\"\", \"\"\",$t_Val);






28     $t_Val = str_replace(\"\\''\", \"'\",$t_Val);






29   }






30   else






31   {






32     $t_Val = str_replace(\"\\"\", \"\"\",$t_Val);






33     $t_Val = str_replace(\"'\", \"'\",$t_Val);






34   }






35   return $t_Val;






36 }








  • 　　在这个函数中，有三个参数：


    $pi_strName: 变量名
    
$pi_Def: 默认值
    
$pi_iType: 数据类型。取值为 XH_PARAM_INT, XH_PARAM_TXT, 分别表示数值型和文本型。    [/mw_shl_code] 
    　　如果请求是数值型，那么调用 is_numeric() 判断是否为数值。如果不是，则返回程序指定的默认值。

    

    　　简单起见，对于文本串，我将用户输入的所有危险字符（包括HTML代码），全部转义。由于 php 函数 
    
addslashes()存在漏洞，我用 str_replace()直接替换。get_magic_quotes_gpc() 函数是 php 
    
的函数，用来判断 magic_quotes_gpc 选项是否打开。

    

     

    

    　　刚才第二节的示例，代码可以这样调用：

    





    01
if ( isset($_POST[\"f_login\"] ) )

    
    


    


    





    02
{

    
    


    


    





    03
  // 连接数据库...

    
    


    


    





    04
  // ...代码略...

    
    


    


    





    05
  

    
    


    


    





    06
  // 检查用户是否存在

    
    


    


    





    07
  $t_strUid = PAPI_GetSafeParam(\"f_uid\", 0, XH_PARAM_INT);

    
    


    


    





    08
  $t_strPwd = PAPI_GetSafeParam(\"f_pwd\", \"\", XH_PARAM_TXT);

    
    


    


    





    09
  $t_strSQL = \"SELECT * FROM tbl_users WHERE uid=$t_strUid AND password = '$t_strPwd' LIMIT 0,1\";

    
    


    


    





    10
  if ( $t_hRes = mysql_query($t_strSQL) )

    
    


    


    





    11
  {

    
    


    


    





    12
    // 成功查询之后的处理. 略...

    
    


    


    





    13
  } 

    
    


    


    





    14
}

    
    


    


    





    15
 

    
    


    


    





    16
 

    
    


    


    





    17
　这样的话，就已经相当安全了。PAPI_GetSafeParam的代码有点长，但牺牲这点效率，对保证安全，是值得的。希望大家多批评指正。：）