[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