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