[glob2-devel] valgrind: conditional jump on uninitialized variable

[Stéphane Magnenat]

I just ran glob2 under valgrind for a while and got the following warnings in 
the gradient code:

==9181== Conditional jump or move depends on uninitialised value(s)
==9181==    at 0x813E75C: Map::updateLocalGradient(Building*, bool) 
(Map.cpp:3374)
==9181==    by 0x813ECDE: Map::buildingAvailable(Building*, bool, int, int, 
int*) (Map.cpp:3969)
==9181==    by 0x80B3813: Building::subscribeToBringRessourcesStep() 
(Building.cpp:1324)
==9181==    by 0x81FDBCF: Team::updateAllBuildingTasks() (Team.cpp:936)
==9181==    by 0x81FF9C7: Team::syncStep() (Team.cpp:1107)
==9181==    by 0x80E1C72: Game::syncStep(int) (Game.cpp:1117)
==9181==    by 0x80DA58A: Engine::run() (Engine.cpp:324)
==9181==    by 0x811A381: Glob2::run(int, char**) (Glob2.cpp:378)
==9181==    by 0x811ABBE: main (Glob2.cpp:502)
==9181==
==9181== Conditional jump or move depends on uninitialised value(s)
==9181==    at 0x813E76F: Map::updateLocalGradient(Building*, bool) 
(Map.cpp:3374)
==9181==    by 0x813ECDE: Map::buildingAvailable(Building*, bool, int, int, 
int*) (Map.cpp:3969)
==9181==    by 0x80B3813: Building::subscribeToBringRessourcesStep() 
(Building.cpp:1324)
==9181==    by 0x81FDBCF: Team::updateAllBuildingTasks() (Team.cpp:936)
==9181==    by 0x81FF9C7: Team::syncStep() (Team.cpp:1107)
==9181==    by 0x80E1C72: Game::syncStep(int) (Game.cpp:1117)
==9181==    by 0x80DA58A: Engine::run() (Engine.cpp:324)
==9181==    by 0x811A381: Glob2::run(int, char**) (Glob2.cpp:378)
==9181==    by 0x811ABBE: main (Glob2.cpp:502)
==9181==
==9181== Conditional jump or move depends on uninitialised value(s)
==9181==    at 0x813E796: Map::updateLocalGradient(Building*, bool) 
(Map.cpp:3375)
==9181==    by 0x813ECDE: Map::buildingAvailable(Building*, bool, int, int, 
int*) (Map.cpp:3969)
==9181==    by 0x80B3813: Building::subscribeToBringRessourcesStep() 
(Building.cpp:1324)
==9181==    by 0x81FDBCF: Team::updateAllBuildingTasks() (Team.cpp:936)
==9181==    by 0x81FF9C7: Team::syncStep() (Team.cpp:1107)
==9181==    by 0x80E1C72: Game::syncStep(int) (Game.cpp:1117)
==9181==    by 0x80DA58A: Engine::run() (Engine.cpp:324)
==9181==    by 0x811A381: Glob2::run(int, char**) (Glob2.cpp:378)
==9181==    by 0x811ABBE: main (Glob2.cpp:502)

This might trigger desync in network games.

Can the people who implemented the current gradient code have a look at it?

Thanks,

Steph

-- 
http://stephane.magnenat.net