Remove useless condition in networkRadvdConfContents

If v6present is false, this code is not reachable.
Also, there is no need to check for errors twice.

diff --git a/src/network/bridge_driver.c b/src/network/bridge_driver.c
index 8624f1e77c349556d49805e0a68b7a926dae279f..149a8fdb25522478b20059e5177076809917e75b 100644 (file)
--- a/src/network/bridge_driver.c
+++ b/src/network/bridge_driver.c
@@ -1535,20 +1535,15 @@ networkRadvdConfContents(virNetworkObjPtr network, char **configstr)
         VIR_FREE(netaddr);
     }
 
-    /* only create the string if we found at least one IPv6 address */
-    if (v6present) {
-        virBufferAddLit(&configbuf, "};\n");
+    virBufferAddLit(&configbuf, "};\n");
 
-        if (virBufferError(&configbuf)) {
-            virReportOOMError();
-            goto cleanup;
-        }
-        if (!(*configstr = virBufferContentAndReset(&configbuf))) {
-            virReportOOMError();
-            goto cleanup;
-        }
+    if (virBufferError(&configbuf)) {
+        virReportOOMError();
+        goto cleanup;
     }
 
+    *configstr = virBufferContentAndReset(&configbuf);
+
     ret = 0;
  cleanup:
     virBufferFreeAndReset(&configbuf);