[libvirt] [PATCH 2/2] VirtualBox support (Resending by fixing a autodetection Bug)
Pritesh Kothari
Pritesh.Kothari at Sun.COM
Tue Apr 14 15:11:28 UTC 2009
Hi Daniel,
> > +char g_szVBoxErrMsg[256];
>
> I don't much like the static fixed size error message buffer
Fixed this.
> > + if (conn->uri == NULL) {
> > + conn->uri = xmlParseURI(uid ? "vbox:///session" :
> > "vbox:///system"); + if (conn->uri == NULL) {
> > + vboxError(conn, VIR_ERR_NO_DOMAIN, NULL);
> > + return VIR_DRV_OPEN_ERROR;
>
> Minor bug here, the VIR_ERR_NO_DOMAIN error isn't the correct code for an
> URI parsing error :-)
Fixed this as well. Will post a patch along with others changes suggested on
the list.
Regards,
Pritesh
More information about the libvir-list
mailing list