C - ポインターへのポインタにreallocを使用すると、セグメンテーションフォールトが発生する

Question

私はコード内のスピードを上げたいのでreallocを使用しようとしています。特定の条件が満たされたとき、より大きなサイズの倍精度浮動小数点型ポインタを再割り当てしたいが、私はセグメンテーション違反を得る。ここにコードがあります。

if (p_bheap->currentSize == p_bheap->arraySize){ 
    p_bheap->arraySize = p_bheap->arraySize*2 + 1; 
    p_bheap->pp_array = realloc(p_bheap->pp_array, p_bheap->arraySize); 
} 

この結果、セグメント化エラーが発生します。しかし、私自身の再割り当て機能を作っても機能します。

if (p_bheap->currentSize == p_bheap->arraySize){ 
    p_bheap->pp_array = bheap_reallocate(p_bheap); 
} 

void** bheap_reallocate(bheap* p_bheap){ 
    p_bheap->arraySize = p_bheap->arraySize*2 + 1; 
    void** pp_newArray = malloc(p_bheap->arraySize*sizeof(void*)); 
    for (int i = 0; i < p_bheap->currentSize; i++){ 
     pp_newArray[i] = p_bheap->pp_array[i]; 
    } 
    free(p_bheap->pp_array); 
    return pp_newArray; 
} 

私が見落としたことのある明確な誤りはありますか？そして私はバイナリヒープをプログラミングしていると思っています。

Answer 1

結果を受け取るために同じメモリスペースを使用することはできません。一時配列を使用するか、別のスペースを割り当ててみてください。

Answer 2

オブジェクトが既に割り当てられている場合は、strとし、そのメモリを別のサイズに調整する場合は、テンポラリ変数を使用して、メモリ損失を防ぐためにreallocが失敗します。イラスト：

char *str = calloc(20, 1); 
if(str) // should always check return of calls to [c][m][re]alloc. 
{ // at this point, str owns 20 bytes of memory. If you use str in a subsequent 
    // call to realloc, and the call fails, then the memory previously allocated in 
    // the original call to calloc() will be lost. (memory leak). 
.... 
char *tmp = {0};//So instead, use a temporary variable to accept memory 
       //and provide a way to recover if allocation fails 

tmp = realloc(str, 40);//newsize (40) in bytes 
if(!tmp) //test reallocation 
{ 
    //reallocation failed. 
    //free previously allocated memory, and decide how to proceed. 
    //In this case, return a NULL, and let the calling function decide. 
    free(str);//if allocation fails, free the previous object and leave 
    return NULL; 
} 
//reallocation succeeded. Proceed normally 
str = tmp;//reallocation succeeded, assign its address to str and proceed normally