WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

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

To: xen-devel@xxxxxxxxxxxxxxxxxxx
Subject: [Xen-devel] [PATCH] [RFC] Domain name issue
From: Masaki Kanno <kanno.masaki@xxxxxxxxxxxxxx>
Date: Sun, 18 Jun 2006 18:24:09 +0900
Delivery-date: Sun, 18 Jun 2006 02:25:39 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
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
<Prev in Thread] Current Thread [Next in Thread>