Question

我一直在使用valgrind，但由于某种原因，我在使用带有两个大小相同的字符串的简单字符串副本时，不断收到内存错误。

守则有效：

node->entry = (char*)malloc(strlen(string)*sizeof(char));
strcpy(node->entry, string);

字符串为：char * string =“Hello There”。错误是：写入大小为2的无效

==2035==    at 0xD494: memmove$VARIANT$sse42 (mc_replace_strmem.c:987)
==2035==    by 0x100001793: __inline_strcpy_chk (_string.h:94)
==2035==    by 0x100001699: createList (main.c:10)
==2035==    by 0x100001BE6: main (main.c:132)
==2035==  Address 0x10000c0fa is 10 bytes inside a block of size 11 alloc'd
==2035==    at 0xB823: malloc (vg_replace_malloc.c:266)
==2035==    by 0x100001635: createList (main.c:9)
==2035==    by 0x100001BE6: main (main.c:132)

感谢您的帮助！

Answer 1

malloc(strlen(string)*sizeof(char));

您没有考虑结尾'\0'。所以应该是(strlen(string) + 1)。

旁注：

type *x;
x = malloc(size * sizeof(*x))

比

更易于维护

type *x;
x = (type *)malloc(size * sizeof(type));

因为你不会重复3次。

Answer 2

您的代码已损坏：

您需要为字符串终止字符分配空间，如果不这样做会使strcpy()写入未分配的内存并导致缓冲区溢出错误。
Don't cast the return value of malloc(), in C.
在写入内存之前，您需要检查分配是否成功。

此外，请注意sizeof (char)始终为1，因此无需涉及它。只需使用：

node->entry = malloc(strlen(string) + 1);

或者，检查您是否有strdup()，因为它结合了以上所有内容。

在一般情况下，正如@Shahbaz指出的那样，最好在取消引用左手指针的结果上使用sizeof运算符，以避免重复类型名称：

type *x;
x = malloc(amount * sizeof *x);

另外，请注意sizeof不是函数，仅当参数是类型名称时才需要括号，并且正如我刚才所说的那样，应避免使用括号。

使用valgrind，strcpy的内存错误

2 个答案: