<p dir="ltr"><br>
On Sep 10, 2014 10:47 PM, "Fabiano Fidêncio" <<a href="mailto:fabiano@fidencio.org">fabiano@fidencio.org</a>> wrote:<br>
><br>
><br>
> On Sep 10, 2014 8:42 PM, "Jonathon Jongsma" <<a href="mailto:jjongsma@redhat.com">jjongsma@redhat.com</a>> wrote:<br>
> ><br>
> > Don't leak priv->uri or priv->domkey from VirtViewer class<br>
> > ---<br>
> >  src/virt-viewer.c | 8 ++++++++<br>
> >  1 file changed, 8 insertions(+)<br>
> ><br>
> > diff --git a/src/virt-viewer.c b/src/virt-viewer.c<br>
> > index c03503e..55780af 100644<br>
> > --- a/src/virt-viewer.c<br>
> > +++ b/src/virt-viewer.c<br>
> > @@ -518,6 +518,14 @@ virt_viewer_dispose (GObject *object)<br>
> >          virDomainFree(priv->dom);<br>
> >      if (priv->conn)<br>
> >          virConnectClose(priv->conn);<br>
> > +    if (priv->uri) {<br>
> > +        g_free(priv->uri);<br>
> > +        priv->uri = NULL;<br>
> > +    }<br>
> > +    if (priv->domkey) {<br>
> > +        g_free(priv->domkey);<br>
> > +        priv->domkey = NULL;<br>
> > +    }<br>
> >      G_OBJECT_CLASS(virt_viewer_parent_class)->dispose (object);<br>
> >  }<br>
> ><br>
> > --<br>
> > 1.9.3<br>
> ><br>
> > _______________________________________________<br>
> > virt-tools-list mailing list<br>
> > <a href="mailto:virt-tools-list@redhat.com">virt-tools-list@redhat.com</a><br>
> > <a href="https://www.redhat.com/mailman/listinfo/virt-tools-list">https://www.redhat.com/mailman/listinfo/virt-tools-list</a><br>
><br>
> Back.</p>
<p dir="ltr">I meant ACK. :)<br>
</p>