[PATCH 08/16] qemuBuildChrChardevStr: Use proper type for the switch statement

Peter Krempa pkrempa at redhat.com
Thu Nov 18 16:33:33 UTC 2021


Add the missing cases.

Signed-off-by: Peter Krempa <pkrempa at redhat.com>
---
 src/qemu/qemu_command.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c
index 9293a413e6..f97cdc70ab 100644
--- a/src/qemu/qemu_command.c
+++ b/src/qemu/qemu_command.c
@@ -5137,7 +5137,7 @@ qemuBuildChrChardevStr(virLogManager *logManager,
     if (!(charAlias = qemuAliasChardevFromDevAlias(alias)))
         return NULL;

-    switch (dev->type) {
+    switch ((virDomainChrType) dev->type) {
     case VIR_DOMAIN_CHR_TYPE_NULL:
         virBufferAsprintf(&buf, "null,id=%s", charAlias);
         break;
@@ -5290,6 +5290,8 @@ qemuBuildChrChardevStr(virLogManager *logManager,
                           dev->data.spiceport.channel);
         break;

+    case VIR_DOMAIN_CHR_TYPE_NMDM:
+    case VIR_DOMAIN_CHR_TYPE_LAST:
     default:
         virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
                        _("unsupported chardev '%s'"),
-- 
2.31.1




More information about the libvir-list mailing list