平时接触到的 python 项目并不多，对 python 的代码审计更是没有接触，偶然朋友发来了一个漏洞 Flask send_file函数导致的绝对路径遍历 ，感觉打开了新世界的大门，于是就以一个初学者的角度，进行复现分析一下。详情也可以根据 Python : Flask Path Traversal Vulnerability 进行分析学习

send_file 的妙用

在以 flask 框架开发的系统中，为了直接实现用户访问某一个 URL 时就可以下载到文件，我们就使用 send_file 来实现

from flask import Flask
from flask import send_file
​
app = Flask(__name__)
​
​
@app.route('/download')
def downloadFile():
    path = "test.txt"
    return send_file(path)
​
​
if __name__ == '__main__':
    app.run()

我们看到 如此运行的效果是直接返回了文件的内容，浏览器并没有识别成一个文件下载下来。

要想让浏览器识别成为文件下载的话，只需要加上 as_attachment=True

from flask import Flask
from flask import send_file
​
app = Flask(__name__)
​
​
@app.route('/download')
def downloadFile():
    path = "test.txt"
    return send_file(path, as_attachment=True)
​
​
if __name__ == '__main__':
    app.run()

当下载的文件名是中文时

from flask import Flask
from flask import send_file
​
app = Flask(__name__)
​
​
@app.route('/download')
def downloadFile():
    path = "测试.txt"
    return send_file(path, as_attachment=True)
​
​
if __name__ == '__main__':
    app.run()
​

Content-Disposition:

Content-Disposition

在常规的 HTTP 应答中，Content-Disposition 响应头指示回复的内容该以何种形式展示，是以内联的形式（即网页或者页面的一部分），还是以附件的形式下载并保存到本地。其可以是inline(默认值,所以可以不指定)或者是attachment,attachment表示附件,浏览器看到这个值一般会弹出一个保持文件的确认框,或者像chrome直接下载。

漏洞分析

漏洞的触发是在 send_file 中，我们跟进看一下

flask.helpers.send_file

继续跟进查看

werkzeug.utils.send_file

我们在本地构造一个简单的语句进行尝试

>>> import os.path
>>> _root_path = "path/to/mySafeStaticDir"
>>> path_or_file = "/../../../../../../../etc/passwd"
>>> os.path.join(_root_path,path_or_file)
'/../../../../../../../etc/passwd'

我们发现 os.path.join 使用不受信任的输入调用时不安全的。当 os.path.join 调用遇到绝对路径时，它会忽略在该点之前遇到的所有参数并开始使用新的绝对路径。当参数可控时，我们控制恶意参数输入绝对路径，os.path.join 会完全忽略静态目录。所以，当 os.path.join 来获取来自 flask.send_file 的不受信任的输入时，可能会目录遍历攻击。

漏洞复现

我们在本地构造简单的代码进行测试，获取从外部传入的参数 filename

from flask import Flask, request
from flask import send_file
​
app = Flask(__name__)
​
​
@app.route('/download')
def downloadFile():
    filename = request.args.get('filename')
    return send_file(filename, as_attachment=True)
​
if __name__ == '__main__':
    app.run()

通过控制 filename 为绝对路径，就实现了目录穿越漏洞

总结反思

这个漏洞非常的有趣，漏洞的修复是可以使用flask.safe_join加入不受信任的路径或用flask.send_file调用替换flask.send_from_directory调用。

这个漏洞虽然很简单，但是在 github 上很多用 python 开发的项目都用了这个函数，如果不加以修复，会造成很大的危害。