[PATCH 5/5] virHostValidateSecureGuests: Drop useless 'return 0' at the end

Michal Privoznik mprivozn at redhat.com
Tue Jun 8 08:45:10 UTC 2021


Previous patches rendered 'return 0' at the end of the function a
dead code. Therefore, the code can be rearranged a bit and the
line can be dropped.

Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
---
 tools/virt-host-validate-common.c | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/tools/virt-host-validate-common.c b/tools/virt-host-validate-common.c
index 9ec4e6f00b..8ce45609e7 100644
--- a/tools/virt-host-validate-common.c
+++ b/tools/virt-host-validate-common.c
@@ -517,11 +517,9 @@ int virHostValidateSecureGuests(const char *hvname,
                            "disabled in firemare.");
             return VIR_HOST_VALIDATE_FAILURE(level);
         }
-    } else {
-        virHostMsgFail(level,
-                       "Unknown if this platform has Secure Guest support");
-        return VIR_HOST_VALIDATE_FAILURE(level);
     }
 
-    return 0;
+    virHostMsgFail(level,
+                   "Unknown if this platform has Secure Guest support");
+    return VIR_HOST_VALIDATE_FAILURE(level);
 }
-- 
2.31.1




More information about the libvir-list mailing list