被释放的指针未分配，即使它之前已分配

Question

我有错误提示 对象 0x7ffbaf002000 错误：未分配指针被释放。但是我已经打印出内存地址，它确实是在 0x7ffbaf002000 之前在循环内的函数allocFlights(Flight**, int)flight[0] = (Flight*) malloc(sizeof(Flight) * 60) 中分配的。所以我在函数deAllocFlights(Flight**, int) 中打印出std::cout << flight[0] << std::endl 的内存地址，看看它是否存在，它是否存在于循环内的0x7ffbaf002000

我不明白我为什么会遇到这个问题。我还是 C++ 新手。

这是飞行结构：

typedef struct {
    int flightNum;
    char origin[20];
    char destination[20];
    Plane *plane;
}Flight;

void getAllFlights(Flight **flight) {
    FILE *file = fopen("reservation.txt", "r");
    int i = 0, totalFlights;

    if(file == NULL)
    {
        perror("Error in opening file");
    }

    fscanf(file, "%d\n", &totalFlights);
    *flight = (Flight*) malloc(sizeof(Flight*) * totalFlights);


    allocFlights(flight, totalFlights);  // Allocate here
    .
    .
    .
    deAllocFlights(flight, totalFlights);  // Error: Deallocate here
    fclose(file);
}

函数 allocFlights

void allocFlights(Flight **flight, int totalFlights) {
    for (int i = 0; i < totalFlights; i++) {
        flight[i] = (Flight*) malloc(sizeof(Flight) * 60);
        std::cout << flight[i] << " " << i << std::endl;  // Print out memory address
    }
}

函数deallocFlights

void deAllocFlights(Flight** flight, int totalFlights) {
    for (int i = 0; i < totalFlights; i++) {
        std::cout << flight[i] << " " << i << std::endl; // Print out memory address
        free (flight[i]);
    }
}

主要：

int main() {
    Flight *flight;
    getAllFlights(&flight);
    free(flight);
    return 0;
}

Answer 1

您要取消分配您的第一个航班两次。所以第二次释放它时，系统会告诉你它还没有被分配，因为虽然它被分配了，但它也被释放了。您不需要在最后致电free(flight);，因为您已经取消了deAllocAllFlights() 中的所有航班。正如 David Schwartz 在 cmets 中提到的那样，这是因为 flight[0] 与 *flight 相同（或者正如他所说的 *(flight + 0)）。

Answer 2

到处都缺一颗星。

代码将原始变量用作指向Flight 的指针数组（或指向Flight 的指针）。因此必须用双星来定义：

int main() {
    Flight **flight;
    getAllFlights(&flight);
    free(flight);
    return 0;
}

每个功能都一样：

void getAllFlights(Flight ***flight) {
    ...
    *flight = (Flight**) malloc(sizeof(Flight*) * totalFlights);

void allocFlights(Flight ***flight, int totalFlights) {
    for (int i = 0; i < totalFlights; i++) {
        // dereference the pointer first and then access array:
        (*flight)[i] = (Flight*) malloc(sizeof(Flight));

void deAllocFlights(Flight*** flight, int totalFlights) {
    for (int i = 0; i < totalFlights; i++) {
        std::cout << (*flight)[i] << " " << i << std::endl; // Print out memory address
        // dereference the pointer first and then access array
        free ((*flight)[i]);

原始代码直接访问指向main 函数中定义的变量的指针，并将其用作数组，这意味着它会转到索引1 的变量后面的地址，甚至更高索引。

另请注意，flights 是变量和所有其他参数的更好名称，因为它实际上是数组。这将使代码更清晰，并有可能更好地避免此类错误。