realloc() 之后的分段错误。无法将分配的内存分配给指针

Question

我正在尝试使用 realloc() 分配一些内存。到目前为止，这有效。但是，如果我想将分配的内存分配给结构变量中的指针，则会出现分段错误：

// in header
typedef struct {
    int a;
    char test[20];
} MyContent;

typedef struct {
    MyContent* values; 
    // simmilar to: MyContent values[]
    // ... some other stuff
} MyData;

// in source
void myFunction(MyData* dataPtr) {
    dataPtr->values = NULL;
    MyData* tempPtr = NULL;

    for (int i = 1; i < 10; i++) {
        tempPtr = (MyContent*) realloc(dataPtr->values, i * sizeof(MyContent));
        if (tempPtr == NULL) {
            free(dataPtr->values);
            break;
        }
        dataPtr->values = tempPtr;  // Here I get the segmentation fault
        dataPtr->values[(i-1)].a = 42;
        // ...
    }
}

我不知道这里出了什么问题。有什么建议么？感谢您的帮助。

Answer 1

好像你编辑了你的代码。编辑后的代码运行良好。

#include<stdio.h>
#include<malloc.h>
#include<string.h>
// in header
typedef struct {
    int a;
    char test[20];
} MyContent;

typedef struct {
    MyContent* values; 
    // simmilar to: MyContent values[]
    // ... some other stuff
} MyData;

// in source
void myFunction(MyData* dataPtr) {
    dataPtr->values = NULL;
    MyData* tempPtr;

    for (int i = 1; i < 10; i++) {
        tempPtr = (MyData*) realloc(dataPtr->values, i * sizeof(MyContent));
        if (tempPtr == NULL) {
            if(dataPtr->values)
                free(dataPtr->values);
            printf("realloc() failed\n");
            return ;
        }
        dataPtr->values = (MyContent*)tempPtr;  // Here I get the segmentation fault
        dataPtr->values[(i-1)].a = 42+i;
        strcpy(dataPtr->values[(i-1)].test,"name");
    }
}

void PrintData(MyData* dataPtr) {
    for (int i = 1; i < 10; i++)
        printf("We have %s at %d\n",dataPtr->values[(i-1)].test,dataPtr->values[(i-1)].a);
}

main() {
    MyData Sample;
    myFunction(&Sample);
    PrintData(&Sample);
}

Answer 2

乍一看，我没有看到可能导致崩溃的问题 - 基于一的寻址有点奇怪，但并非不正确。代码中可能存在您未显示的问题，导致堆或堆栈损坏，而 realloc 调用会使情况变得更糟。或者，如果您正在使用优化进行编译，您的调试器可能会对崩溃实际发生的位置感到困惑。您还混淆了MyData 和MyContent，但我认为这只是因为您在编辑代码时出错了。

还要注意，如果realloc 失败，您将在您指定的行 之后崩溃，因为您将写入一个空指针。如果tempPtr is NULL，您需要中止，而不仅仅是释放旧指针。但是，这再次导致与您指示的线路不同的故障。

我建议在 valgrind 下运行您的程序，看看它报告错误的位置 - 第一个发生的此类错误很可能是罪魁祸首。

Answer 3

您检查新分配的内存的条件是错误的。应该是：

if (tempPtr == NULL) {
  // handle error condition or continue with original 'dataPtr->values'
}
else {
  dataPtr->values = tempPtr;
}

请记住，realloc() 不一定将一个块转移到另一个块。有时它可能会在同一个指针区域分配内存。