]> xenbits.xensource.com Git - libvirt.git/commitdiff
conf: validate serial port model in ABI checks
authorDaniel P. Berrangé <berrange@redhat.com>
Wed, 2 Feb 2022 11:43:09 +0000 (11:43 +0000)
committerDaniel P. Berrangé <berrange@redhat.com>
Thu, 3 Feb 2022 10:59:03 +0000 (10:59 +0000)
The serial port model cannot be allowed to change across migration
as it affects ABI.

Reviewed-by: Andrea Bolognani <abologna@redhat.com>
Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
src/conf/domain_conf.c

index e931aef6c40010d8b2423def41a4dd1c07c5c833..c8df6dcd4d3afb58463882da0e5ab96183edb971 100644 (file)
@@ -21094,6 +21094,14 @@ virDomainSerialDefCheckABIStability(virDomainChrDef *src,
         return false;
     }
 
+    if (src->targetModel != dst->targetModel) {
+        virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
+                       _("Target serial model %s does not match source %s"),
+                       virDomainChrSerialTargetModelTypeToString(dst->targetModel),
+                       virDomainChrSerialTargetModelTypeToString(src->targetModel));
+        return false;
+    }
+
     if (src->target.port != dst->target.port) {
         virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
                        _("Target serial port %d does not match source %d"),