free() 导致 SEG 故障 C

Question

int ** ARR;
int LENGTH = 1;
int DEPTH = 1;

void loadt(int ** terr)
{
    terr = (int**)malloc(sizeof(int*) * (LENGTH + 1));
    int i, j;
    for(i = 1; i <= LENGTH; i++)
        terr[i] = (int*)malloc(sizeof(int) * (DEPTH + 1));
    for(i = 1; i <= LENGTH; i++)
        for(j = 1; j <= DEPTH; j++)
            scanf("%d", &terr[i][j]);
}
void freet(int ** terr)
{
    int i;
    for(i = 1; i <= LENGTH; i++){
        free(terr[i]);
    }
    free(terr);
}

int main(int argc, char* argv[])
{
    loadt(ARR);
    freet(ARR);  
    return 0;
}

您好。我可能在这里错过了一些非常基本的东西，但是在我运行程序崩溃之后。"Segmentation fault (core dumped)" 为什么？

Answer 1

因为在 c 中函数的参数总是按值传递，所以你不必要的全局变量不会在函数内部重新分配，因为你将它作为参数传递并因此制作了它的副本，这是实际重新分配的那个。

所以你在调用 free() 一个未初始化的指针。

这样试试

int **loadt(int LENGTH, int DEPTH)
{
    int **terr;
    int i, j;
    terr = malloc(sizeof(int*) * (LENGTH + 1));
    if (terr == NULL)
        return NULL;
    for (i = 1; i <= LENGTH; i++) {
        terr[i] = malloc(sizeof(int) * (DEPTH + 1));
        if (terr[i] == NULL) {
            for (j = i ; j >= 0 ; --j) {
                free(terr[j]);
            }
            free(terr);
            return NULL;
        }
    }
    for (i = 1; i <= LENGTH; i++) {
        for (j = 1; j <= DEPTH; j++) {
            scanf("%d", &terr[i][j]);
        }
    }
    return terr;
}

void freet(int **terr)
{
    int i;
    if (terr == NULL)
        return; // `free()' also accepts NULL pointers
    for (i = 1; i <= LENGTH; i++) {
        free(terr[i]);
    }
    free(terr);
}

int main(int argc, char* argv[])
{

    int **ARR;
    int LENGTH = 1;
    int DEPTH = 1;

    ARR = loadt(LENGTH, DEPTH);
    freet(ARR);  
    return 0;
}

另一个问题是，您在 i = 1 开始循环，这很好，因为您分配了足够的 space，但这不是您应该这样做的方式。相反 for (i = 0 ; i < LENGTH ; ++i) 将是 c 程序员的做法。请注意，您正在浪费数组的第一个元素。

free() 导致 SEG 故障 C

free() causes SEG fault C

c

malloc

free