GitHub - SeriousAlpha/py-security-audit-tool: python security audit tool,用于python源码的代码审计,支持命令注入,sql注入
2019-09-12 08:37:04 Author: github.com(查看原文) 阅读量:227 收藏

根据王垠的python静态分析工具[PySonar](https://github.com/yinwang0/pysonar2)得到静态语法树,这是一个庞大的dict结构,递归去除一些不必要的参数得到稍微简单点的一个语法树,以免影响后续分析。

这是文件test_lookuparg.py得到的解析树,body里面包含两个dict,每个dict都是一个函数,另外还有type字段表明这个节点的类型。 下图是一个函数的基本实现: 首先是”type”:”FunctionDef” 表明这一段代码是函数定义,函数中则会有args,表明函数的参数,lineno是代码所在的行,name是函数名。更详细的接口文档见https://greentreesnakes.readthedocs.org/en/latest/nodes.html 在这里包含了各个结构的定义,分析整个树就可以依照这个来实现。 1.2危险函数+可控参数 危险函数有eval,system,popen等系统函数,同时也有咱们自定义的包含这些危险函数的函数,如果这些函数的参数是可控的,就会认为这行代码是有注入风险的,那么这个函数也是有注入风险的。 可控参数,首先会从函数参数入手,认为函数参数是可控的,分析程序会根据前面的语法树去分析代码结构,发现有将函数参数赋值的操作,并且这个赋值是简单的转换,这些简单的转换包含如下类型:

  • (1) 简单的取属性,如get取值,对request单独处理,只认ä=acp#onPopupPost() ¸ºGET,POST,FILES可控,其他request字段如META,user,session,url等都是不可控的。
  • (2) 字符串拼接,被拼接的字符串中包含可控参数,则认为赋值后的值也是可控的
  • (3) 列表解析式,如果列表解析式是基于某个可控因子进行迭代的,则认为赋值后的列表也是可控的
  • (4) 分片符取值,一般认为分片后的值也是可控的,当然这个也不绝对。
  • (5) 一般的函数处理过程:a,函数是常见的字符串操作函数(str,encode,strip等)或者是简单的未过滤函数;b,处理属性;c,如果经过了未知的函数处理则将赋值后的值从可控列表中去掉。
  • (6) 如果代码中的if中有exists,isdigit等带可控参数的的return语句,则将该参数从可控参数列表中去掉(if not os.path.isdir(parentPath):return None),或者将可控参数定死在某个范围之内的(if type not in ["R", "B"]:return HttpResponse("2"))

使用方法如下: liaoxinxi$ python judge_injection.py -h Usage: judge_injection.py [options]

Options: -h, --help show this help message and exit -d FILE_PATH, --dir=FILE_PATH files to be checked -c, --cmd cmd check -s, --sql sql check -a, --all cmd check and sql check -v, --verbose print all unsafe func

一个judge_injection类,首先是初始化得到一个简化的python语法树,提炼出代码中包含的函数语句,分析每一行语句,在碰到函数的时候会调用look_up_arg函数,该函数就是可以得出函数中的可变变量。如果可变变量在危险函数中出现了,则认为该函数是危险的。

(1)目前只做了两层函数分析,对于django的web代码来说已经足够应付
(2)对类的分析还不彻底
(3)目前是基于单个文件来进行分析的,没有考虑模块导入
(4)该模块还可以扩展为分析任意文件下载,任意文件删除等。


文章来源: https://github.com/SeriousAlpha/py-security-audit-tool
如有侵权请联系:admin#unsafe.sh