zakarya/puzzles
1
0
Fork 0
mirror of git://git.tartarus.org/simon/puzzles.git synced 2025-04-21 08:01:30 -07:00
Code Issues Packages Projects Releases Wiki Activity

Fix two memory leaks reported by Tiago Dionizio in recent Loopy

Browse Source 
development.

[originally from svn r9163]
This commit is contained in:
Simon Tatham
2011-04-26 13:44:27 +00:00
parent 8266b71475
commit 4bab5e531b
2 changed files with 10 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
grid.c
View File

@ -453,6 +453,14 @@ static void grid_trim_vigorously(grid *g)
for (i = newdots = 0; i < g->num_dots; i++) for (i = newdots = 0; i < g->num_dots; i++)
dots[i] = (dots[i] ? newdots++ : -1); dots[i] = (dots[i] ? newdots++ : -1);
/*
* Free the dynamically allocated 'dots' pointer lists in faces
* we're going to discard.
*/
for (i = 0; i < g->num_faces; i++)
if (faces[i] < 0)
sfree(g->faces[i].dots);
/* /*
* Go through and compact the arrays. * Go through and compact the arrays.
*/ */

2
loopy.c
View File

@ -917,6 +917,8 @@ static game_drawstate *game_new_drawstate(drawing *dr, game_state *state)
static void game_free_drawstate(drawing *dr, game_drawstate *ds) static void game_free_drawstate(drawing *dr, game_drawstate *ds)
{ {
sfree(ds->textx);
sfree(ds->texty);
sfree(ds->clue_error); sfree(ds->clue_error);
sfree(ds->clue_satisfied); sfree(ds->clue_satisfied);
sfree(ds->lines); sfree(ds->lines);