diff options
author | celeron55 <celeron55@gmail.com> | 2011-09-24 00:55:05 -0700 |
---|---|---|
committer | celeron55 <celeron55@gmail.com> | 2011-09-24 00:55:05 -0700 |
commit | 63eca0feb2a08aa15f8be1164c4815a44246a2e6 (patch) | |
tree | 06dbb4a78fbe7bf0f631c1aa56527c9c18d0e3ee /src/environment.cpp | |
parent | 789c88509c1eab8407d3451d55eb31f65150e54a (diff) | |
parent | ae561d705fc6f0d762203016c24c26a3a3a6b986 (diff) | |
download | minetest-63eca0feb2a08aa15f8be1164c4815a44246a2e6.tar.gz minetest-63eca0feb2a08aa15f8be1164c4815a44246a2e6.tar.bz2 minetest-63eca0feb2a08aa15f8be1164c4815a44246a2e6.zip |
Merge pull request #10 from joshbeck/master
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; |