Re: [libvirt] [PATCHv2 0/4] Add LibSSH2 transport
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(a)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 :|