On Wed, Jan 05, 2011 at 05:15:22PM -0700, Eric Blake wrote:
virDomainChrTcpProtocol only accepts particular <protocol
type=...>
values, but we weren't enforcing that in the RelaxNG. The valid
types are also already documented in docs/formatdomain.html.in.
* docs/schemas/domain.rng (qemucdevSrcDef): Restrict list of
supported <protocol type=> values.
---
Noticed this while working on <smartcard>, but it is an independent
issue worth fixing whenever it gets an ack.
docs/schemas/domain.rng | 9 ++++++++-
1 files changed, 8 insertions(+), 1 deletions(-)
diff --git a/docs/schemas/domain.rng b/docs/schemas/domain.rng
index eee9864..a524e4b 100644
--- a/docs/schemas/domain.rng
+++ b/docs/schemas/domain.rng
@@ -1448,7 +1448,14 @@
<optional>
<element name="protocol">
<optional>
- <attribute name="type"/>
+ <attribute name="type">
+ <choice>
+ <value>raw</value>
+ <value>telnet</value>
+ <value>telnets</value>
+ <value>tls</value>
+ </choice>
+ </attribute>
</optional>
</element>
</optional>
ACK, looks fine,
Daniel
--
Daniel Veillard | libxml Gnome XML XSLT toolkit
http://xmlsoft.org/
daniel(a)veillard.com | Rpmfind RPM search engine
http://rpmfind.net/
http://veillard.com/ | virtualization library
http://libvirt.org/