On Tue, Jan 29, 2013 at 05:45:00PM +0100, Marc-André Lureau wrote: > On Tue, Jan 29, 2013 at 5:10 PM, Christophe Fergeau <cferg...@redhat.com> > wrote: > > The patch seems to have been pushed without addressing this comment? > > Ah sorry, I missed the comments. There is already a > g_clear_object(&s->proxy); or I don't get what you mean.
The code is
static void update_proxy(SpiceSession *self, const gchar *str)
{
SpiceSessionPrivate *s = self->priv;
SpiceProxy *proxy = NULL;
GError *error = NULL;
if (str == NULL)
str = g_getenv("SPICE_PROXY");
if (str == NULL || *str == 0)
return;
update_proxy(session, NULL); will never unset an already set proxy.
Christophe
pgpXGZwutRsaA.pgp
Description: PGP signature
_______________________________________________ Spice-devel mailing list Spice-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/spice-devel
