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;
}

你好。我可能在这里错过了非常基本的sth，但是在我运行程序后它崩溃了。分段错误(核心转储("为什么？

Answer 1

因为在 c 中，函数的参数

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

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

像这样尝试

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 开始循环，这很好，因为您分配了足够的空间，但这不是您应该这样做的方式。相反for (i = 0 ; i < LENGTH ; ++i) c程序员会如何做到这一点。请注意，您正在浪费数组的第一个元素。