|
|
Das erste Problem ist, dass write_game mit remove_empty_factions beginnt, welches sowohl destroyfaction als auch free(f) aufruft, damit ist der Speicher dann also zum Zeitpunkt des späteren Zugriffs unbrauchbar. Das muss evtl. nicht gemacht werden, denn die Partei ist ja nach destroyfaction nicht mehr in der Parteien-Liste, wird also eh nicht ins Datenfile geschrieben.
Zweites Problem: setalliance(f, NULL) wird in destroyfaction nur gemacht, wenn die Partei der Anführer der Allianz ist. DAs löscht sie dann nicht aus der Liste, warum auch immer. Ich glaube, die Bedingung kann da einfach weg. |
|
|
Oh, Konfusion. Der Crash ist wegen f->allies, nicht f->alliance. Das ist der HELFE Status, da löscht die Partei natürlich niemand raus (das Reverse Lookup dafür ist zu aufwändig). D.h. die Partei muss zum Zeitpunkt von write_game noch existieren, und f->_alive gecheckt werden. |
