On Thursday 07 May 2009 10:50:54 Guido Günther wrote:
Hi,
virsh fails with:
10:47:37.104: debug : do_open:922 : no name, allowing driver auto-select
10:47:37.104: debug : do_open:930 : trying driver 0 (Test) ...
10:47:37.104: debug : do_open:936 : driver 0 Test returned DECLINED
10:47:37.104: debug : do_open:930 : trying driver 1 (OPENVZ) ...
10:47:37.105: debug : do_open:936 : driver 1 OPENVZ returned DECLINED
10:47:37.105: debug : do_open:930 : trying driver 2 (VBOX) ...
10:47:37.106: debug : do_open:936 : driver 2 VBOX returned ERROR
10:47:37.106: debug : virUnrefConnect:210 : unref connection 0x10032b10 1
10:47:37.106: debug : virReleaseConnect:171 : release connection 0x10032b10
error: failed to connect to the hypervisor
if one has built with vbox support but vbox is not installed. Attached
patch calls VBoxCGlueInit() early so we can return 'declined' instead of
'error'. O.k. to apply?
the above patch leads to memory leaks as
vboxUninitialize() needs to be called
if VBoxCGlueInit() is called, so modifying it as below solves both the
problems. (declined is right here cause the only error condition is while
conn->uri == NULL is already being handled, and thus the following patch)
Regards,
Pritesh