[libvirt] [PATCH] Catch dnsmasq start failures
Guido Günther
agx at sigxcpu.org
Mon Jul 25 20:37:41 UTC 2011
On Sun, Jul 24, 2011 at 10:08:13AM -0400, Laine Stump wrote:
> On 07/23/2011 05:26 PM, Guido Günther wrote:
> >While we checked the return value we didn't maks sure ret != 0 which
> >resulted in dnsmasq errors being ignored.
> >---
> > src/network/bridge_driver.c | 4 +++-
> > 1 files changed, 3 insertions(+), 1 deletions(-)
> >
> >diff --git a/src/network/bridge_driver.c b/src/network/bridge_driver.c
> >index 554a8ac..4882753 100644
> >--- a/src/network/bridge_driver.c
> >+++ b/src/network/bridge_driver.c
> >@@ -731,8 +731,10 @@ networkStartDhcpDaemon(virNetworkObjPtr network)
> > if (ret< 0)
> > goto cleanup;
> >
> >- if (virCommandRun(cmd, NULL)< 0)
> >+ ret = virCommandRun(cmd, NULL);
> >+ if (ret< 0) {
> > goto cleanup;
> >+ }
>
> ACK. Thanks for finding this!
Pushed. Thanks,
-- Guido
More information about the libvir-list
mailing list