cpp/openttd-patchpack/source Changeset - r133:9f98592d67bf

Changeset - r133:9f98592d67bf

Parent rev.

Child rev.

[Not reviewed]

master

0 2 0

truelight - 20 years ago 2004-08-24 20:54:40
truelight@openttd.org

(svn r134) -Fix: [976583] parent_list was too small
-Fix: [981934] Memoryleak in parent_list
-Add: General protection around Sprites

2 files changed with 28 insertions and 11 deletions:

viewport.c

viewport.h

0 comments (0 inline, 0 general)

viewport.c

➞

Show inline comments

@@ @@ -7,7 +7,7 @@ @@
 #include "gfx.h"
 #include "town.h"
 #define VIEWPORT_DRAW_MEM (65536)
+#define VIEWPORT_DRAW_MEM (65536 * 2)
 static bool _added_tile_sprite;
 static bool _offset_ground_sprites;
@@ @@ -57,6 +57,7 @@ typedef struct ViewportDrawer { @@
 	ChildScreenSpriteToDraw **last_child;
 	ParentSpriteToDraw **parent_list;
 	ParentSpriteToDraw **eof_parent_list;
 	byte combine_sprites;
@@ @@ -326,7 +327,7 @@ Point GetTileZoomCenter(bool in) @@
 	return GetTileFromScreenXY(x, y);
+}
-void DrawGroundSpriteAt(uint32 image, int x, int y, byte z)
+void DrawGroundSpriteAt(uint32 image, int16 x, int16 y, byte z)
+{
 	ViewportDrawer *vd = _cur_vd;
 	TileSpriteToDraw *ts;
@@ @@ -340,7 +341,7 @@ void DrawGroundSpriteAt(uint32 image, in @@
+	}
 	vd->spritelist_mem += sizeof(TileSpriteToDraw);
 	ts->image = image;
 	ts->next = NULL;
 	ts->x = x;
@@ @@ -410,6 +411,18 @@ void AddSortableSpriteToDraw(uint32 imag @@
 		DEBUG(misc, 0) ("Out of sprite mem\n");
 		return;
+	}
 	if (vd->parent_list >= vd->eof_parent_list) {
 		// This can happen rarely, mostly when you zoom out completely
 		//  and have a lot of stuff that moves (and is added to the
 		//  sort-list, this function). To solve it, increase
 		//  parent_list somewhere below to a higher number.
 		// This can not really hurt you, it just gives some black
 		//  spots on the screen ;)
 		DEBUG(misc, 0) ("Out of sprite mem (parent_list)\n");
 		return;
+	}
 	vd->spritelist_mem += sizeof(ParentSpriteToDraw);
 	ps->image = image;
 	ps->tile_x = x;
@@ @@ -435,7 +448,6 @@ void AddSortableSpriteToDraw(uint32 imag @@
 	ps->child = NULL;
 	vd->last_child = &ps->child;
 	vd->spritelist_mem += sizeof(ParentSpriteToDraw);
 	*vd->parent_list++ = ps;
 	if (vd->combine_sprites == 1) {
@@ @@ -457,6 +469,8 @@ void AddChildSpriteScreen(uint32 image, @@
+{
 	ViewportDrawer *vd = _cur_vd;
 	ChildScreenSpriteToDraw *cs;
 	assert( (image & 0x3fff) < NUM_SPRITES);
 	cs = (ChildScreenSpriteToDraw*) vd->spritelist_mem;
 	if ((byte*)cs >= vd->eof_spritelist_mem) {
@@ @@ -466,12 +480,12 @@ void AddChildSpriteScreen(uint32 image, @@
 	if (vd->last_child == NULL)
 		return;
 	vd->spritelist_mem += sizeof(ChildScreenSpriteToDraw);
 	*vd->last_child = cs;
 	vd->last_child = &cs->next;
 	vd->spritelist_mem += sizeof(ChildScreenSpriteToDraw);
 	cs->image = image;
 	cs->x = x;
 	cs->y = y;
@@ @@ -1122,7 +1136,7 @@ void ViewportDoDraw(ViewPort *vp, int le @@
 	DrawPixelInfo *old_dpi;
 	byte mem[VIEWPORT_DRAW_MEM];
-	ParentSpriteToDraw *parent_list[750];
+	ParentSpriteToDraw *parent_list[1000];
 	_cur_vd = &vd;
@@ @@ -1147,6 +1161,7 @@ void ViewportDoDraw(ViewPort *vp, int le @@
 	vd.dpi.dst_ptr = old_dpi->dst_ptr + x - old_dpi->left + (y - old_dpi->top) * old_dpi->pitch;
 	vd.parent_list = parent_list;
 	vd.eof_parent_list = &parent_list[lengthof(parent_list)];
 	vd.spritelist_mem = mem;
 	vd.eof_spritelist_mem = &mem[sizeof(mem) - 0x40];
 	vd.last_string = &vd.first_string;
@@ @@ -1165,11 +1180,13 @@ void ViewportDoDraw(ViewPort *vp, int le @@
 	ViewportAddCheckpoints(&vd.dpi);
 #endif
 	// This assert should never happen (because the length of the parent_list
 	//  is checked)
 	assert(vd.parent_list - parent_list <= lengthof(parent_list));
 	if (vd.first_tile != NULL)
 		ViewportDrawTileSprites(vd.first_tile);
 	assert(vd.parent_list - parent_list <= lengthof(parent_list));
 	/* null terminate parent sprite list */
 	*vd.parent_list = NULL;
@@ @@ -1178,7 +1195,7 @@ void ViewportDoDraw(ViewPort *vp, int le @@
 	if (vd.first_string != NULL)
 		ViewportDrawStrings(&vd.dpi, vd.first_string);
 	_cur_dpi = old_dpi;
+}

viewport.h

➞

Show inline comments

@@ @@ -24,7 +24,7 @@ void UpdateViewportPosition(Window *w); @@
 void OffsetGroundSprite(int x, int y);
 void DrawGroundSprite(uint32 image);
-void DrawGroundSpriteAt(uint32 image, int x, int y, byte z);
+void DrawGroundSpriteAt(uint32 image, int16 x, int16 y, byte z);
 void AddSortableSpriteToDraw(uint32 image, int x, int y, int w, int h, byte dz, byte z);
 void *AddStringToDraw(int x, int y, StringID string, uint32 params_1, uint32 params_2);
 void AddChildSpriteScreen(uint32 image, int x, int y);

0 comments (0 inline, 0 general)