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

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

*声明：《缺陷周话》栏目系列文章由奇安信代码卫士团队原创出品。未经许可，禁止转载。转载请注明“转自奇安信代码卫士 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）使用源代码静态分析工具，可以有效对该类问题进行检测。

推荐阅读

*奇安信代码卫士团队原创出品。未经许可，禁止转载。转载请注明“转自奇安信代码卫士 www.codesafe.cn”。

奇安信代码卫士 (codesafe)

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