Fix mistakes in checking return values

Thre was a syntax error in checking virRegisterStateDriver in
the remote driver, and bogus checking of a void return type
of virDomainConfNWFilterRegister in nwfilter.

Signed-off-by: Daniel P. Berrange <berrange@redhat.com>

diff --git a/src/nwfilter/nwfilter_driver.c b/src/nwfilter/nwfilter_driver.c
index 228794de772afc2b7f761f485939f5b9220cedf3..8ce2b7df830adad1d34890d69f1f36a709e5e393 100644 (file)
--- a/src/nwfilter/nwfilter_driver.c
+++ b/src/nwfilter/nwfilter_driver.c
@@ -720,7 +720,6 @@ int nwfilterRegister(void) {
         return -1;
     if (virRegisterStateDriver(&stateDriver) < 0)
         return -1;
-    if (virDomainConfNWFilterRegister(&domainNWFilterDriver) < 0)
-        return -1;
+    virDomainConfNWFilterRegister(&domainNWFilterDriver);
     return 0;
 }

diff --git a/src/remote/remote_driver.c b/src/remote/remote_driver.c
index c9711bbd1c3d956892ba7291c35128ca2456b5d6..e85f34df440d710b88325c5e3e80c68173300dd2 100644 (file)
--- a/src/remote/remote_driver.c
+++ b/src/remote/remote_driver.c
@@ -7840,7 +7840,7 @@ remoteRegister(void)
     if (virRegisterNWFilterDriver(&nwfilter_driver) < 0)
         return -1;
 #ifdef WITH_LIBVIRTD
-    if (virRegisterStateDriver(&state_driver) == -1) < 0)
+    if (virRegisterStateDriver(&state_driver) < 0)
         return -1;
 #endif