查看原文
其他

【缺陷周话】第 33期:错误的资源关闭

奇安信代码卫士 代码卫士 2022-04-06

 聚焦源代码安全,网罗国内外最新资讯!


*声明:《缺陷周话》栏目系列文章由奇安信代码卫士团队原创出品。未经许可,禁止转载。转载请注明“转自奇安信代码卫士 www.codesafe.cn”。


代码审计是使用静态分析发现源代码中安全缺陷的方法,辅助开发或测试人员在软件上线前较为全面地了解安全问题,防患于未然,因此一直以来都是学术界和产业界研究的热点,并已成为安全开发生命周期 SDL 和 DevSecOps 等保障体系的重要技术手段。

奇安信代码卫士团队基于自主研发的国内首款源代码安全检测商用工具,以及十余年漏洞技术研究的积累,推出“缺陷周话”系列栏目。每周针对 CWE、OWASP 等标准中的一类缺陷,结合实例和工具使用进行详细介绍,旨在为广大开发和安全人员提供代码审计的基础性标准化教程。

缺陷周话 • 第33期

错误的资源关闭   

1、错误的资源关闭

程序员手动创建或申请的资源,都需要进行相应的关闭操作,选择的关闭方法应当和创建或申请的方法相对应。例如C语言中常见的打开文件的函数 fopen(),其函数原型为:

FILE *fopen(char *pname,char *mode);

当使用 fopen() 函数打开文件时,应选择其对应的关闭函数 fclose(),避免使用错误的资源关闭函数。此外还有 freopen()_open()CreateFile() 等函数,在使用时也需要注意此类问题。

详细请参见CWE-404: Improper ResourceShutdown or Release。 

2、 错误的资源关闭的危害


使用错误的资源关闭方法,将会导致非预期的程序行为,甚至可能导致程序崩溃。

3、示例代码

示例源于Samate Juliet Test Suitefor C/C++ v1.3 (https://samate.nist.gov/SARD/testsuite.php),源文件名:CWE404_Improper_Resource_Shutdown__fopen_w32_close_01.c。


3.1缺陷代码

在上述示例代码中,第27行使用 fopen() 函数打开文件,在第31行使用 _close() 进行关闭,由于没有使用 fopen() 函数对应的关闭函数 fclose(),因此存在“错误的资源关闭”问题。

使用代码卫士对上述示例代码进行检测,可以检出“错误的资源关闭”缺陷,显示等级为高。如图1所示:

图1:错误的资源关闭检测示例

3.2 修复代码

在上述修复代码中,Samate给出的修复方式为: 在第28行使用 fopen() 函数打开文件,并在第32行使用 fclose() 进行关闭。从而避免了错误的资源关闭问题。

使用代码卫士对修复后的代码进行检测,可以看到已不存在“错误的资源关闭”缺陷。如图2:

图2:修复后检测结果


4、  如何避免错误的资源关闭


要避免错误的资源关闭,需要注意以下几点:

(1)在进行资源关闭时,要根据资源申请或创建使用的方法来合理选择关闭方法,避免错误的资源关闭。

(2)使用源代码静态分析工具,可以有效对该类问题进行检测。






推荐阅读

【缺陷周话】第 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)

国内首个专注于软件开发安全的产品线。




您可能也对以下帖子感兴趣

文章有问题?点此查看未经处理的缓存