Dedecms include\dialog\select_soft_post.php Upload Any Files To The Specified Directory Via Variable Not Initial Flaw Bypass Extension Defence

浏览数：63 / 时间：2015年06月09日

1. 漏洞描述
2. 漏洞触发条件
3. 漏洞影响范围
4. 漏洞代码分析
5. 防御方法
6. 攻防思考

1. 漏洞描述

综合来说，这个漏洞的根源是"register_globals = on"，在这个漏洞的前提下，攻击者可以发动这样的攻击向量

1. 当前网站的"register_globals = on"已开启
2. 在代码中没有显式声明、初始化的变量
3. 在用户提交的HTML表单中提交了相同名字的字段
4. 在以上的前提下，黑客可以任意控制代码中变量的值

大体来说，黑客可以有两种利用、及入侵方式

1. 通过注入、覆盖变量$cfg_basedir，导致可以饶过身份认证和系统变量初始化文件，导致可以上传任意文件到指定目录
2. 通过注入、覆盖变量$cfg_not_allowall、$newname等字段，绕过文件扩展名检查防御代码，向目标文件夹直接上传WEBSHELL

这两种攻击向量的最终目的都是向服务端上传WEBSHELL(.PHP、.ASP、.htaccess)等文件

Relevant Link:

http://huaidan.org/archives/3386.html
http://sebug.net/vuldb/ssvid-12518
http://4byte.cn/learning/51122.html
https://code.google.com/p/webfiles/source/browse/files/localhost/htdocs/cxtx/include/dialog/select_soft_post.php?spec=svn40&r=17

2. 漏洞触发条件

1. register_globals = on: 网站本身的安全设置
2. 攻击者可以通过自定义表单为相关的变量赋值: 伪造HTTP数据包

3. 漏洞影响范围

1. Dedecms 5.5
2. DEDECMS v5.5 Final

4. 漏洞代码分析

/include/dialog/select_soft_post.php

Relevant Link:

http://www.verydemo.com/demo_c116_i56826.html

5. 防御方法

0x1: 升级到最新的DEDECMS版本

0x2: Patch Code

我们知道，这个漏洞的根源是因为PHP变量覆盖导致的恶意文件上传，没法直接针对性的防御，要进行代码级的防御，可以从堵住入侵GETSHELL的最终目的这个角度入手

1. 在代码流的相对末尾位置布置文件名检测
2. 采取正则匹配文件名中是否有".php、.asp"等特征，来达到防止WEBSHELL上传的目的

/include/dialog/select_soft_post.php

defence code

....
if (preg_match(‘#\.(php|pl|cgi|asp|aspx|jsp|php5|php4|php3|shtm|shtml)$#i‘, trim($filename))) 
{
    ShowMsg("你指定的文件名被系统禁止！",‘javascript:;‘);
    exit();
}
$fullfilename = $cfg_basedir.$activepath.‘/‘.$filename;
$fullfileurl = $activepath.‘/‘.$filename;
....

6. 攻防思考

0x1: 官方修复方案

文件上传属于WEB系统通用的功能，最佳的防御实践是将防御代码做成通用的代码模块，放置在WEB系统的关键节点位置，使用面向对象OO的思想对所有的文件上传进行防御

DEDECMS本身已经做了这方面的工作了

1. include/common.inc.php
//检查和注册外部提交的变量
function CheckRequest(&$val) 
{
    if (is_array($val)) 
    {
        foreach ($val as $_k=>$_v) 
        {
            CheckRequest($_k);
            CheckRequest($val[$_k]);
        }
    } 
    else
    {
        if( strlen($val)>0 && preg_match(‘#^(cfg_|GLOBALS)#‘,$val) )
        {
            exit(‘Request var not allow!‘);
        }
    }
}


2. /include/uploadsafe.inc.php
//为了防止用户通过注入的可能性改动了数据库
//这里强制限定的某些文件类型禁止上传
$cfg_not_allowall = "php|pl|cgi|asp|aspx|jsp|php3|shtm|shtml";
...
if(!empty(${$_key.‘_name‘}) && (eregi("\.(".$cfg_not_allowall.")$",${$_key.‘_name‘}) || !ereg("\.",${$_key.‘_name‘})) )
{
    if(!defined(‘DEDEADMIN‘))
    {
        exit(‘Upload filetype not allow !‘);
    }
}

0x2: 针对select_soft_post.php文件的修复方案

....
if (preg_match(‘#\.(php|pl|cgi|asp|aspx|jsp|php5|php4|php3|shtm|shtml)$#i‘, trim($filename))) 
{
    ShowMsg("你指定的文件名被系统禁止！",‘javascript:;‘);
    exit();
}
$fullfilename = $cfg_basedir.$activepath.‘/‘.$filename;
$fullfileurl = $activepath.‘/‘.$filename;
....