[PATCH] Fixed memory leak placing components.
Peter Clifton
pcjc2 at cam.ac.uk
Sat Jun 23 23:21:39 EDT 2007
When placing components we free'd the complex_place_list without removing
the objects it pointed to. The code now uses s_delete_object_glist(..)
rather than g_list_free(..) to free the complex_place list and its objects.
---
gschem/src/o_complex.c | 3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/gschem/src/o_complex.c b/gschem/src/o_complex.c
index 684b392..dfe0806 100644
--- a/gschem/src/o_complex.c
+++ b/gschem/src/o_complex.c
@@ -437,7 +437,8 @@ void o_complex_end(TOPLEVEL *w_current, int screen_x, int screen_y)
/*! \todo redraw has to happen at the end of all this hack or
* maybe not? */
- g_list_free(w_current->page_current->complex_place_list);
+ s_delete_object_glist(w_current, w_current->page_current->
+ complex_place_list);
w_current->page_current->complex_place_list = NULL;
/* This doesn't allow anything else to be in the selection
--
1.5.2.2
--=-Fu81KyS4ENhW1nvJNOij--
More information about the geda-dev
mailing list