From 5c1f82efae32d5bcd286dc07a2b3549176dc5093 Mon Sep 17 00:00:00 2001 From: Roman Bogorodskiy Date: Fri, 13 Jun 2014 18:56:59 +0400 Subject: [PATCH] bhyve: do not cleanup unallocated networks on fail virBhyveProcessStart() calls bhyveNetCleanup() if it fails. However, it might fail earlier than networks are allocated, so modify bhyveNetCleanup() to check if net->ifname is not NULL before going further with the cleanup. --- src/bhyve/bhyve_process.c | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/src/bhyve/bhyve_process.c b/src/bhyve/bhyve_process.c index a5ad68d33e..b8db076366 100644 --- a/src/bhyve/bhyve_process.c +++ b/src/bhyve/bhyve_process.c @@ -78,10 +78,12 @@ bhyveNetCleanup(virDomainObjPtr vm) int actualType = virDomainNetGetActualType(net); if (actualType == VIR_DOMAIN_NET_TYPE_BRIDGE) { - ignore_value(virNetDevBridgeRemovePort( - virDomainNetGetActualBridgeName(net), - net->ifname)); - ignore_value(virNetDevTapDelete(net->ifname)); + if (net->ifname) { + ignore_value(virNetDevBridgeRemovePort( + virDomainNetGetActualBridgeName(net), + net->ifname)); + ignore_value(virNetDevTapDelete(net->ifname)); + } } } }