0ct0pu5/wesnoth
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix unit [resistance] and [jamming_costs] not being used in sync checks

Browse source 
The typo this commit fixes would direct the game to a non-existent (and
invalid) [jamming_costs resistance] tag instead.

Reported by reluctant_deity on Reddit.
This commit is contained in:
Ignacio R. Morelle 2015-08-19 01:40:13 -03:00
parent c98f3e029d
commit b7e172994f
2 changed files with 3 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
changelog
View file

@ -20,6 +20,8 @@ Version 1.12.4+dev:
specially-crafted input.
* Fixed RECRUIT_UNIT_VARIATIONS core WML macro leaking an internal temporary
variable ($recruited_unit_random_variation).
* Fixed unit [resistance] and [jamming_costs] not being considered for sync
check.
Version 1.12.4:
* Security fixes:

2
src/unit.cpp
View file

@ -3280,7 +3280,7 @@ std::string get_checksum(const unit& u) {
child.recursive_clear_value("name");
}
const std::string child_keys[] = {"advance_from", "defense", "movement_costs", "vision_costs", "jamming_costs" "resistance", ""};
const std::string child_keys[] = {"advance_from", "defense", "movement_costs", "vision_costs", "jamming_costs", "resistance", ""};
for (int i = 0; !child_keys[i].empty(); ++i)
{