聚焦源代码安全,网罗国内外最新资讯!
*声明:《缺陷周话》栏目系列文章由奇安信代码卫士团队原创出品。未经许可,禁止转载。转载请注明“转自奇安信代码卫士 www.codesafe.cn”。
代码审计是使用静态分析发现源代码中安全缺陷的方法,辅助开发或测试人员在软件上线前较为全面地了解安全问题,防患于未然,因此一直以来都是学术界和产业界研究的热点,并已成为安全开发生命周期 SDL 和 DevSecOps 等保障体系的重要技术手段。
奇安信代码卫士团队基于自主研发的国内首款源代码安全检测商用工具,以及十余年漏洞技术研究的积累,推出“缺陷周话”系列栏目。每周针对 CWE、OWASP 等标准中的一类缺陷,结合实例和工具使用进行详细介绍,旨在为广大开发和安全人员提供代码审计的基础性标准化教程。
缺陷周话 • 第43期
不当的函数地址使用
1、不当的函数地址使用
错误的将函数地址当作函数、条件表达式、运算操作对象使用,甚至参与逻辑运算,将导致各种非预期的程序行为发生。比如在如下if语句,其中func()为程序中定义的一个函数:
if (func == NULL)
由于使用 func 而不是 func(),也就是使用的是 func 的地址而不是函数的返回值,而函数的地址不等于 NULL,如果用函数地址与 NULL 作比较时,将使其条件判断恒为false。
详细请参见CWE-480: Use of IncorrectOperator。
2、"不当的函数地址使用”的危害
不当的函数地址使用可能会导致非预期的程序行为发生,如条件永远不会被触发而出现的逻辑错误,又或者由于条件恒为真而导致的无限循环,从而造成资源耗尽、拒绝服务等攻击。
3、示例代码
示例源于 Samate Juliet TestSuite for C/C++ v1.3 (https://samate.nist.gov/SARD/testsuite.php),源文件名:CWE480_Use_of_Incorrect_Operator__basic_01.c。
3.1 缺陷代码
在上述示例代码中,第53行使用 helperBad==NULL 作为 if 语句的判断条件,其中helperBad() 函数的定义在21行,helperBad==NULL 操作导致 if 语句恒为 false,第55行 printLine() 函数永远不会被执行,存在“不当的函数地址使用”问题。
使用代码卫士对上述示例代码进行检测,可以检出“不当的函数地址使用”缺陷,显示等级为中。如图1所示:
图1:“不当的函数地址使用”检测示例
3.2 修复代码
在上述修复代码中,Samate给出的修复方式为:在第52行使用函数返回值替代函数地址进行条件判断,从而避免了“不当的函数地址使用”问题。
使用代码卫士对修复后的代码进行检测,可以看到已不存在“不当的函数地址使用”缺陷。如图2:
图2:修复后检测结果
4、如何避免“不当的函数地址使用”
需要明确操作使用的是函数地址还是函数返回值,避免由于编码错误造成的对函数地址的直接使用。
推荐阅读【缺陷周话】第42期 — Cookie:未经过SSL加密
【缺陷周话】第41期:忽略返回值
【缺陷周话】第40期:JSON 注入
【缺陷周话】第 39期:解引用未初始化的指针
【缺陷周话】第 38期——不安全的反序列化:XStream
【缺陷周话】第 37期:未初始化值用于赋值操作
【缺陷周话】第 36 期:弱验证
【缺陷周话】第 35 期:除数为零
【缺陷周话】第 34 期:重定向
【缺陷周话】第 33期:错误的资源关闭
【缺陷周话】第 32期:弱加密
【缺陷周话】第 31 期:错误的内存释放方法
【缺陷周话】第 30 期:不安全的哈希算法
【缺陷周话】第 29 期:返回栈地址
【缺陷周话】第 28 期:被污染的内存分配
【缺陷周话】第 27 期:不安全的随机数
【缺陷周话】第 26期:被污染的格式化字符串
【缺陷周话】第 25期:硬编码密码
【缺陷周话】第 24期:在scanf 函数中没有对 %s 格式符进行宽度限制
【缺陷周话】第 23期:双重检查锁定
【缺陷周话】第 22期:错误的内存释放对象
【缺陷周话】第 21 期:数据库访问控制
【缺陷周话】第 20 期:无符号整数回绕
【缺陷周话】第19期:LDAP 注入
【缺陷周话】第18 期 XPath 注入
【缺陷周话】第17 期:有符号整数溢出
【缺陷周话】第 16 期 — 资源未释放:流
【缺陷周话】第 15 期 — 资源未释放:文件
【缺陷周话】第 14 期 :HTTP 响应截断
【缺陷周话】第 13期 :二次释放
【缺陷周话】第 12期 :存储型 XSS
【缺陷周话】第 11期 :释放后使用
【缺陷周话】第 10 期 :反射型 XSS
【缺陷周话】第 9 期 :缓冲区下溢
【缺陷周话】第 8 期 :路径遍历
【缺陷周话】第 7 期 :缓冲区上溢
【缺陷周话】第 6 期 :命令注入
【缺陷周话】第5期 :越界访问
【缺陷周话】第4期 :XML 外部实体注入
【缺陷周话】第3期 :内存泄漏
【缺陷周话】第 2 期 :SQL 注入
【缺陷周话】第1期 :空指针解引用
*奇安信代码卫士团队原创出品。未经许可,禁止转载。转载请注明“转自奇安信代码卫士 www.codesafe.cn”。
奇安信代码卫士 (codesafe)
国内首个专注于软件开发安全的产品线。