NWN: Make sure we don't deref a nullptr in the WalkmeshLoader

This fixes Coverity Scan issue #1399041.
xoreos · Feb 25, 2019 · 658d1d3 · 658d1d3
1 parent 0e2f827
commit 658d1d3
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/src/engines/nwn/walkmeshloader.cpp b/src/engines/nwn/walkmeshloader.cpp
@@ -332,6 +332,9 @@ Common::AABBNode *WalkmeshLoader::buildAABBTree(std::vector<float> &vertices,
 			compatibility = compatTest;
 		}
 
+		if (!bestNode)
+			continue;
+
 		// Try to flatten the tree by pushing to the back the best node so at a following
 		// iteration it will not be choosen if another node has the same compatibility.
 		nodes.remove(bestNode);