On Wed, Oct 07, 2015 at 09:24:14AM +0200, Christophe Fergeau wrote: > Hey, > > On Tue, Oct 06, 2015 at 05:40:59PM +0100, Daniel P. Berrange wrote: > > The 'lang' attribute needs to be free'd with xmlFree. > > > > The install script config param needs to be unref'd > > since when it is added to the install script an extra > > ref is taken. > > > > The install script avatar needs to be unref'd since > > when it is added to the intsall script an extra ref is > > s/intsall/install/ > > > taken. > > > > Signed-off-by: Daniel P. Berrange <berrange@xxxxxxxxxx> > > --- > > osinfo/osinfo_loader.c | 3 +++ > > 1 file changed, 3 insertions(+) > > > > diff --git a/osinfo/osinfo_loader.c b/osinfo/osinfo_loader.c > > index 4e8f942..0c7ddfb 100644 > > --- a/osinfo/osinfo_loader.c > > +++ b/osinfo/osinfo_loader.c > > @@ -342,6 +342,7 @@ static void osinfo_loader_entity(OsinfoLoader *loader, > > break; > > 'lang' will still be leaked when this 'break' is reached. > > ACK otherwise. Thanks will fix before pushing. Regards, 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 :| _______________________________________________ Libosinfo mailing list Libosinfo@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libosinfo
