[Xen-devel] [PATCH] [RFC] Domain name issue

[Masaki Kanno <kanno.masaki@xxxxxxxxxxxxxx>]

Hi,

When we tested xm commands, we found a issue about a domain name.
The domain name can use character '-'. However, when character '-' 
was used for the top of the domain name character string, the 
following issue was found. 

# xm list
Name                              ID Mem(MiB) VCPUs State  Time(s)
-domUtemp                          1      512     1 r-----    43.2
Domain-0                           0      492     1 r-----   199.2
# xm list -domUtemp
Error: option -d not recognized

Some xm commands mistook such the domain name for a option. 
We suggest the following patch. The patch changes XendDomainInfo.py 
not to be able to use character '-' for the top of the domain name 
character string. 
Because we are not well informed about python, if there is a good 
idea to solve the issue, please give us idea. 


Signed-off-by: Hiroyuki Yamamoto <yamamoto.hiroyu@xxxxxxxxxxxxxx>
Signed-off-by: Masaki Kanno <kanno.masaki@xxxxxxxxxxxxxx>

Best regards,
 Kan

diff -r 0d1dab1d9b67 tools/python/xen/xend/XendDomainInfo.py
--- a/tools/python/xen/xend/XendDomainInfo.py   Fri Jun 16 10:18:54 2006 -0600
+++ b/tools/python/xen/xend/XendDomainInfo.py   Sun Jun 18 17:09:56 2006 +0900
@@ -1189,6 +1189,8 @@ class XendDomainInfo:
         """
         if name is None or name == '':
             raise VmError('missing vm name')
+        if name[0] == '-':
+            raise VmError('invalid vm name')
         for c in name:
             if c in string.digits: continue
             if c in '_-.:/+': continue

Attachment: domainname.patch
Description: Binary data

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel