c - 我在下面的代码中被中止(核心转储)

Question

您好，在我下面用于测试 realloc() 的代码中，我遇到了错误 Aborted (Core dumped)。

代码的输出如下

$ ./realloc
My Name // <<-- This works
Aborted (core dumped) // <<-- why this error occur?

第一次调用 copyThis() 函数给出了正确的结果并且没有产生错误。同样，如果我第二次调用 copyThis()，它会出错。我不明白为什么会这样。任何人都可以指出问题出在哪里以及我应该做什么调整吗？

代码如下

#include <stdio.h>
#include <string.h>
#include <stdlib.h>

char *copyThis(char *str1, const char *str2, size_t size);

int main(void)
{
    char *ptr1 = "My Name ";
    char *ptr2 = "is Alice";
    char *ptr3;
    char *ptr4;

    ptr3 = copyThis(ptr3, ptr1, strlen(ptr1) + 1); // This works
    printf("%s\n", ptr3);

    ptr4 = copyThis(ptr4, ptr2, strlen(ptr2) +1); // this line make Aborted (core dumped)
    printf("%s\n", ptr4);
}

char *copyThis(char *str1, const char *str2, size_t size)
{
    str1 = (char *) realloc(str1, size);
    strncat(str1, str2, strlen(str2));
    str1[size] = '\0';
    return str1;
}

注意:请指出任何可以帮助我在 c/中很好地掌握字符串操作的好教程网站

Answer 1

错误在于:

char *ptr3;
char *ptr4;

您应该将它们初始化为 NULL:

char *ptr3 = NULL;
char *ptr4 = NULL;

由于 ptr3 和 ptr4 不是 NULL，realloc 假定它们是指向有效内存的指针。来自 man realloc:

Unless ptr is NULL, it must have been returned by an earlier call to malloc(), calloc() or realloc().

由于这些指针指向内存中的随机地址，因此 realloc 在实际使用它们时会感到困惑。

---

作为更一般的说明，在这种情况下，在编译代码时启用额外的诊断非常有帮助。例如，如果您使用 gcc 并使用 -Wall 编译代码，则会发出以下警告:

file.c:19:1: warning: control reaches end of non-void function [-Wreturn-type]
file.c:14:10: warning: ‘ptr3’ is used uninitialized in this function [-Wuninitialized]
file.c:17:10: warning: ‘ptr4’ is used uninitialized in this function [-Wuninitialized]

如何启用这些额外的警告取决于您用于开发的 IDE(如果有的话)；否则，如果您使用命令行，只需将 -Wall 添加到对 gcc 的调用即可。