Question

我在使用std copy函数时收到了警告。

我有一个我声明的byte数组。

byte *tstArray = new byte[length];

然后我有另外一些声明并使用一些十六进制值初始化的字节数组，我想根据一些初始用户输入来使用它们。

我有一系列if语句，基本上用于解析原始输入，根据某些字符串，选择要使用的字节数组，并在此过程中将结果复制到原始tstArray中。

例如:

if(substr1 == "15")
{
   std::cout<<"Using byte array rated 15"<<std::endl;
   std::copy(ratedArray15,ratedArray15+length,tstArray);
}

The warning i get is warning C4996: std::copy : Function call with parameters that may be unsafe - this call relies on the caller to check that the passed values are correct.

一个可能的解决方案是使用-D_SCL_SECURE_NO_WARNINGS来禁用此警告，我正在研究这个。

但是，我不确定这是否意味着我的代码真的不安全，我实际上需要进行一些检查？

Answer 1

C4996表示你正在使用标记为__declspec (deprecated)的函数。可能使用D_SCL_SECURE_NO_WARNINGS将只是#ifdef消除了弃用。你可以去读头文件来确定。

但问题是为什么已被弃用？MSDN 在 std::copy() 页面上似乎没有任何说明，但我可能查看了错误的页面。通常，在 XPSP2 的巨大安全推动期间，所有“不安全字符串操作函数”都会被弃用。由于您没有将目标缓冲区的长度传递给 std::copy，如果尝试向其中写入过多数据，它将愉快地写入缓冲区的末尾。

如果要确定您的使用是否不安全，我们需要回顾您的整个代码。通常，当以这种方式弃用函数时，他们会推荐更安全的版本。您可以以其他方式复制字符串。这篇文章似乎深入探讨了此问题。他们似乎在暗示您应该使用std :: checked_array_iterator，而不是常规的OutputIterator。

类似于：

stdext::checked_array_iterator<char *> chkd_test_array(tstArray, length);
std::copy(ratedArray15, ratedArray15+length, chkd_test_array);

如果我理解你的代码正确的话。

Answer 2

基本上，这个警告告诉你的是，你必须非常确定 tstArray 指向的数组足够大，可以容纳"length"个元素，因为 std::copy 没有检查这一点。

Answer 3

嗯，我认为微软单方面弃用stdlib也包括将传递给。（实际上，他们搞乱了整个函数范围。）

我认为它的部分内容具有一定的价值（fopen() 接触了全局 ERRNO，因此不支持多线程），但其他决策似乎不是很合理（我想他们对整个事情的影响过于大。应该有不同的等级，比如不支持多线程、无法检查等）。

我建议阅读有关每个函数的MS-doc，如果您想了解每种情况的问题，尽管每个函数为什么有警告已经相当好地记录下来了，但通常每种情况的原因都不同。

Answer 4

至少目前看起来，VC++ 2010 RC在默认警告级别下未发出该警告。

友情链接