From: Vadim Zeitlin <vadim@wxwidgets.org>
Date: Thu, 22 Jan 2004 14:39:29 +0000 (+0000)
Subject: fix what seems to be a big memory leaks in SetPageImage()
X-Git-Url: https://git.saurik.com/wxWidgets.git/commitdiff_plain/b870b105ae3f29d5e2d974e5eb6ff904509708d8

fix what seems to be a big memory leaks in SetPageImage()


git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@25305 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
---

diff --git a/src/gtk/notebook.cpp b/src/gtk/notebook.cpp
index 76baec9d8e..07fdd247c7 100644
--- a/src/gtk/notebook.cpp
+++ b/src/gtk/notebook.cpp
@@ -430,17 +430,18 @@ bool wxNotebook::SetPageImage( size_t page, int image )
     {
         /* Case 2) or 4). There is already an image in the gtkhbox. Let's find it */
 
-        GList *child = gtk_container_children(GTK_CONTAINER(nb_page->m_box));
-        while (child)
+        GList *children = gtk_container_children(GTK_CONTAINER(nb_page->m_box));
+        for ( child = children; child; child = child->next )
         {
             if (GTK_IS_PIXMAP(child->data))
             {
                 pixmapwid = GTK_WIDGET(child->data);
                 break;
             }
-            child = child->next;
         }
 
+        g_list_free(children);
+
         /* We should have the pixmap widget now */
         wxASSERT(pixmapwid != NULL);
 
diff --git a/src/gtk1/notebook.cpp b/src/gtk1/notebook.cpp
index 76baec9d8e..07fdd247c7 100644
--- a/src/gtk1/notebook.cpp
+++ b/src/gtk1/notebook.cpp
@@ -430,17 +430,18 @@ bool wxNotebook::SetPageImage( size_t page, int image )
     {
         /* Case 2) or 4). There is already an image in the gtkhbox. Let's find it */
 
-        GList *child = gtk_container_children(GTK_CONTAINER(nb_page->m_box));
-        while (child)
+        GList *children = gtk_container_children(GTK_CONTAINER(nb_page->m_box));
+        for ( child = children; child; child = child->next )
         {
             if (GTK_IS_PIXMAP(child->data))
             {
                 pixmapwid = GTK_WIDGET(child->data);
                 break;
             }
-            child = child->next;
         }
 
+        g_list_free(children);
+
         /* We should have the pixmap widget now */
         wxASSERT(pixmapwid != NULL);