]> xenbits.xensource.com Git - libvirt.git/commitdiff
tools: remove unnecessary defaultConn variable
authorJán Tomko <jtomko@redhat.com>
Wed, 6 Apr 2016 08:42:02 +0000 (10:42 +0200)
committerJán Tomko <jtomko@redhat.com>
Wed, 6 Apr 2016 12:47:41 +0000 (14:47 +0200)
vshStrdup returns NULL without exiting on NULL input.

tools/virsh.c

index 8c616d6373953dd6ef6e0155ad864aacded4adb4..fe338397cec74f53c192692e16c12b4ab2ee35f6 100644 (file)
@@ -908,7 +908,6 @@ main(int argc, char **argv)
 {
     vshControl _ctl, *ctl = &_ctl;
     virshControl virshCtl;
-    const char *defaultConn;
     bool ret = true;
 
     memset(ctl, 0, sizeof(vshControl));
@@ -977,8 +976,7 @@ main(int argc, char **argv)
         exit(EXIT_FAILURE);
     }
 
-    if ((defaultConn = virGetEnvBlockSUID("VIRSH_DEFAULT_CONNECT_URI")))
-        ctl->connname = vshStrdup(ctl, defaultConn);
+    ctl->connname = vshStrdup(ctl, virGetEnvBlockSUID("VIRSH_DEFAULT_CONNECT_URI"));
 
     if (!ctl->imode) {
         ret = vshCommandRun(ctl, ctl->cmd);