【缺陷周话】第53期:不当的循环终止

阅读量    157952 |

分享到: QQ空间 新浪微博 微信 QQ facebook twitter

1、不当的循环终止

C/C++语言中的循环操作包括 for() 循环,while() 循环、do{}while() 循环等,在使用循环操作时,需要设定恰当的循环终止条件,避免造成死循环。

 

2、“不当的循环终止”的危害

不当的循环终止通常会导致死循环的发生,进而导致拒绝服务攻击、程序崩溃等漏洞。

CVE中也有一些与之相关的漏洞信息,从2018年1月至2019年9月,CVE中就有3条相关漏洞信息。漏洞信息如下:

CVE 概述
CVE-2019-14207 Foxit PhantomPDF 是中国福昕(Foxit)公司的一款PDF文档阅读器。Foxit PhantomPDF 8.3.11 之前版本中存在安全漏洞。攻击者可利用该漏洞造成应用程序崩溃(死循环)。
CVE-2018-19826 LibSass是一个开源的使用C语言编写的Sass(CSS扩展语言)解析器。LibSass 3.5.5版本中的 inspect.cpp 文件存在安全漏洞(死循环)
CVE-2018-10316 NetwideAssembler(NASM) 是一个基于 Linux 的汇编器,它能够创建二进制文件并编写引导加载程序。NASM2.14rc0 版本中的asm/nasm.c文件的 ‘assemble_file’ 函数存在整数溢出漏洞。攻击者可借助特制的文件利用该漏洞造成拒绝服务(死循环)。

 

3、示例代码

示例源于 toyota-itc-benchmarks-master (https://github.com/regehr/itc-benchmarks),源文件名:endless_loop.c。

3.1 缺陷代码

上述示例代码中在第39行使用 for 语句进行循环操作,且在 for 语句中省略了循环控制变量,在 for 循环体中也没有对循环控制变量进行更新,从而导致死循环的发生,因此存在“不当的循环终止”问题。

使用代码卫士对上述示例代码进行检测,可以检出“不当的循环终止”缺陷,显示等级为高。如图1所示:

图1:“不当的循环终止”检测示例

3.2 修复代码

上述修复代码中,在第42行使用 i++; 语句对循环控制变量进行更新,当 i 的值等于10时循环退出,从而避免了死循环问题。

使用代码卫士对修复后的代码进行检测,可以看到已不存在“不当的循环终止”缺陷。如图2:

图2:修复后检测结果

 

4、如何避免“不当的循环终止”

(1)在使用循环操作时,设定正确的循环条件;

(2)在循环体中,满足循环条件的情况下也可以通过执行 break、return 等语句终止循环。

分享到: QQ空间 新浪微博 微信 QQ facebook twitter
|推荐阅读
|发表评论
|评论列表
加载更多