diff options
| author | Joshua Beck <jxb091000@utdallas.edu> | 2011-09-24 02:11:43 -0500 |
|---|---|---|
| committer | Joshua Beck <jxb091000@utdallas.edu> | 2011-09-24 02:11:43 -0500 |
| commit | ae561d705fc6f0d762203016c24c26a3a3a6b986 (patch) | |
| tree | 06dbb4a78fbe7bf0f631c1aa56527c9c18d0e3ee /src/environment.cpp | |
| parent | 789c88509c1eab8407d3451d55eb31f65150e54a (diff) | |
| download | minetest-ae561d705fc6f0d762203016c24c26a3a3a6b986.tar.gz minetest-ae561d705fc6f0d762203016c24c26a3a3a6b986.tar.bz2 minetest-ae561d705fc6f0d762203016c24c26a3a3a6b986.zip | |
Fixed null pointer dereference errors found by cppcheck
Diffstat (limited to 'src/environment.cpp')
| -rw-r--r-- | src/environment.cpp | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/src/environment.cpp b/src/environment.cpp index 8103b7110..07c535b70 100644 --- a/src/environment.cpp +++ b/src/environment.cpp @@ -1345,8 +1345,6 @@ void ServerEnvironment::deactivateFarObjects(bool force_delete) i.atEnd()==false; i++) { ServerActiveObject* obj = i.getNode()->getValue(); - u16 id = i.getNode()->getKey(); - v3f objectpos = obj->getBasePosition(); // This shouldn't happen but check it if(obj == NULL) @@ -1357,9 +1355,12 @@ void ServerEnvironment::deactivateFarObjects(bool force_delete) continue; } + u16 id = i.getNode()->getKey(); + v3f objectpos = obj->getBasePosition(); + // The block in which the object resides in v3s16 blockpos_o = getNodeBlockPos(floatToInt(objectpos, BS)); - + // If block is active, don't remove if(m_active_blocks.contains(blockpos_o)) continue; |