2011/6/28 Eric Blake <eblake@xxxxxxxxxx>: > On 06/28/2011 06:08 AM, Matthias Bolte wrote: >> --- >> >> This patch depends on >> >> https://www.redhat.com/archives/libvir-list/2011-June/msg01423.html > > which is now ACK'd. > >> >> src/network/bridge_driver.c | 23 +++++++++++++---------- >> src/util/dnsmasq.c | 10 ++++------ >> src/util/dnsmasq.h | 4 ++-- >> 3 files changed, 19 insertions(+), 18 deletions(-) >> >> diff --git a/src/network/bridge_driver.c b/src/network/bridge_driver.c >> index f48fdcb..f54a966 100644 >> --- a/src/network/bridge_driver.c >> +++ b/src/network/bridge_driver.c >> @@ -446,7 +446,8 @@ networkBuildDnsmasqHostsfile(dnsmasqContext *dctx, >> for (i = 0; i < ipdef->nhosts; i++) { >> virNetworkDHCPHostDefPtr host = &(ipdef->hosts[i]); >> if ((host->mac) && VIR_SOCKET_HAS_ADDR(&host->ip)) >> - dnsmasqAddDhcpHost(dctx, host->mac, &host->ip, host->name); >> + if (dnsmasqAddDhcpHost(dctx, host->mac, &host->ip, host->name) < 0) >> + return -1; >> } > > Good catch. ACK. Thanks, pushed. -- Matthias Bolte http://photron.blogspot.com -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list