On Mon, Jul 23, 2007 at 08:55:52PM +0100, Daniel P. Berrange wrote:
> If using the 'virDomainCreateLinux' call to create a VM, a so called
> 'transient' domain will be created - ie one without any config file.
> There is special code in the qemudShutdownVMDaemon method to cleanup
> the resources associated with such domains, in particuarly free'ing
> the struct qemud_vm object. Unfortunately in the virDomainCreateLinux
> codepath this is a problem, because we still need the 'struct qemud_vm'
> object in certain edge cases, and so the caller has to free it. We
> currently have a double free() in that scenario. This patch removes
> the call to qemudFreeVMDaemon from qemudShutdownVMDaemon. Instead it
> is now always the caller's responsibility to cleanup after transient
> domains.
I think a bit weird that a routine which may still fail in some
way can be switched back from returning an int to void and still
generate an improvement. The way we used the return value of Shutdown
is not needed anymore but I wonder about the possibility of other error
case.
+1 for the fix still, that's important
Related bug is
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=249072
for those wondering.
Daniel
--
Red Hat Virtualization group http://redhat.com/virtualization/
Daniel Veillard | virtualization library http://libvirt.org/
veillard@xxxxxxxxxx | libxml GNOME XML XSLT toolkit http://xmlsoft.org/
http://veillard.com/ | Rpmfind RPM search engine http://rpmfind.net/
--
Libvir-list mailing list
Libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list