代码之家 › 专栏 › 技术社区 › Federico

应特别注意确保安全使用堆栈上分配的字符数组

code-analysis security c

Federico · 技术社区 · 7 年前

我用SonarQube和 RATS

在Ubuntu Shell下,我执行

老鼠——安静——xml-w 1>/rats_报告。xml

获取将导入SonarQube的报告。

我遇到了如下错误:

应特别注意确保安全使用堆栈上分配的字符数组。它们是缓冲区溢出攻击的主要目标。

static char* GetQueryStringForValue( const char* valueLabel )
{
  static char queryString[QUERY_LEN + 1];

  memcpy( queryString, '\0', sizeof(queryString) );
  snprintf( queryString, sizeof(queryString), "{'%s'", valueLabel );

  return queryString;
}

我知道问题与分配到堆栈中的缓冲区有关。

我的问题是:防止缓冲区溢出攻击的最佳做法是什么?

谢谢你的帮助!

费德里科

1 回复 | 直到 7 年前

user2371524 user2371524 7 年前

这是误报,这里“在堆栈上”没有分配任何内容。和 static 存储类说明符, queryString 静态存储持续时间 这意味着它存在于程序的整个执行期间。C的任何实现都不会放置这样的对象 在堆栈上 .

memcpy( queryString, '\0', sizeof(queryString) );

这是试图取消引用 NULL 指针(指针 NUL 字符常量隐式转换为 无效的

memset(queryString, 0, sizeof queryString);

这就是说,如果你仍然收到这个警告,就把它当成是什么:一个警告。它警告你要格外小心。固定使用代码 memset() 查询字符串 在这里

线程安全 静止的 变量最好让调用者为 .

推荐文章

Community wiki · C中有哪些耗时的操作?

1 年前

Tintenfisch · 传递参数:array与C和C中的*&array和&array[0]之间的差异++

1 年前

daryldxn · Windows筛选平台计算通过TCP连接发送的字节和接收的字节

1 年前

Mike Balts · 它们将被打印多少次,为什么?我知道:“阿尔法”一次,“贝塔”两次,“欧米茄”两次但我不知道为什么

1 年前

Mohammed Eid · 数据类型“char”是否可以被视为数据类型“int”?

1 年前

Community wiki · 将所有处理器电源都投入到任务中

1 年前

Community wiki · 在C&数据结构中实现不同数据结构的聪明方法,应该更频繁地使用

1 年前

Community wiki · C++为C添加了什么?[已关闭]

1 年前

Abhinav Kumar · 如何将#define的数据类型设置为长双精度?

1 年前

Community wiki · 打印1到1000,不带循环或条件

1 年前