]> xenbits.xensource.com Git - libvirt.git/commitdiff
qemu: Introduce qemuDomainGetChardevTLSObjects for hotplug
authorJohn Ferlan <jferlan@redhat.com>
Fri, 21 Oct 2016 13:38:18 +0000 (09:38 -0400)
committerJohn Ferlan <jferlan@redhat.com>
Mon, 24 Oct 2016 11:44:10 +0000 (07:44 -0400)
As it turns out more than one place will need these objects, so rather
than cut-copy-paste in each, make a helper

src/qemu/qemu_hotplug.c

index 258826c532c188692dfbd1af5d3f012d8cca27d4..8dd48ee4914bccb443d630668006104436faf763 100644 (file)
@@ -1474,6 +1474,32 @@ qemuDomainAttachHostPCIDevice(virQEMUDriverPtr driver,
 }
 
 
+static int
+qemuDomainGetChardevTLSObjects(virQEMUDriverConfigPtr cfg,
+                               qemuDomainObjPrivatePtr priv,
+                               virDomainChrSourceDefPtr dev,
+                               char *charAlias,
+                               virJSONValuePtr *tlsProps,
+                               char **tlsAlias)
+{
+    if (dev->type != VIR_DOMAIN_CHR_TYPE_TCP || !cfg->chardevTLS)
+        return 0;
+
+    if (qemuBuildTLSx509BackendProps(cfg->chardevTLSx509certdir,
+                                     dev->data.tcp.listen,
+                                     cfg->chardevTLSx509verify,
+                                     priv->qemuCaps,
+                                     tlsProps) < 0)
+        return -1;
+
+    if (!(*tlsAlias = qemuAliasTLSObjFromChardevAlias(charAlias)))
+        return -1;
+    dev->data.tcp.tlscreds = true;
+
+    return 0;
+}
+
+
 int qemuDomainAttachRedirdevDevice(virQEMUDriverPtr driver,
                                    virDomainObjPtr vm,
                                    virDomainRedirdevDefPtr redirdev)
@@ -1730,19 +1756,9 @@ int qemuDomainAttachChrDevice(virQEMUDriverPtr driver,
     if (qemuDomainChrPreInsert(vmdef, chr) < 0)
         goto cleanup;
 
-    if (dev->type == VIR_DOMAIN_CHR_TYPE_TCP &&
-        cfg->chardevTLS) {
-        if (qemuBuildTLSx509BackendProps(cfg->chardevTLSx509certdir,
-                                         dev->data.tcp.listen,
-                                         cfg->chardevTLSx509verify,
-                                         priv->qemuCaps,
-                                         &tlsProps) < 0)
-            goto cleanup;
-
-        if (!(tlsAlias = qemuAliasTLSObjFromChardevAlias(charAlias)))
-            goto cleanup;
-        dev->data.tcp.tlscreds = true;
-    }
+    if (qemuDomainGetChardevTLSObjects(cfg, priv, dev, charAlias,
+                                       &tlsProps, &tlsAlias) < 0)
+        goto cleanup;
 
     qemuDomainObjEnterMonitor(driver, vm);
     if (tlsAlias) {