复现禅道V17.4的sql注入漏洞

漏洞详情

　　简述：禅道是第一款国产的开源项目管理软件，它的核心管理思想基于敏捷方法 scrum，内置了产品管理和项目管理，同时又根据国内研发现状补充了测试管理、计划管理、发布管理、文档管理、事务管理等功能，在一个软件中就可以将软件研发中的需求、任务、bug、用例、计划、发布等要素有序地跟踪管理起来，完整地覆盖了项目管理的核心流程。

漏洞危害

　　该漏洞是由于禅道项目管理系统权限认证存在缺陷导致，攻击者可利用该漏洞在未授权的情况下，通过权限绕过在服务器执行sql进而可以写入webshell。

影响范围

• V17.4<= 禅道 <= V18.0.beta1（开源版）
• V3.4<= 禅道 <= V4.0.beta1（旗舰版）
• V7.4<= 禅道 <= V8.0.beta1（企业版）

复现过程

我下载了开源版V17.4的程序来进行代码审计

其中关键部分在于权限绕过，

代码文件：framework\base\control.class.php

if($this->config->installed && !in_array($this->moduleName, $this->config->openModules) && empty($this->app->user) && !$this->loadModel('common')->isOpenMethod($this->moduleName, $this->methodName))
{
    $uri = $this->app->getURI(true);
    if($this->moduleName == 'message' and $this->methodName == 'ajaxgetmessage')
    {
        $uri = helper::createLink('my');
    }
    elseif(helper::isAjaxRequest())
    {
        die(json_encode(array('result' => false, 'message' => $this->lang->error->loginTimeout)));
    }
 
    $referer = helper::safe64Encode($uri);
    die(js::locate(helper::createLink('user', 'login', "referer=$referer")));
}

empty($this->app->user) 处进行了权限判断，但只判断了user不为空的情况，而user的信息主要存在session中，所以只要能构造一个user的键即可。

对允许访问的模块和方法进行审计发现了在misc模块的captcha方法中可以伪造session，且只能控制键不能控制值，正好可以用来绕过权限认证：

代码文件：module\misc\control.php

public function captcha($sessionVar = 'captcha', $uuid = '')
{
    $obLevel = ob_get_level();
    for($i = 0; $i < $obLevel; $i++) ob_end_clean();
 
    header('Content-Type: image/jpeg');
    $captcha = $this->app->loadClass('captcha');
    $this->session->set($sessionVar, $captcha->getPhrase());
    $captcha->build()->output();
}

权限认证绕过之后，发现可以访问一部分代码，在对后台的功能审计时候发现，在导入数据时候的数据库判断存在sql注入

代码文件：module\convert\model.php

public function dbExists($dbName = '')
{
    $sql = "SHOW DATABASES like '{$dbName}'";
    return $this->dbh->query($sql)->fetch();
}

审查该段代码可以发现，存在sql注入

同文件下的该段代码一样存在sql注入，且为堆叠注入

public function connectDB($dbName = '')
{
    $dsn = "mysql:host={$this->config->db->host}; port={$this->config->db->port};dbname={$dbName}";
    try
    {
        $dbh = new PDO($dsn, $this->config->db->user, $this->config->db->password);
        $dbh->setAttribute(PDO::ATTR_DEFAULT_FETCH_MODE, PDO::FETCH_OBJ);
        $dbh->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
        $dbh->exec("SET NAMES {$this->config->db->encoding}");
        $this->sourceDBH = $dbh;
        return $dbh;
    }
    catch (PDOException $exception)
    {
        return $exception->getMessage();
    }
}

至此已复现完毕！

回顾权限验证函数

catch(EndResponseException $endResponseException)
{
    echo $endResponseException->getContent();
}

其中这个就是权限不满足时候执行的，原本这时候应该就要结束了，但是这里采用的是echo 所以导致了可以继续执行代码，所以只要上面的验证绕过，这里就可以继续执行。

在官方发布的新版中这里修改为了die，如下：

die($endResponseException->getContent());

代码不能继续执行了。

禅道的rce poc已经在github上发布了，作者在这就不做过多赘述，sql注入的poc已放上github：https://github.com/spmonkey/zentao_sql_poc

 

结语

本文只用于参考学习，增强自己的代码审计能力，一切用于违法犯罪的均与作者无关。

网络不是法外之地，请不要轻易挑战法律底线。