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

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

缺陷编号:wooyun-2013-046471

漏洞标题:Thinksaas SQL注入#3

相关厂商:thinksaas.cn

漏洞作者: xfkxfk

提交时间:2013-12-19 16:40

修复时间:2014-03-16 16:41

公开时间:2014-03-16 16:41

漏洞类型:SQL注射漏洞

危害等级:高

自评Rank:20

漏洞状态:漏洞已经通知厂商但是厂商忽略漏洞

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

Tags标签:

4人收藏 收藏
分享漏洞:


漏洞详情

披露状态:

2013-12-19: 细节已通知厂商并且等待厂商处理中
2013-12-19: 厂商主动忽略漏洞,细节向第三方安全合作伙伴开放
2014-02-12: 细节向核心白帽子及相关领域专家公开
2014-02-22: 细节向普通白帽子公开
2014-03-04: 细节向实习白帽子公开
2014-03-16: 细节向公众公开

简要描述:

Thinksaas SQL注入#3

详细说明:

在Thinksaas 的小组功能处,发完贴后,可以补贴。
在编辑补贴时,过滤不完整,导致SQL注入,直接可以注入出数据,显示到帖子内容中。
/app/group/action/after.php文件,在边编辑补贴内容时:

//编辑执行
case "edo":

if($_POST['token'] != $_SESSION['token']) {
tsNotice('非法操作!');
}

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

$strAfter = $new['group']->find('group_topic_add',array(
'id'=>$afterid,
));

if($strAfter['userid'] == $userid || $TS_USER['user']['isadmin']==1){

$content = tsClean($_POST['content']);
$title = tsClean($_POST['title']);

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

$new['group']->update('group_topic_add',array(
'id'=>$afterid,
),array(
'title'=>$title,
'content'=>$content,
'uptime'=>time(),
));


利用tsClean函数过滤,/thinksaas/tsFunction.php:

/**
* 过滤脚本代码
* @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|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;
}
/**
* 输入安全过滤
* @param unknown $text
* @return mixed
*/
function tsClean($text) {
$text = cleanJs ( $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 );
}


我们输入的内容带入了update的row中,没有过滤,导致sql注入。

漏洞证明:

先发一个帖子,然后跟一个补贴。
在来编辑方才跟的那个补贴内容:

1.png


看看数据库的执行日志:

2.png


再来看我们跟的那个补贴的内容发生的变化:

3.png


成功注入。

修复方案:

过滤

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


漏洞回应

厂商回应:

危害等级:无影响厂商忽略

忽略时间:2014-03-16 16:41

厂商回复:

经官方人员测试,只是内容输入,不存在sql注入后被修改问题。

最新状态:

2013-12-20:非常感谢@xfkxfk的漏洞反馈,经再次确认,漏洞存在,请管理补上20rank,谢谢。此漏洞已经处理。


漏洞评价:

评论

  1. 2013-12-19 20:25 | xfkxfk 认证白帽子 ( 核心白帽子 | Rank:2179 漏洞数:338 | 呵呵!)

    @ThinkSAAS 是真的我看错了还是你们看错了?

  2. 2013-12-19 20:30 | 未来还没来 ( 路人 | Rank:16 漏洞数:5 | 每个人都期望成为主角,却不知道世界上从来...)

    有图有真相、下次贴图

  3. 2013-12-19 20:39 | xfkxfk 认证白帽子 ( 核心白帽子 | Rank:2179 漏洞数:338 | 呵呵!)

    @未来还没来 有代码,有图,有真相。官方svn上也存在的。估计他们测试错了