freeBuf
主站

分类

漏洞 工具 极客 Web安全 系统安全 网络安全 无线安全 设备/客户端安全 数据安全 安全管理 企业安全 工控安全

特色

头条 人物志 活动 视频 观点 招聘 报告 资讯 区块链安全 标准与合规 容器安全 公开课

官方公众号企业安全新浪微博

FreeBuf.COM网络安全行业门户,每日发布专业的安全资讯、技术剖析。

FreeBuf+小程序

FreeBuf+小程序

【缺陷周话】第24期:在 scanf 函数中没有对 %s 格式符进行宽度限制
2019-03-04 20:52:43

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

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

1、在 scanf 函数中没有对 %s 格式符进行宽度限制

scanf()是C语言中的一个输入函数,与printf函数一样,都被声明在头文件stdio.h中。函数原型:

int scanf(constchar*restrict format,...);

函数的第一个参数是格式字符串,它指定了输入的格式,并按照格式说明符解析输入对应位置的信息并存储于可变参数列表中对应的指针所指位置。

常见的格式说明符有 %c、%d、%s、%o、%p、%x、%X 等等,其中, %s 的功能是用来接收输入字符串并放入到一个字符数组中,在输入时以非空白字符串开始,以遇到第一个空白字符结束。scanf()函数在匹配非空字符时,使用字符指针指向数组,当没有对最大字段宽度进行限制时,可能会导致缓冲区溢出问题。

scanf() 函数存在相同问题的还有 vscanf()fscanf()vsscanf()

2、 "在 scanf 函数中没有对 %s 格式符进行宽度限制" 的危害

scanf()vscanf()fscanf()vsscanf() 函数接收外部输入,如果输入的长度超出了目标缓冲区长度,就会覆盖其他数据区导致缓冲区溢出。

3、开源代码检测计划某项目示例

开源代码检测计划是一项免费的公益计划。通过使用360代码卫士对开源项目进行源代码检测和审计,找到源代码中存在的安全缺陷并及时与项目开发人员进行沟通和确认,使得开源项目的安全性得到提高。

以下示例代码地址:https://github.com/cisco/thor,存在该缺陷的分支为 y4m,文件名:enc/strings.c。

3.1缺陷代码

25-3-1代码.png

在缺陷代码中,第88行使用 fscanf() 函数从输入流 (stream) 中读入数据,使用%s格式说明符,但没有对宽度进行限制。当输入字符超过数组a的最大长度时,存在安全隐患。

使用360代码卫士对上述示例代码进行检测,可以检出“在scanf 函数中没有对%s格式符进行宽度限制”缺陷,显示等级为高。如图1所示:

25-3-1图.png

图1:“在 scanf 函数中没有对 %s 格式符进行宽度限制”的检测示例

3.2 修复代码

25-3-2代码.png

针对该问题的提出,开发人员对相关代码进行了修复。在88行对 %s 格式符的宽度进行了限制,从而避免了“在 scanf 函数中没有对%s格式符进行宽度限制”问题。

使用360代码卫士对修复后的代码进行检测,可以看到已不存在“在scanf 函数中没有对%s格式符进行宽度限制”缺陷。如图2:


25-3-2图.png

图2:修复后检测结果

4 、如何避免 “在 scanf 函数中未对 %s 格式符进行宽度限制”

(1)在使用 scanf()vscanf()fscanf()vsscanf(),如果使用 %s 格式说明符,请对最大字段宽度进行限制;

(2)使用源代码静态分析工具进行自动化的检测,可以有效的发现源代码中的“在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期 :空指针解引用

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

扫一扫,获取更多干货

qrcode_for_gh_bba053bd7494_1280.jpg


# 代码审计 # 安全缺陷分析
本文为 独立观点,未经允许不得转载,授权请联系FreeBuf客服小蜜蜂,微信:freebee2022
被以下专辑收录,发现更多精彩内容
+ 收入我的专辑
+ 加入我的收藏
相关推荐
  • 0 文章数
  • 0 关注者