解放されたポインターが割り当てられていたにもかかわらず割り当てられていませんでした

Question

オブジェクト0x7ffbaf002000でエラーが発生しました：解放されたポインターが割り当てられませんでした。しかし、私はメモリアドレスをプリントアウトしています。実際には0x7ffbaf002000にallocFlights(Flight**, int)のflight[0] = (Flight*) malloc(sizeof(Flight) * 60)の中の関数allocFlights(Flight**, int)に割り当てられています。だから私はこの問題を持っている理由私は理解していない私はそれがありますかどうかを確認するために機能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

最初のフライトを2回割り当て解除します。 2回目に割り当てを解除すると、割り当てられていないにも割り当てが解除されているため、割り当てられていないことが示されます。 deAllocAllFlights()のフライトをすべて割り当て解除したので、最後にfree(flight);に電話する必要はありません。コメントにDavid Schwartzが述べたように、flight[0]は*flightと同じ（または彼がそれを*(flight + 0)と置くので）ためです。

Answer 2

どこにも1つの星がありません。

コードは、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は、実際に配列されているように、変数と他のすべてのパラメータの方がはるかに優れています。それはコードをより明確にし、潜在的にこのような間違いを避ける良いチャンスを与えます。