当前位置:WooYun >> 漏洞信息

漏洞概要 关注数(24) 关注此漏洞

缺陷编号:wooyun-2014-051473

漏洞标题:ThinkSAAS 最新版SQL注入之一

相关厂商:thinksaas.cn

漏洞作者: xfkxfk

提交时间:2014-02-20 10:03

修复时间:2014-05-21 10:04

公开时间:2014-05-21 10:04

漏洞类型:SQL注射漏洞

危害等级:中

自评Rank:20

漏洞状态:厂商已经确认

漏洞来源: http://www.wooyun.org,如有疑问或需要帮助请联系 [email protected]

Tags标签:

4人收藏 收藏
分享漏洞:


漏洞详情

披露状态:

2014-02-20: 细节已通知厂商并且等待厂商处理中
2014-02-20: 厂商已经确认,细节仅向厂商公开
2014-02-23: 细节向第三方安全合作伙伴开放
2014-04-16: 细节向核心白帽子及相关领域专家公开
2014-04-26: 细节向普通白帽子公开
2014-05-06: 细节向实习白帽子公开
2014-05-21: 细节向公众公开

简要描述:

ThinkSAAS 最新版2.1,官方2月15日更新,SQL注入第一弹

详细说明:

文件/app/photo/action/album.php

//批量修改执行
case "info_do":

//用户是否登录
$userid = aac('user')->isLogin();

$albumid = intval($_POST['albumid']);

$albumface = tsClean($_POST['albumface']);//进行了过滤,但未过滤完全

$arrPhotoId = intval($_POST['photoid']);
$arrPhotoDesc = $_POST['photodesc'];

if($TS_USER['user']['isadmin']==0){

foreach($arrPhotoDesc as $key=>$item){

//过滤内容开始
aac('system')->antiWord($item);
//过滤内容结束

}
}

foreach($arrPhotoDesc as $key=>$item){
if($item){
$photoid = intval($arrPhotoId[$key]);

$new['photo']->update('photo',array(
'photoid'=>$photoid,
),array(

'photodesc'=>tsClean($item),

));

}
}

//更新相册封面
if($albumface){
$new['photo']->update('photo_album',array(
'userid'=>$userid,
'albumid'=>$albumid,
),array(
'albumface'=>$albumface,//带入数据库查询导致SQL注入
));
}

header("Location: ".tsUrl('photo','album',array('id'=>$albumid)));

break;


虽然进行了tsClean过滤,但是我们来看看函数:

function tsClean($text) {
$text = cleanJs ( $text );
return $text;
}
/**
* 过滤脚本代码
* @param unknown $text
* @return mixed
*/
function cleanJs($text) {
$text = trim ( $text );
$text = stripslashes ( $text );
// 完全过滤注释
$text = preg_replace ( '/<!--?.*-->/', '', $text );
// 完全过滤动态代码
$text = preg_replace ( '/<\?|\?>/', '', $text );
// 完全过滤js
$text = preg_replace ( '/<script?.*\/script>/', '', $text );
// 过滤多余html
$text = preg_replace ( '/<\/?(html|head|meta|link|base|body|title|style|script|form|iframe|frame|frameset)[^><]*>/i', '', $text );
// 过滤on事件lang js
while ( preg_match ( '/(<[^><]+)(lang|data|onfinish|onmouse|onexit|onerror|onclick|onkey|onload|onchange|onfocus|onblur)[^><]+/i', $text, $mat ) ) {
$text = str_replace ( $mat [0], $mat [1], $text );
}
while ( preg_match ( '/(<[^><]+)(window\.|javascript:|js:|about:|file:|document\.|vbs:|cookie)([^><]*)/i', $text, $mat ) ) {
$text = str_replace ( $mat [0], $mat [1] . $mat [3], $text );
}
return $text;
}


只是用来过滤js代码。
再来看看update的处理:

public function update($table, $conditions, $row) {
$where = "";
if (empty ( $row ))
return FALSE;
if (is_array ( $conditions )) {
$join = array ();
foreach ( $conditions as $key => $condition ) {
$condition = $this->escape ( $condition );
$join [] = "{$key} = {$condition}";
}
$where = "WHERE " . join ( " AND ", $join );
} else {
if (null != $conditions)
$where = "WHERE " . $conditions;
}
foreach ( $row as $key => $value ) {
$vals [] = "`$key` = '$value'";
}
$values = join ( ", ", $vals );
$sql = "UPDATE " . dbprefix . "{$table} SET {$values} {$where}";

return $this->db->query ( $sql );
}


虽然在conditions处做了转义但是在row处没有处理,这里正好在row处,导致SQL注入。

漏洞证明:

构造请求如图,即可:

1.png

修复方案:

严格过滤啊

版权声明:转载请注明来源 xfkxfk@乌云


漏洞回应

厂商回应:

危害等级:高

漏洞Rank:20

确认时间:2014-02-20 10:29

厂商回复:

谢谢反馈,已经修复。

最新状态:

暂无


漏洞评价:

评论

  1. 2014-02-20 12:55 | phith0n 认证白帽子 ( 核心白帽子 | Rank:656 漏洞数:107 | 一个想当文人的黑客~)

    我发的乌云就没审核。。改交360了

  2. 2014-02-20 12:58 | phith0n 认证白帽子 ( 核心白帽子 | Rank:656 漏洞数:107 | 一个想当文人的黑客~)

    @xsser 我搞不懂同一个cms的(我估计是一个漏洞),两个人发就是不一样,我17号发的,到现在没审核。是不是审核的人不是看漏洞审核,是看人审核

  3. 2014-02-20 13:01 | xfkxfk 认证白帽子 ( 核心白帽子 | Rank:2179 漏洞数:338 | 呵呵!)

    @phith0n 没审核通过?

  4. 2014-02-20 13:03 | phith0n 认证白帽子 ( 核心白帽子 | Rank:656 漏洞数:107 | 一个想当文人的黑客~)

    @xfkxfk 一直没审核。后来我就交到360去了

  5. 2014-02-20 13:05 | xfkxfk 认证白帽子 ( 核心白帽子 | Rank:2179 漏洞数:338 | 呵呵!)

    @phith0n http://loudong.360.cn/vul/info/id/3844恭喜

  6. 2014-02-20 13:09 | phith0n 认证白帽子 ( 核心白帽子 | Rank:656 漏洞数:107 | 一个想当文人的黑客~)

    @xfkxfk 等于说交到两个地方了……抱歉刚才有点小激动……

  7. 2014-02-20 20:06 | ′ 雨。 ( 普通白帽子 | Rank:1231 漏洞数:190 | Only Code Never Lie To Me.)

    @phith0n 师傅 顶你。 有时候得发个短消息催催。

  8. 2014-02-20 20:29 | xfkxfk 认证白帽子 ( 核心白帽子 | Rank:2179 漏洞数:338 | 呵呵!)

    @′ 雨。 @phith0n 淡定,我去年的漏洞还没审核

  9. 2014-10-10 11:45 | D&G ( 普通白帽子 | Rank:523 漏洞数:103 | going)

    @xfkxfk @phith0n 请教个问题,看 WooYun: Thinksaas最新版注入无视GPC 这里这个程序2.1版应该就已经有全局的get。post。cookie addslash处理了。这样的话即使update没有进行处理,这里不应该也是无法引入单引号的么?现在2.2版的程序update里也进行了一次escape,所以会进行两次转义处理了。