On Wed, Mar 27, 2013 at 3:53 PM, Christophe Fergeau <cferg...@redhat.com> wrote:
> +    if (c->conn) {
> +        g_object_unref(c->conn);
> +        c->sock = NULL;
> +        c->conn = NULL;
> +    }
>      if (c->sock) {
>          g_object_unref(c->sock);
>          c->sock = NULL;

Can we remove the if (c->sock) block below?

It looks quite suspicious that we can have either a strong or a weak
ref on c->sock.


-- 
Marc-André Lureau
_______________________________________________
Spice-devel mailing list
Spice-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/spice-devel

Reply via email to