Warum gibt mir diese Realloc-Instanz SIGABRT?

Question

Hier ist der Code in Frage, es ist eine Funktion, die einen int-Array reallocs und fügt eine Zahl am Ende davon

int *add_to_array(int *arr, unsigned int num, int newval) 
{ 
    if(arr != NULL){ 

     int *newarr = realloc(arr, sizeof(int) * (num+1)); 
      if(!newarr){ 
       free(arr); 

      }else{ 
       free(newarr); 
      } 
      arr[num] = newval; 
      return arr; 
    } 
    else{ 

    return NULL; 

    } 

} 

Problem ist, wenn ich diese Funktion einmal anrufen, es funktioniert gut, aber wenn ich nennen es ein zweites Mal, der Debugger gibt mir eine SIGABRT an der Linie

int *newarr = realloc(arr, sizeof(int) * (num+1)); 

Hier ist, wie ich die Funktion

Aufruf bin

array = add_to_array(array, 5, 10); 
array = add_to_array(array, 6, 100); 

EDIT: habe ich es geschafft zu s Ølve dies mit diesem neuen Code

int *add_to_array(int *arr, unsigned int num, int newval) 
{ 
if(arr != NULL){ 

int *newarr =realloc(arr, sizeof(int) * (num+1)); 
if(!newarr){ 
    return NULL; 
}else{ 
newarr[num] = newval; 
return newarr; 
} 
} 
else{ 

    return NULL; 

} 

} 

Noch eine Frage, für den Fall, wobei realloc den gegebenen Zeiger arr realloc ausfällt, tut arr in diesem Fall null wird oder ist es immer noch gültig?

Answer 1

Wenn realloc versagt Sie wahrscheinlich das alte Array speichern wollen und nichts zu tun, so:

 if(!newarr){ 
      free(arr); 

werden sollten:

 if(!newarr){ 
      return NULL // Allocation fails, the old array saved. Check the return value outside the function 
     } 

Wenn realloc erfolgreich zu sein, Sie sicher nicht wollen, kostenlos das neue Array, aktualisieren sie nur den letzten Wert darin, so:

 }else{ 
      free(newarr); 
     } 
     arr[num] = newval; 
     return arr; 

werden sollten:

 newarr[num] = newval; 
     return newarr; 

Answer 2

nicht nur nicht

else{ 
      free(newarr); 
     } 

Sinn machen, sondern auch trotzt sie den Zweck der

int *newarr = realloc(arr, sizeof(int) * (num+1)); 

Wenn realloc erfolgreich ist, arr=newarr zu tun.

---

Referenz

[ realloc ] Referenz sagt:

auf Erfolg, den Zeiger auf den Anfang des neu zugewiesenen Speicher zurückgibt. Der zurückgegebene Zeiger muss mit free() oder realloc() freigegeben werden. Der ursprüngliche Zeiger ptr ist ungültig und jeder Zugriff auf ist ein undefiniertes Verhalten (auch wenn die Neuzuordnung in-Place erfolgte).

Bei einem Fehler wird ein Nullzeiger zurückgegeben. Der ursprüngliche Zeiger ptr bleibt gültig und muss möglicherweise mit free() oder realloc() freigegeben werden.

Answer 3

Ihr Code führt zu undefiniertem Verhalten, wenn Sie nach der Neuzuweisung versuchen, in den ursprünglichen Block zu schreiben. Wenn die Neuzuweisung fehlschlägt - Sie befreien diesen Block manuell, bevor Sie ihn schreiben. In anderen Fällen dürfen Sie diesen Block nicht berühren, da realloc() einen neuen zurückgegeben hat und der ursprüngliche eine Schacht intern freigegeben wurde.Und SIGABRT - ist eine implementierungsspezifische Benachrichtigung über erkannte Heap-Beschädigung oder Missbrauch der malloc-family-API. Lesen Sie ungefähr realloc() sorgfältig.