[libvirt] [PATCH 1/2] Check if qemu-bridge-helper exists and is executable
Guido Günther
agx at sigxcpu.org
Thu Aug 13 19:36:26 UTC 2015
On Thu, Aug 13, 2015 at 03:59:53PM +0200, Martin Kletzander wrote:
> On Thu, Aug 13, 2015 at 02:42:44PM +0200, Guido Günther wrote:
> >Otherwise the error is just
> >
> > error: Failed to create domain from test1.xml
> > error: failed to retrieve file descriptor for interface: Transport endpoint is not connected
> >
> >since we don't get a sensible error after the fork.
> >---
> >src/qemu/qemu_command.c | 6 ++++++
> >1 file changed, 6 insertions(+)
> >
> >diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c
> >index 5a92ad4..bddc955 100644
> >--- a/src/qemu/qemu_command.c
> >+++ b/src/qemu/qemu_command.c
> >@@ -295,6 +295,12 @@ static int qemuCreateInBridgePortWithHelper(virQEMUDriverConfigPtr cfg,
> > return -1;
> > }
> >
> >+ if (!virFileIsExecutable(cfg->bridgeHelperName)) {
> >+ virReportSystemError(errno, _("'%s' not a suitable bridge helper"),
>
> I think you meant s/ not/is not/
Fixed up and pushed.
Thanks,
-- Guido
More information about the libvir-list
mailing list