From 876c8b3bd3fcb8407825bc3adb38875cedf24935 Mon Sep 17 00:00:00 2001
From: "Daniel P. Berrange" <berrange@redhat.com>
Date: Thu, 20 Oct 2011 14:56:30 +0100
Subject: [PATCH] Default console target type with no <target> element

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 | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index e88a1cfdac..84743c1952 100644
--- a/src/conf/domain_conf.c
+++ b/src/conf/domain_conf.c
@@ -3997,6 +3997,7 @@ virDomainChrDefParseXML(virCapsPtr caps,
     const char *nodeName;
     virDomainChrDefPtr def;
     int remaining;
+    bool seenTarget = false;
 
     if (!(def = virDomainChrDefNew()))
         return NULL;
@@ -4026,6 +4027,7 @@ virDomainChrDefParseXML(virCapsPtr caps,
         while (cur != NULL) {
             if (cur->type == XML_ELEMENT_NODE) {
                 if (xmlStrEqual(cur->name, BAD_CAST "target")) {
+                    seenTarget = true;
                     if (virDomainChrDefParseTargetXML(caps, def, cur) < 0) {
                         goto error;
                     }
@@ -4035,6 +4037,10 @@ virDomainChrDefParseXML(virCapsPtr caps,
         }
     }
 
+    if (!seenTarget &&
+        ((def->targetType = virDomainChrDefaultTargetType(caps, def->deviceType)) < 0))
+        goto cleanup;
+
     if (def->source.type == VIR_DOMAIN_CHR_TYPE_SPICEVMC) {
         if (def->targetType != VIR_DOMAIN_CHR_CHANNEL_TARGET_TYPE_VIRTIO) {
             virDomainReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
-- 
2.47.2