On Mon, May 18, 2015 at 05:44:20PM +0300, Maxim Nestratov wrote: > Cleanup code in prlsdkLoadDomain doesn't take into account the fact > if private domain freeing function is assigned or not. In case it is, > we shouldn't call it manually because virDomainObjListRemove calls it. > Also, allocated def structure should be freed only if it's not > assigned to domain. Otherwise it will be called twice one time by > virDomainObjListRemove and the second by prlsdkLoadDomain itself. > > Signed-off-by: Maxim Nestratov <mnestratov@xxxxxxxxxxxxx> > --- > src/parallels/parallels_sdk.c | 17 ++++++++++++++--- > 1 file changed, 14 insertions(+), 3 deletions(-) ACK, but will let Dmitry sanity check and push this series too. 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 :| -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list