On 10/20/2011 08:47 AM, Daniel P. Berrange wrote:
From: "Daniel P. Berrange"<[email protected]>

When no<target>  element was set at all, the default console
target type was not being honoured

* src/conf/domain_conf.c: Set default target type for consoles
   with no<target>
---
  src/conf/domain_conf.c |    3 +++
  1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index f90bece..a4d91ed 100644
--- a/src/conf/domain_conf.c
+++ b/src/conf/domain_conf.c
@@ -3913,6 +3913,9 @@ virDomainChrDefParseXML(virCapsPtr caps,
                               nodeName);
      }

+    /* Initialize this now, in case there is no actual 'target' element at all 
*/
+    def->targetType = virDomainChrDefaultTargetType(caps, def->deviceType);
+

ACK.

--
Eric Blake   [email protected]    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

--
libvir-list mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to