[Gnucash-changes] r13472 - gnucash/trunk - Fixes to memory deallocation suggested by Phil Longstaff.

[Joshua Sled]

Author: jsled
Date: 2006-03-04 10:50:20 -0500 (Sat, 04 Mar 2006)
New Revision: 13472
Trac: http://svn.gnucash.org/trac/changeset/13472

Modified:
   gnucash/trunk/ChangeLog
   gnucash/trunk/src/gnome-utils/gnc-menu-extensions.c
Log:
Fixes to memory deallocation suggested by Phil Longstaff.


Modified: gnucash/trunk/ChangeLog
===================================================================
--- gnucash/trunk/ChangeLog	2006-03-04 06:56:27 UTC (rev 13471)
+++ gnucash/trunk/ChangeLog	2006-03-04 15:50:20 UTC (rev 13472)
@@ -1,3 +1,9 @@
+2006-03-04  Joshua Sled  <jsled at asynchronous.org>
+
+	* src/gnome-utils/gnc-menu-extensions.c (gnc_extension_path):
+	Fixes to memory deallocation suggested by Phil Longstaff
+	<plongstaff at newearth.org>.
+
 2006-03-04  David Hampton  <hampton at employees.org>
 
 	* src/gnome-utils/dialog-preferences.c:

Modified: gnucash/trunk/src/gnome-utils/gnc-menu-extensions.c
===================================================================
--- gnucash/trunk/src/gnome-utils/gnc-menu-extensions.c	2006-03-04 06:56:27 UTC (rev 13471)
+++ gnucash/trunk/src/gnome-utils/gnc-menu-extensions.c	2006-03-04 15:50:20 UTC (rev 13472)
@@ -127,6 +127,7 @@
   SCM path;
   gchar **strings;
   gint i;
+  gint num_strings;
 
   initialize_getters();
 
@@ -136,7 +137,8 @@
     return;
   }
 
-  strings = g_new0(gchar *, scm_ilength(path) + 2);
+  num_strings = scm_ilength(path) + 2;
+  strings = g_new0(gchar *, num_strings);
   strings[0] = "/menubar";
 
   i = 1;
@@ -169,6 +171,14 @@
 
   *fullpath = g_strjoinv("/", strings);
 
+  for (i = 1; i < num_strings; i++)
+  {
+     if (strings[i] != NULL)
+     {
+        g_free(strings[i]);
+     }
+  }	
+
   g_free(strings);
 }