专栏首页python3python 安全编码&代码审计

python 安全编码&代码审计

1 前言

现在一般的web开发框架安全已经做的挺好的了,比如大家常用的django,但是一些不规范的开发方式还是会导致一些常用的安全问题,下面就针对这些常用问题做一些总结。代码审计准备部分见《php代码审计》,这篇文档主要讲述各种常用错误场景,基本上都是咱们自己的开发人员犯的错误,敏感信息已经去除。

2 XSS

未对输入和输出做过滤,场景:

  1. def xss_test(request):
  2. name = request.GET['name']
  3. return HttpResponse('hello %s' %(name))

在代码中一搜,发现有大量地方使用,比较正确的使用方式如下:

  1. def xss_test(request):
  2. name = request.GET['name']
  3. #return HttpResponse('hello %s' %(name))
  4. return render_to_response('hello.html', {'name':name})

更好的就是对输入做限制,比如说一个正则范围,输出使用正确的api或者做好过滤。

3 CSRF

对系统中一些重要的操作要做CSRF防护,比如登录,关机,扫描等。django 提供CSRF中间件django.middleware.csrf.CsrfViewMiddleware,写入到settings.py的中间件即可。另外再在函数前加上@csrf_exempt修饰器。

4 命令注入

审计代码过程中发现了一些编写代码的不好的习惯,体现最严重的就是在命令注入方面,本来python自身的一些函数库就能完成的功能,偏偏要调用os.system来通过shell 命令执行来完成,老实说最烦这种写代码的啦。下面举个简单的例子:

  1. def myserve(request, filename, dirname):
  2.   re = serve(request=request,path=filename,document_root=dirname,show_indexes=True)
  3.   filestr='authExport.dat'
  4.   re['Content-Disposition'] = 'p_w_upload; filename="' + urlquote(filestr) +'"'fullname=os.path.join(dirname,filename)
  5.   os.system('sudo rm -f %s'%fullname)
  6. return re

很显然这段代码是存在问题的,因为fullname是用户可控的。正确的做法是不使用os.system接口,改成python自有的库函数,这样就能避免命令注入。python的三种删除文件方式: (1)shutil.rmtree 删除一个文件夹及所有文件 (2)os.rmdir 删除一个空目录 (3)os.remove,unlink 删除一个文件

使用了上述接口之后还得注意不能穿越目录,不然整个系统都有可能被删除了。常见的存在命令执行风险的函数如下:

  1. os.system,os.popen,os.spaw*,os.exec*,os.open,os.popen*,commands.call,commands.getoutput,Popen*

推荐使用subprocess模块,同时确保shell=True未设置,否则也是存在注入风险的。

5 sql注入

如果是使用django的api去操作数据库就应该不会有sql注入了,但是因为一些其他原因使用了拼接sql,就会有sql注入风险。下面贴一个有注入风险的例子:

  1. def getUsers(user_id=None):
  2. conn = psycopg2.connect("dbname='××' user='××' host='' password=''")
  3. cur = conn.cursor(cursor_factory=psycopg2.extras.DictCursor)
  4. if user_id==None:
  5. str = 'select distinct * from auth_user'
  6. else:
  7. str='select distinct * from auth_user where id=%d'%user_id
  8. res = cur.execute(str)
  9. res = cur.fetchall()
  10. conn.close()
  11. return res

像这种sql拼接就有sql注入问题,正常情况下应该使用django的数据库api,如果实在有这方面的需求,可以按照如下方式写:

  1. def user_contacts(request):
  2. user = request.GET['username']
  3. sql = "SELECT * FROM user_contacts WHERE username = %s"
  4. cursor = connection.cursor()
  5. cursor.execute(sql, [user])
  6. # do something with the results
  7. results = cursor.fetchone() #or results = cursor.fetchall()
  8. cursor.close()

直接拼接的是万万不可的,如果采用ModelInstance.objects.raw(sql,[]),或者connection.objects.execute(sql,[]) ,通过列表传进去的参数是没有注入风险的,因为django会有处理。

6 代码执行

一般是由于eval和pickle.loads的滥用造成的,特别是eval,大家都没有意识到这方面的问题。下面举个代码中的例子:

  1. @login_required
  2. @permission_required("accounts.newTask_assess")
  3. def targetLogin(request):
  4. req = simplejson.loads(request.POST['loginarray'])
  5. req=unicode(req).encode("utf-8")
  6. loginarray=eval(req)
  7. ip=_e(request,'ipList')
  8. #targets=base64.b64decode(targets)
  9. (iplist1,iplist2)=getIPTwoList(ip)
  10. iplist1=list(set(iplist1))
  11. iplist2=list(set(iplist2))
  12. loginlist=[]
  13. delobjs=[]
  14. holdobjs=[]

这一段代码就是就是因为eval的参数不可控,导致任意代码执行,正确的做法就是literal.eval接口。再取个pickle.loads的例子:

  1. >>> import cPickle
  2. >>> cPickle.loads("cos\nsystem\n(S'uname -a'\ntR.")
  3. Linux RCM-RSAS-V6-Dev 3.9.0-aurora #4 SMP PREEMPT Fri Jun 7 14:50:52 CST 2013 i686 Intel(R) Core(TM) i7-2600 CPU @ 3.40GHz GenuineIntel GNU/Linux
  4. 0

7 文件操作

文件操作主要包含任意文件下载,删除,写入,覆盖等,如果能达到写入的目的时基本上就能写一个webshell了。下面举个任意文件下载的例子:

  1. @login_required
  2. @permission_required("accounts.newTask_assess")
  3. def exportLoginCheck(request,filename):
  4. if re.match(r“*.lic”,filename):
  5. fullname = filename
  6. else:
  7. fullname = "/tmp/test.lic"
  8. print fullname
  9. return HttpResponse(fullname)

这段代码就存在着任意.lic文件下载的问题,没有做好限制目录穿越,同理

8 文件上传

8.1 任意文件上传

这里主要是未限制文件大小,可能导致ddos,未限制文件后缀,导致任意文件上传,未给文件重命名,可能导致目录穿越,文件覆盖等问题。

8.2 xml,excel等上传

在我们的产品中经常用到xml来保存一些配置文件,同时也支持xml文件的导出导入,这样在libxml2.9以下就可能导致xxe漏洞。就拿lxml来说吧:

  1. root@kali:~/python# cat test.xml
  2. <?xml version="1.0" encoding="utf-8"?>
  3. <!DOCTYPE xdsec [ <!ENTITY xxe SYSTEM "file:///etc/passwd" >
  4. ]>
  5. <root>
  6. <node id="11" name="bb" net="192.168.0.2-192.168.0.37" ltd="" gid="" />test&xxe;</root>
  7. >>> from lxml import etree
  8. >>> tree1 = etree.parse('test.xml')
  9. >>> print etree.tostring(tree1.getroot())
  10. <root>
  11. <node id="11" name="bb" net="192.168.0.2-192.168.0.37" ltd="" gid=""/>testroot:x:0:0:root:/root:/bin/bash
  12. daemon:x:1:1:daemon:/usr/sbin:/bin/sh
  13. bin:x:2:2:bin:/bin:/bin/sh
  14. sys:x:3:3:sys:/dev:/bin/sh
  15. sync:x:4:65534:sync:/bin:/bin/sync
  16. games:x:5:60:games:/usr/games:/bin/sh
  17. man:x:6:12:man:/var/cache/man:/bin/sh

这是因为在lxml中默认采用的XMLParser导致的:

  1. class XMLParser(_FeedParser)
  2. | XMLParser(self, encoding=None, attribute_defaults=False, dtd_validation=False, load_dtd=False, no_network=True, ns_clean=False, recover=False, XMLSchema schema=None, remove_blank_text=False, resolve_entities=True, remove_comments=False, remove_pis=False, strip_cdata=True, target=None, compact=True)

关注其中两个关键参数,其中resolve_entities=True,no_network=True,其中resolve_entities=True会导致解析实体,no_network会为True就导致了该利用条件比较有效,会导致一些***f问题,不能将数据带出。在python中xml.dom.minidom,xml.etree.ElementTree不受影响

9 不安全的封装

9.1 eval 封装不彻底

仅仅是将__builtings__置为空,如下方式即可绕过,可参见bug84179

  1. >>> s2="""
  2. ... [x for x in ().__class__.__bases__[0].__subclasses__()
  3. ... if x.__name__ == "zipimporter"][0](
  4. ... "/home/liaoxinxi/eval_test/configobj-4.4.0-py2.5.egg").load_module(
  5. ... "configobj").os.system("uname")
  6. ... """
  7. >>> eval(s2,{'__builtins__':{}})
  8. Linux
  9. 0

9.2 执行命令接口封装不彻底

在底层封装函数没有过滤shell元字符,仅仅是限定一些命令,但是其参数未做控制,可参见bug86011

10 总结

一切输入都是不可靠的,做好严格过滤。

本文参与腾讯云自媒体分享计划,欢迎正在阅读的你也加入,一起分享。

我来说两句

0 条评论
登录 后参与评论

相关文章

  • python写的系统常用命令(一)

          python写的系统常用命令,linux和windows通用,用的时候直接from util import *导入即可使用,很方便,本来是一个脚本,...

    py3study
  • Flask send_file requ

    py3study
  • Python之函数

    Python函数分为内置函数和自定义函数。内置函数可以理解为一些类的方法的快捷方式,供我们创建对象时来调用,内置函数大概有以下这么多

    py3study
  • CSP

    该评论被提交后,会存储在数据库中,当其他用户打开该页面时,该代码会被自动执行,用户就会被攻击到。

    不作声
  • 基于MIMO的悬臂梁振动响应有限元计算原理及应用

    “本文介绍了梁的有限元动力学分析基本原理,并基于梁有限元模型,运用MIMO(多输入多输出)算法,计算梁在多个输入力下的振动响应。单自由度质量-弹簧-阻尼系统的振...

    用户7573907
  • Java源码阅读绘图规范手册--[捷特版]

    张风捷特烈
  • SSM框架(二)Spring的AOP

    代理模式是 AOP(Aspect Oriented Programming 面向切面编程)的核心实现机制。

    HcodeBlogger
  • Android Service组件(1)

    android service 和其他服务一样,并没有实际运行的界面,它运行在android 后台。一般通过service为应用程序提供服务(比如,从Inter...

    水击三千
  • Qt Style Sheet实践(二):组合框QComboBox的定制

    导读      组合框是一个重要且应用广泛的组件,一般由两个子组件组成:文本下拉单部分和按钮部分。在许多既需要用户选择、又需要用户手动输入的应用场景下,组合框能...

    24K纯开源

扫码关注云+社区

领取腾讯云代金券