[CVE-2014-8959] phpmyadmin任意文件包含漏洞分析

首发360播报：http://bobao.360.cn/learning/detail/113.html

最近写的文章比较喜欢投递到各大平台，一是能赚点学费养家糊口，二是提高一下原创性。我发现如果文章发到一些社区或直接发博客，知名度不高很容易被转载以后不署名，导致转来转去后来就不知道谁是作者了。写了这么久文章也没给博客带来什么知名度，大概也就是因此吧。不如先发到各大平台再发到自己博客，总之也能赚点钱。

有时候好东西不是不愿分享，就是国内分享环境太差，开源程序不遵守协议、转载文章不带链接、插件模板随便去除版权，屡屡皆是。特别是“红黑联盟”等百度SEO做的比较好的网站，爬下来东西从来不带链接，到头来我是转载他是原创，那么我分享的意义何在，就是在为嘿产妞们铺平道路么？

这段时间和牙僧哥都在对open_basedir做一些研究，也发现了一些好玩的东西，php全版本已经可以绕过open_basedir读文件、列目录了，不知道离写文件还有多远，还是很期待的。

0x01 补丁分析

看到bobao.360.cn上提到了这个漏洞，于是我写个小分析吧，给渗透正没思路的人一个思路，也给学习代码审计的朋友一点资料。

前几天phpmyadmin出了个新的补丁，地址在此：http://www.phpmyadmin.net/home_page/security/PMASA-2014-14.php

修复了一个phpmyadmin4.x版本中的任意文件包含漏洞，我们看一下4.0版本的补丁：https://github.com/phpmyadmin/phpmyadmin/commit/2e3f0b9457b3c8f78beb864120bd9d55617a11b5

在文件libraries/gis/pma_gis_factory.php中对$type_lower多加了个判断。由此我们可以猜测，文件包含的点就出在$type_lower这里。

0x02 漏洞代码分析

我们来到libraries/gis/pma_gis_factory.php 29行：

<?php
public static function factory($type)
{
    include_once './libraries/gis/pma_gis_geometry.php';

    $type_lower = strtolower($type);
    if (! file_exists('./libraries/gis/pma_gis_' . $type_lower . '.php')) {
        return false;
    }
    if (include_once './libraries/gis/pma_gis_' . $type_lower . '.php') {
        switch(strtoupper($type)) {
        case 'MULTIPOLYGON' :
            return PMA_GIS_Multipolygon::singleton();
        case 'POLYGON' :
            return PMA_GIS_Polygon::singleton();
        case 'MULTIPOINT' :
            return PMA_GIS_Multipoint::singleton();
        case 'POINT' :
            return PMA_GIS_Point::singleton();
        case 'MULTILINESTRING' :
            return PMA_GIS_Multilinestring::singleton();
        case 'LINESTRING' :
            return PMA_GIS_Linestring::singleton();
        case 'GEOMETRYCOLLECTION' :
            return PMA_GIS_Geometrycollection::singleton();
        default :
            return false;
        }
    } else {
        return false;
    }
}

将传入的参数$type转换小写以后赋值给$type_lower，直接拼接成路径进行include_once。

我们来搜一下factory这个函数：

很多地方在调用，但最直接的还是/gis_data_editor.php，进来看看：

<?php
// Get data if any posted
$gis_data = array();
if (PMA_isValid($_REQUEST['gis_data'], 'array')) {
    $gis_data = $_REQUEST['gis_data'];
}

$gis_types = array(
    'POINT',
    'MULTIPOINT',
    'LINESTRING',
    'MULTILINESTRING',
    'POLYGON',
    'MULTIPOLYGON',
    'GEOMETRYCOLLECTION'
);

// Extract type from the initial call and make sure that it's a valid one.
// Extract from field's values if availbale, if not use the column type passed.
if (! isset($gis_data['gis_type'])) {
    if (isset($_REQUEST['type']) && $_REQUEST['type'] != '') {
        $gis_data['gis_type'] = strtoupper($_REQUEST['type']);
    }
    if (isset($_REQUEST['value']) && trim($_REQUEST['value']) != '') {
        $start = (substr($_REQUEST['value'], 0, 1) == "'") ? 1 : 0;
        $gis_data['gis_type'] = substr(
            $_REQUEST['value'], $start, strpos($_REQUEST['value'], "(") - $start
        );
    }
    if ((! isset($gis_data['gis_type']))
        || (! in_array($gis_data['gis_type'], $gis_types))
    ) {
        $gis_data['gis_type'] = $gis_types[0];
    }
}
$geom_type = $gis_data['gis_type'];

// Generate parameters from value passed.
$gis_obj = PMA_GIS_Factory::factory($geom_type);

首先$gis_data = $_REQUEST['gis_data'];获取到gis_data，判断$gis_data['gis_type']是否已经存在，如果存在则跳过那一大串if子句。最后就将$gis_data['gis_type'];赋值给$geom_type，并传入PMA_GIS_Factory::factory函数。

实际这个利用方法很简单，简单到其实就是获取$_REQUEST['gis_data']['gis_type']并拼接到include_once中，造成任意文件包含。

0x03 利用过程及POC

那我们来说说利用。这个漏洞为何没火，因为在我看来他需要两个条件：

1.登录到phpmyadmin
2.需要截断

相对比较鸡肋。但实际上这两个条件也不难满足，很多时候我们通过任意文件可能能够获得某些数据库的访问权限，我们通过这个漏洞就能成功提权。

首先我的测试环境为php 5.2.17 + phpmyadmin 4.0.3 （想想我为什么选这样的环境）

创建一个普通用户test，没有任何权限，登录后只能看到test和information_schema表：

构造好URL直接访问（pma的上层目录放着一个包含phpinfo()的图片马u1.gif）：

居然一片空白，没有出现我想要的phpinfo！？

这又涉及到phpmyadmin的一个防御CSRF机制了，来到libraries/common.inc.php 463行：

<?php
$token_mismatch = true;
if (PMA_isValid($_REQUEST['token'])) {
    $token_mismatch = ($_SESSION[' PMA_token '] != $_REQUEST['token']);
}

if ($token_mismatch) {
    /**
     *  List of parameters which are allowed from unsafe source
     */
    $allow_list = array(
        /* needed for direct access, see FAQ 1.34
         * also, server needed for cookie login screen (multi-server)
         */
        'server', 'db', 'table', 'target', 'lang',
        /* Session ID */
        'phpMyAdmin',
        /* Cookie preferences */
        'pma_lang', 'pma_collation_connection',
        /* Possible login form */
        'pma_servername', 'pma_username', 'pma_password',
        /* Needed to send the correct reply */
        'ajax_request',
        /* Permit to log out even if there is a token mismatch */
        'old_usr'
    );
    /**
     * Allow changing themes in test/theme.php
     */
    if (defined('PMA_TEST_THEME')) {
        $allow_list[] = 'set_theme';
    }
    /**
     * Require cleanup functions
     */
    include './libraries/cleanup.lib.php';
    /**
     * Do actual cleanup
     */
    PMA_remove_request_vars($allow_list);

}

他检查了$_SESSION[' PMA_token ']是否等于$_REQUEST['token']，如果不等于，最后会进入PMA_remove_request_vars函数，进去看看：

<?php
function PMA_remove_request_vars(&$whitelist)
{
    // do not check only $_REQUEST because it could have been overwritten
    // and use type casting because the variables could have become
    // strings
    $keys = array_keys(
        array_merge((array)$_REQUEST, (array)$_GET, (array)$_POST, (array)$_COOKIE)
    );

    foreach ($keys as $key) {
        if (! in_array($key, $whitelist)) {
            unset($_REQUEST[$key], $_GET[$key], $_POST[$key], $GLOBALS[$key]);
        } else {
            // allowed stuff could be compromised so escape it
            // we require it to be a string
            if (isset($_REQUEST[$key]) && ! is_string($_REQUEST[$key])) {
                unset($_REQUEST[$key]);
            }
            if (isset($_POST[$key]) && ! is_string($_POST[$key])) {
                unset($_POST[$key]);
            }
            if (isset($_COOKIE[$key]) && ! is_string($_COOKIE[$key])) {
                unset($_COOKIE[$key]);
            }
            if (isset($_GET[$key]) && ! is_string($_GET[$key])) {
                unset($_GET[$key]);
            }
        }
    }
}