Browse Source

Fixed crash on visiting multi-creature dwellings (1874)

Ivan Savenko 11 years ago
parent
commit
69c6a3fa79
2 changed files with 7 additions and 6 deletions
  1. 3 5
      lib/mapObjects/CGTownInstance.cpp
  2. 4 1
      lib/mapObjects/CommonConstructors.cpp

+ 3 - 5
lib/mapObjects/CGTownInstance.cpp

@@ -34,13 +34,11 @@ void CGDwelling::initObj()
 
 			if (getOwner() != PlayerColor::NEUTRAL)
 				cb->gameState()->players[getOwner()].dwellings.push_back (this);
-		}
-			//putStack(SlotID(0), new CStackInstance(CreatureID::GOLD_GOLEM, 9));
-			//putStack(SlotID(1), new CStackInstance(CreatureID::DIAMOND_GOLEM, 6));
 
-			//putStack(SlotID(0), new CStackInstance(CreatureID::EARTH_ELEMENTAL, 12));
+			assert(!creatures.empty());
+			assert(!creatures[0].second.empty());
 			break;
-
+		}
 	case Obj::REFUGEE_CAMP:
 		//is handled within newturn func
 		break;

+ 4 - 1
lib/mapObjects/CommonConstructors.cpp

@@ -158,7 +158,9 @@ void CDwellingInstanceConstructor::initTypeData(const JsonNode & input)
 				availableCreatures[i][j] = VLC->creh->creatures[index];
 			});
 		}
+		assert(!availableCreatures[i].empty());
 	}
+
 	guards = input["guards"];
 }
 
@@ -185,10 +187,11 @@ void CDwellingInstanceConstructor::configureObject(CGObjectInstance * object, CR
 	CGDwelling * dwelling = dynamic_cast<CGDwelling*>(object);
 
 	dwelling->creatures.clear();
-	dwelling->creatures.resize(availableCreatures.size());
+	dwelling->creatures.reserve(availableCreatures.size());
 
 	for (auto & entry : availableCreatures)
 	{
+		dwelling->creatures.resize(dwelling->creatures.size() + 1);
 		for (const CCreature * cre : entry)
 			dwelling->creatures.back().second.push_back(cre->idNumber);
 	}