[libvirt] [libvirt-sandbox][PATCH 1/2] Fix logical judgement in get_name

Alex Jia ajia at redhat.com
Fri Aug 9 10:26:46 UTC 2013


Signed-off-by: Alex Jia <ajia at redhat.com>
---
 bin/virt-sandbox-service | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/bin/virt-sandbox-service b/bin/virt-sandbox-service
index 03873c9..26b4a40 100755
--- a/bin/virt-sandbox-service
+++ b/bin/virt-sandbox-service
@@ -453,8 +453,12 @@ WantedBy=multi-user.target
 
     def get_name(self):
         if self.config:
-            return self.config.get_name()
-        raise ValueError([_("Name not configured")])
+            name = self.config.get_name()
+            if not name:
+                raise ValueError([_("Name not configured")])
+            return name
+        sys.stderr.write("The configuration %s does not exist\n" % self.config)
+        sys.exit(1)
 
     def set_copy(self, copy):
         self.copy = copy
-- 
1.8.3.1




More information about the libvir-list mailing list