Re: [Xen-devel] no digit beginning name?

[Tim Freeman <tfreeman@xxxxxxxxxxx>]

On Thu, 7 Jul 2005 16:25:51 +0100
"Ian Pratt" <m+Ian.Pratt@xxxxxxxxxxxx> wrote:

>  
> 
> > Is it going to break anyone's setup if this patch does not 
> > include '/' as a valid character in a name?  As I reported in 
> > the beginning of the thread, it seems to cause an 
> > unrecoverable crash to include '/'
> 
> Disallowing the usual 'problem' characters is probably a good thing
> (e.g. quotes)

Well, I'm asking especially because these five characters are currently
explicitly allowed: 

            if c in '_-.:/+': continue

I was wondering if that was for a reason.  They all seem to work from my limited
testing, except for '/'

Thanks,
Tim


> 
> Ian
> 
> > On Sun, 3 Jul 2005 20:40:56 +0100
> > "Ian Pratt" <m+Ian.Pratt@xxxxxxxxxxxx> wrote:
> > 
> > >  
> > > > In XendDomainInfo.py --> check_name() there is a restriction in 
> > > > place that a domain can not begin with a digit.  If I comment out 
> > > > this check:
> > > > 
> > > >         if name[0] in string.digits:
> > > >             raise VmError('invalid vm name')
> > > > 
> > > > ... I can start and stop a domain whose name starts with a digit 
> > > > just fine.  Is there a reason this restriction is in place?
> > > 
> > > You can also manipulate domains by their domid, so I'm 
> > guess the test 
> > > is an attempt to avoid confusion. Preventing all-numeric 
> > names would 
> > > be more sensible -- can you post a fix.
> > > 
> > > Thanks.
> > > Ian
> > > 
> > > 
> > > _______________________________________________
> > > Xen-devel mailing list
> > > Xen-devel@xxxxxxxxxxxxxxxxxxx
> > > http://lists.xensource.com/xen-devel
> > > 
> > 
> 

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