aly/servatrice
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Fix regression from #4281 which caused crash if card was null (#4340)

Browse source
This commit is contained in:
Zach H 2021-05-09 13:10:13 -04:00 committed by GitHub
parent ff6f28390a
commit b9c4b496e4
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
common/server_game.cpp
View file

@ -578,6 +578,10 @@ void Server_Game::unattachCards(GameEventStorage &ges, Server_Player *player)
QMapIterator<QString, Server_CardZone *> zoneIterator(player->getZones());
for (Server_CardZone *zone : player->getZones()) {
for (Server_Card *card : zone->getCards()) {
if (card == nullptr) {
continue;
}
// Make a copy of the list because the original one gets modified during the loop
QList<Server_Card *> attachedCards = card->getAttachedCards();
for (Server_Card *attachedCard : attachedCards) {