diff options
| author | peter1138 <peter1138@openttd.org> | 2006-04-21 20:59:04 +0000 |
|---|---|---|
| committer | peter1138 <peter1138@openttd.org> | 2006-04-21 20:59:04 +0000 |
| commit | 112af05a485c3c58c72dab65cbc3426656e1f996 (patch) | |
| tree | 69d65477aa50cc556c35fbddc1e86f8648a76011 /newgrf_engine.c | |
| parent | 42059d45eaea00dd60a9ac6e6076ccd3f28aa0e0 (diff) | |
| download | openttd-112af05a485c3c58c72dab65cbc3426656e1f996.tar.xz | |
(svn r4510) - NewGRF bounds checking:
- check Engine ID is within range
- don't try setting a vehicle name if the string id is not a valid Engine ID
Diffstat (limited to 'newgrf_engine.c')
| -rw-r--r-- | newgrf_engine.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/newgrf_engine.c b/newgrf_engine.c index 679a88cbd..841cba1a9 100644 --- a/newgrf_engine.c +++ b/newgrf_engine.c @@ -102,6 +102,7 @@ static SpriteGroup *engine_custom_sprites[TOTAL_NUM_ENGINES][NUM_GLOBAL_CID]; void SetCustomEngineSprites(EngineID engine, byte cargo, SpriteGroup *group) { + assert(engine < TOTAL_NUM_ENGINES); if (engine_custom_sprites[engine][cargo] != NULL) { DEBUG(grf, 6)("SetCustomEngineSprites: engine `%d' cargo `%d' already has group -- replacing.", engine, cargo); } @@ -610,6 +611,7 @@ StringID _engine_custom_names[TOTAL_NUM_ENGINES]; void SetCustomEngineName(EngineID engine, StringID name) { + assert(engine < lengthof(_engine_custom_names)); _engine_custom_names[engine] = name; } |