Mallocと複数のポインタを解放する

Question

私は複数のバッファを使用するC++プログラムで作業していますが、プログラムを実行するには数分のエラーが発生します。私はそれについて正しい方法を確信しています。私は私に障害を与えていると思うのコードは次のような現在：

int *myBuf1, *myBuf2; 
myBuf1 = (int*)malloc(sizeof(int) * maxPacketSize); 
myBuf2 = myBuf1; 
// do work 

if(myBuf1) { 
    free(myBuf1); 
} 
myBuf1 = NULL; 
myBuf2 = NULL; 

私の最初の質問は次のとおりです。代入「myBuf2 = myBuf1」で、はsizeof（int型）で、メモリの異なるブロックを割り当てるmyBuf2ん* maxPacketSize、またはそれはmyBuf1によって割り当てられた同じメモリブロックへのポインタですか？

次に、myBuf2とmyBuf1を解放してから、両方をNULLに設定する必要がありますか？私はmyBuf1だけを解放しています。または、それを行うのが最善でしょうか：

myBuf1 = (int*)malloc(sizeof(int) * maxPacketSize); 
myBuf2 = (int*)malloc(sizeof(int) * maxPacketSize); 

無料ですか？

Answer 1

Q：割り当てが 'myBuf2 = myBuf1'の場合、myBuf2は 異なるメモリブロックを割り当てますか？

A：いいえ

Q：第二に、私はmyBuf2解放されなければならないだけでなく、myBuf1？

A：いいえ1つのmalloc（）、1つのfree（）。

Q: [Should I] then set both to NULL? 

回答：すべてのポインタをNULLに設定することを強く推奨します。

Answer 2

まず、2つのポインタを宣言していません。

int *myBuf1, myBuf2; 

これはmyBuf1int（ないポインタ）としてint*ようmyBuf2を宣言する。

int *myBuf1, *mBuf2; 

第二には、いや、メモリをコピーしません別のポインタを割り当て、それだけで同じものを指す2つのポインタを作成します：あなたが記述する必要があります。

あなたはfreeが必要です。また、後でNULLに設定する必要はありませんが、いくつかのトリッキーなバグを回避するのに役立つこともあります。

Answer 3

myBuf1からmyBuf2を割り当てると、myBuf2は、単にmyBuf1と同じ場所を指します。

myBuf1 = (int*)malloc(sizeof(int) * maxPacketSize); 
myBuf2 = (int*)malloc(sizeof(int) * maxPacketSize); 

をしてからmyBuf1 = myBuf2;はただ理由もなくmyBuf2にメモリを割り当てることになります。そのため、これを行います。

結論：いずれか1つを解放するだけで十分です。

あなたのmyBuf2はちょうどintで、int*ではありません。おそらくsegfaultの原因になっています。

int *myBuf1, *myBuf2; 

へ

Answer 4

変更

int *myBuf1, myBuf2; 

あなたはメモリの別々のブロックに持たせたい場合は、2つの別々の割り当てが必要になります。

myBuf1 = (int*)malloc(sizeof(int) * maxPacketSize); 
myBuf2 = (int*)malloc(sizeof(int) * maxPacketSize); 

はcallocを使用することを検討してください。行の後

Answer 5

：

myBuf1 = (int*)malloc(sizeof(int) * maxPacketSize); 

あなたなければならないのmalloc（）が失敗したことを示す、NULLポインタを返しませんでしたことをチェック。例えば：

myBuf1 = (int*)malloc(sizeof(int) * maxPacketSize); 
if (myBuf1 == NULL) { 
    printf ("malloc () failed!\n"); 
    exit(1); 
} 

Answer 6

Qは：[私はべき]その後、両方がNULLに設定しますか？

回答：はい、すべてのポインタをNULLに設定することを強く推奨します。

ブロック内にいる場合は、nullポインタを戻すように設定する必要はありません。コンパイラにとっては何もしていませんが、読者にとってはもっと混乱します。

C++では、ブロックを使用して識別子（特にポインタ）の範囲を制限することをお勧めします。