This patch adds a check to make sure the config file passed to

vnet-create is correct.  This is better than throwing IOError, because
IOError's are currently reported as xend communication failures.
Also, fix the IOError handling at the end of the file so we don't get
a stack dump if that path is taken.

Signed-off-by: Dan Smith <danms@us.ibm.com>
This commit is contained in:
kaf24@firebug.cl.cam.ac.uk committed 2005-11-04 11:06:41 +01:00
1 parent 06c12463fc
commit d0f9b588c0
1 file changed
+5 -1
+5 -1
View File
@@ -666,6 +666,10 @@ def xm_vnet_list(args):
def xm_vnet_create(args):
arg_check(args, 1, "vnet-create")
conf = args[0]
if not os.access(conf, os.R_OK):
print "File not found: %s" % conf
sys.exit(1)
from xen.xend.XendClient import server
server.xend_vnet_create(conf)
@@ -800,7 +804,7 @@ def main(argv=sys.argv):
except KeyboardInterrupt:
print "Interrupted."
sys.exit(1)
except IOError:
except IOError, ex:
if os.geteuid() != 0:
err("Most commands need root access. Please try again as root.")
else:
Reference in new issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.