Ginger88895
/
warzone2100
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

newnet: Check if blocking map pointer is valid by comparing game time to a variable that isn't inside the blocking map structure. 

Must have gotten confused when mixing/refactoring/blending/frying data structures, even the comment was weird. Fixes valgrind warning and probably fixes ticket:2016.
master
Cyp 2010-07-31 11:38:43 +02:00
parent b59b089ca5
commit 1854f49a0c
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/astar.cpp
View File

@ -112,6 +112,8 @@ struct PathBlockingMap
std::vector<bool> map;
};
/// Game time for all blocking maps in fpathBlockingMaps.
static uint32_t fpathCurrentGameTime;
// Data structures used for pathfinding, can contain cached results.
struct PathfindContext
@ -124,8 +126,8 @@ struct PathfindContext
}
bool matches(PathBlockingMap const *blockingMap_, PathCoord tileS_) const
{
// Must check that myGameTime == blockingMap->type.gameTime, otherwise blockingMap be a deleted pointer which coincidentally compares equal to the valid pointer blockingMap_.
return myGameTime == blockingMap->type.gameTime && blockingMap == blockingMap_ && tileS == tileS_;
// Must check myGameTime == fpathCurrentGameTime, otherwise blockingMap be a deleted pointer which coincidentally compares equal to the valid pointer blockingMap_.
return myGameTime == fpathCurrentGameTime && blockingMap == blockingMap_ && tileS == tileS_;
}
void assign(PathBlockingMap const *blockingMap_, PathCoord tileS_)
{
@ -166,8 +168,6 @@ static std::list<PathfindContext> fpathContexts;
static std::list<PathBlockingMap> fpathBlockingMaps;
/// Lists of blocking maps from previous tick, will be cleared next tick (since it will be no longer needed after that).
static std::list<PathBlockingMap> fpathPrevBlockingMaps;
/// Game time for all blocking maps in fpathBlockingMaps.
static uint32_t fpathCurrentGameTime;
// Convert a direction into an offset
// dir 0 => x = 0, y = -1