瓦伦警告:我应该认真对待它吗？

Question

背景:我有一个模仿fgets(character, 2, fp)的小例程，除了它从字符串而不是流中提取一个字符。newBuff被动态分配，字符串作为参数传递，字符声明为char character[2]。

例行事务：

character[0] = newBuff[0];

character[1] = '\0';

strcpy(newBuff, newBuff+1);

当读取每个字符时，strcpy复制丢失的信息。

问题:华润确实警告我这个活动，“strcpy(0x419b818，0x419b819)中的源和目标重叠”。

我该担心这个警告吗？

Answer 1

是的，而且您还应该担心您的函数的性能在病理上很差(对于应该是O(n^2)的任务来说是O(n))。每次读取字符时，都会用一个字符将字符串的全部内容移回来，这是对时间的极大浪费。相反，您应该保留一个指向当前位置的指针，并增加该指针。

当您发现自己需要memmove或类似的(在重叠的缓冲区之间进行复制)的情况下，几乎总是表示存在设计缺陷。通常，这不仅仅是实现中的一个缺陷，而是接口上的一个缺陷。

Answer 2

是的--只有当源和dest不重叠时，strcpy的行为才会被定义。您可以考虑将strlen和memmove组合起来。

Answer 3

是的，你应该担心。C标准指出，当源对象和目标对象重叠时，strcpy的行为是未定义。未定义的行为意味着它有时可以工作，或者它可能失败，或者它看起来成功，但在程序的其他地方却明显失败。