一次安全测试引发的对Django框架文件上传安全机制的初步分析

2018-09-11 239053人围观 ,发现 8 个不明物体 WEB安全

*本文原创作者:ForrestX386,本文属FreeBuf原创奖励计划,未经许可禁止转载

0×00. 起因

我司的堡垒机是基于jumpserver 0.3版本进行二次开发,进行了大量的重构和新功能的添加,基本满足了公司安全运维的需求。在对文件上传接口进行安全审计的时候发现,其对上传文件名没有过滤处理,然后直接写入磁盘(部分代码如下)

bastion_upload_file.png

隐隐觉得可以搞搞任意文件写入漏洞(jumpserver web 控制面板都是以root权限运行,如果可以任意文件写入,危害呵呵,你懂得)。启动burpsuite ,拦截请求,修改文件名(含有目录穿越字符),但是结果没成功,调试发现upload_file.name 已经是../../等目录穿越字符过滤后的结果,有点奇怪,莫非是框架自动帮我过滤掉了,好奇心驱使我必须弄明白其中的原理,于是有了本文。


0×01. 分析过程

切入点就是request.FILES 对象的由来,整个流程涉及到5个模块,如下:

django.core.handlers.wsgi

django.http.request

django.http.multipartparser

django.core.files.uploadhandler

django.core.files.uploadedfile

request.FILES 是一个类似于dict的对象,上传文件输入框name属性的值为键名,键值指向处理后的文件对象(框架会调用指定的文件处理器处理),这个文件对象就是django.core.files.uploadedfile 模块中UploadedFile类的实例。详细分析如下:

访问request.FILES 就是访问 django.core.handlers.wsgi 模块中WSGIRequest(继承至django.http.request模块的HttpRequest类)类的FILES属性

WSGIRqest_FILES.png

也即访问WSGIRequest._get_files,这个方法会先判断是否已经解析过上传的文件(也即判断是否有_files属性,其实FILES 就是_files,MultiValueDict 类的实例),跟进_load_post_and_files 方法(这是继承至其父类django.http.request模块的HttpRequest类中的方法),如下:

HttpRequest_load_post_and_files.png

跟进parse_file_upload方法(这里的data为WSGIRequest 的实例),如下:

WSGIRequest_parse_file_upload.png

这里要先说下upload_handlers 成员,如下:

upload_handlers.png

初始化upload_handlers的时候会调用django.core.files.uploadhandler模块的load_handler加载系统默认的文件处理器,如下:

uploadhandler_load_handler.png

settings.FILE_UPLOAD_HANDLERS

默认就是指的红框中的两个文件处理器,大于2.5M的就用TemporaryFileUploadHandler 处理器,否则用MemoryFileUploadHandler。

初始化文件上传处理器之后,就开始调用django.http.multipartparser 模块的MultiPartParser 类的parse 方法对上传文件进行解析处理,在解析处理过程中,会调用 handle_file_complete 对上传后的文件进行再次处理(处理完成后就返回一个django.core.files.uploadedfile.UploadedFile类的实例, 这个实例对象会被添加到_files 对象中,然后由parse 方法返回此对象, 这个过程就包含文件名被过滤掉的过程),如下:

upload_file_handler_complete.png

图中的old_filed_name 即为上一个解析完毕的文件,跟进handle_file_complete ,如下:

handle_file_complete.png

跟进文件处理器的file_complete方法, 这个方法返回的就是处理后的文件对象,也就是0×00 图中upload_file 变量指向的文件对象,这里我们以MemoryFileUploadHandler 文件处理器为例进行说明:

memory_file_handler.png也就是说0×00 中的upload_file 也即InMemoryUploadedFile 类的实例,所以调用upload_file.name 即调用InMemoryUploadedFile 的name属性,如下:

InMemory_set_name.png调用InMemoryUploadedFile 的name属性,即调用_get_name方法,在InMemoryUploadedFile  实例话的过程中有name的赋值操作(在其父类__init__方法中)如下:

_set_name.png

赋值操作就会触发_set_name方法的执行:

vul_defend.png

在_set_name中就会对上传的文件进行过滤处理,os.path.basename(name)防止了目录穿越漏洞,所以我们在0×00 图中使用uploadfile.name获取到的是经过os.path.basename 处理后的文件名,当然没法任意文件写入了


0×02. 总结

梳理完成之后,终于对Django 文件上传中的安全机制有了一些了解,解决了我的困惑,像Django 这种现代的web框架对传统的安全漏洞(比如XSS,CSRF、文件上传等)都做了比较好的处理,在开发中,这些都是值得我们借鉴和学习的地方。

*本文原创作者:ForrestX386,本文属FreeBuf原创奖励计划,未经许可禁止转载

相关推荐

这些评论亮了

  • test 回复
    我来说一下作者的心态:
    看到第一张图的时候,作者:“卧槽,牛逼了,这有个0day!”
    看到最后一张图的时候,作者:“卧槽,原来是我想多了!”
    )10( 亮了
发表评论

已有 8 条评论

取消
Loading...

特别推荐

推荐关注

活动预告

填写个人信息

姓名
电话
邮箱
公司
行业
职位
css.php