kieranmillar
Level: Smitemaster
Rank Points: 3787
Registered: 07-11-2014
IP: Logged

DROD has a static array in TileConstants.h called TILE_LAYER, it's an array, indexed by the tile number, that returns what layer it is on. Any time DROD wants to know what layer a tile is on, it access this array with TILE_LAYER[tile]

This is bad because some tiles are virtual tiles, with negative values (then cast to be unsigned) which are massively out of bounds. We should be bounds checking this array everywhere we use it.

What makes more sense is to not allow access to this array directly, but rather via a static method GetLayerForTile(tileId) which bounds checks it every time it is used, and returns some value to represent an error if the tile was a virtual one out of the bounds of the array, then ASSERT or manage this wherever it is used.

Should be done for classic DROD and RPG.