[Galago-commits] r2796 - in trunk/libgalago: . libgalago

[galago-commits at freedesktop.org]

Author: chipx86
Date: 2006-05-05 23:18:11 -0700 (Fri, 05 May 2006)
New Revision: 2796

Modified:
   trunk/libgalago/ChangeLog
   trunk/libgalago/NEWS
   trunk/libgalago/libgalago/galago-core.c
Log:
Patch by Ross Burton to fix a memory leak in connection registration. Two strings were being duplicated that shouldn't have been.


Modified: trunk/libgalago/ChangeLog
===================================================================
--- trunk/libgalago/ChangeLog	2006-05-04 08:31:34 UTC (rev 2795)
+++ trunk/libgalago/ChangeLog	2006-05-06 06:18:11 UTC (rev 2796)
@@ -1,3 +1,9 @@
+Fri May 05 23:00:56 PDT 2006  Christian Hammond <chipx86 at chipx86.com>
+
+	* libgalago/galago-core.c:
+	  - Patch by Ross Burton to fix a memory leak in connection registration.
+	    Two strings were being duplicated that shouldn't have been.
+
 Thu Apr 27 22:08:09 PDT 2006  Christian Hammond <chipx86 at chipx86.com>
 
 	* AUTHORS:

Modified: trunk/libgalago/NEWS
===================================================================
--- trunk/libgalago/NEWS	2006-05-04 08:31:34 UTC (rev 2795)
+++ trunk/libgalago/NEWS	2006-05-06 06:18:11 UTC (rev 2796)
@@ -1,4 +1,5 @@
 version 0.5.1:
+	* Fixed a memory leak in connection registration. Patch by Ross Burton.
 	* Added a German translation from Andreas Kohn (Bug #48)
 
 version 0.5.0 (April 20, 2006):

Modified: trunk/libgalago/libgalago/galago-core.c
===================================================================
--- trunk/libgalago/libgalago/galago-core.c	2006-05-04 08:31:34 UTC (rev 2795)
+++ trunk/libgalago/libgalago/galago-core.c	2006-05-06 06:18:11 UTC (rev 2796)
@@ -1238,8 +1238,8 @@
 	}
 
 	/* TODO: Do safety checks. */
-	_core->priv->uid           = g_strdup((char *)list->data);
-	_core->priv->conn_obj_path = g_strdup((char *)list->next->data);
+	_core->priv->uid           = list->data;
+	_core->priv->conn_obj_path = list->next->data;
 	galago_context_set_obj_path_prefix(_core->priv->conn_obj_path);
 
 	g_list_free(list);