[libvirt] [PATCH] util: removing dead code inside virstoragefile.

Julio Faracco jcfaracco at gmail.com
Fri Jun 2 01:56:09 UTC 2017


The host address or the socket path have already been checked at the begining 
of the function virStorageSourceParseNBDColonString(). So, when the parameter 
is not a unix socket, there is no reason to check the address again because 
if it does not exists, the logic will fail in the first IF conditional.

Signed-off-by: Julio Faracco <jcfaracco at gmail.com>
---
 src/util/virstoragefile.c | 7 -------
 1 file changed, 7 deletions(-)

diff --git a/src/util/virstoragefile.c b/src/util/virstoragefile.c
index b43acf6..6d8ada9 100644
--- a/src/util/virstoragefile.c
+++ b/src/util/virstoragefile.c
@@ -2635,13 +2635,6 @@ virStorageSourceParseNBDColonString(const char *nbdstr,
             goto cleanup;
 
    } else {
-        if (!backing[1]) {
-            virReportError(VIR_ERR_INTERNAL_ERROR,
-                           _("missing host name in nbd string '%s'"),
-                           nbdstr);
-            goto cleanup;
-        }
-
         if (VIR_STRDUP(src->hosts->name, backing[1]) < 0)
             goto cleanup;
 
-- 
2.7.4




More information about the libvir-list mailing list