Mahtan/wagic
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fixed a rare crash when more then one creature had either treason or unearth in the battlefield at the same turn...they would be put in graveyard and the z->nbcards would change in value, meaning that the incrementer was now able to accidently step out of the vector and hit an invalid pointer. now once it hits the bottom of each pass if the number of cards changed in the field start the cycle over.

Browse Source
This commit is contained in:
omegablast2002@yahoo.com
2010-12-19 21:20:30 +00:00
parent 9d68164132
commit d8d4977e5b
1 changed files with 27 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

35
projects/mtg/src/GameObserver.cpp
View File

@@ -549,20 +549,14 @@ void GameObserver::gameStateBasedEffects()
for (int j = 0; j < nbcards; ++j)
{
MTGCardInstance * c = z->cards[j];
if(!c)break;
while (c->flanked)
{//undoes the flanking on a card
c->power += 1;
c->addToToughness(1);
c->flanked -= 1;
}
if (c->has(Constants::TREASON))
{
WEvent * e = NEW WEventCardSacrifice(c);
GameObserver * game = GameObserver::GetInstance();
game->receiveEvent(e);
p->game->putInGraveyard(c);
}
if (c->has(Constants::UNEARTH)) p->game->putInExile(c);
if (c->fresh) c->fresh = 0;
if (c->has(Constants::ONLYONEBOTH))
{
@@ -571,6 +565,31 @@ void GameObserver::gameStateBasedEffects()
}
}
for (int t = 0; t < nbcards; t++)
{
MTGCardInstance * c = z->cards[t];
if (c->has(Constants::TREASON))
{
WEvent * e = NEW WEventCardSacrifice(c);
GameObserver * game = GameObserver::GetInstance();
game->receiveEvent(e);
p->game->putInGraveyard(c);
}
if (c->has(Constants::UNEARTH))
{
p->game->putInExile(c);
}
if(nbcards > z->nb_cards)
{
t = 0;
nbcards = z->nb_cards;
}
}
MTGGameZone * f = p->game->graveyard;
for (int k = 0; k < f->nb_cards; k++)
{