On Thu, May 07, 2009 at 10:25:34AM +0100, Daniel P. Berrange wrote: > On Thu, May 07, 2009 at 11:19:01AM +0200, Pritesh Kothari wrote: > > 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) > > Ok, thanks for checking this. ACK to this patch instead of Guido's. Applied now. -- Guido -- Libvir-list mailing list Libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list