[prev in list] [next in list] [prev in thread] [next in thread] 

List:       freedesktop-dbus
Subject:    Re: Memory leaks in glib bindings
From:       Colin Walters <walters () verbum ! org>
Date:       2005-06-22 23:34:48
Message-ID: 1119483288.4541.8.camel () nexus ! verbum ! private
[Download RAW message or body]


On Tue, 2005-06-21 at 10:47 +0100, Ross Burton wrote:
> On Mon, 2005-06-20 at 15:55 -0400, Colin Walters wrote:
> > Ok, I made that change, and also added a g_free (context) in the
> > _return_error function.  I'm less sure if this is right though because
> > there are no async server tests in test/glib (hint :)).
> 
> This change slipped through the net when you rewrote the patch:
> 
> Index: glib/dbus-gobject.c
> ===================================================================
> --- glib/dbus-gobject.c	(.../upstream/current)	(revision 201)
> +++ glib/dbus-gobject.c	(.../trunk)	(revision 201)
> @@ -1593,6 +1595,7 @@
>    dbus_g_connection_unref (context->connection);
>    dbus_g_message_unref (context->message);
>    g_free (context);
> +  g_free (out_sig);
>  }

Thanks, applied.

["signature.asc" (application/pgp-signature)]

[prev in list] [next in list] [prev in thread] [next in thread]