[libvirt] [PATCHv2 0/4] Add LibSSH2 transport

[Daniel P. Berrange]

On Sun, Aug 12, 2012 at 03:30:16PM -0500, Doug Goldstein wrote:
> On Sat, Aug 11, 2012 at 4:20 PM, Peter Krempa <pkrempa at redhat.com> wrote:
> > This is a second respin of the LibSSH2 transport patch series.
> 
> Annoyingly Gmail has decided to eat patch 1/4 so I've been reviewing
> it on the ML archives. I've noticed the following:
> 
> virNetLibSSH2CheckHostKey() the virBuffer buff is leaked in a few
> places. After the following line:
> 
> keyhash = virBufferContentAndReset(&buff);
> 
> buff isn't freed in all the cases that you can return.

Actually its ok. The virBuffer instance for 'buff' is stack allocated.
The only dynamically allocated memory is the string inside the buf which
is what virBufferContentAndReset is giving you. So 'keyhash' is what
needs to be free'd & it looks to be.

I thought we had this documented on the website somewhere, but I guess
not. Here's the original API description for reference:

  https://www.redhat.com/archives/libvir-list/2008-April/msg00371.html

Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|