]> xenbits.xensource.com Git - libvirt.git/commitdiff
qemu: Add missing 'break' statement in couple of switch()-es
authorMichal Privoznik <mprivozn@redhat.com>
Wed, 9 Nov 2022 12:52:40 +0000 (13:52 +0100)
committerMichal Privoznik <mprivozn@redhat.com>
Wed, 9 Nov 2022 12:52:40 +0000 (13:52 +0100)
In recent commits migration of TPM on shared storage was
introduced. However, I've only complied it with gcc and thus did
not notice that clang build fails due to missing break; at the
end of some (empty) cases in switch() statements.

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
src/qemu/qemu_domain.c
src/qemu/qemu_tpm.c

index 41e616ca480ce082bec92f6e0203db82276d1eea..3435da5bdc4c27e009a6326bb815bdd6ed9a582b 100644 (file)
@@ -1202,6 +1202,7 @@ qemuDomainTPMPrivateFormat(const virDomainTPMDef *tpm,
 
     case VIR_DOMAIN_TPM_TYPE_PASSTHROUGH:
     case VIR_DOMAIN_TPM_TYPE_LAST:
+        break;
     }
 
     return 0;
index dc15514ca6a7724986baa5a1d0286cfa59c72e01..15ee7db7573b224128a9137764f299ccc92f355d 100644 (file)
@@ -1019,6 +1019,7 @@ qemuTPMHasSharedStorage(virDomainDef *def)
             return virFileIsSharedFS(tpm->data.emulator.storagepath) == 1;
         case VIR_DOMAIN_TPM_TYPE_PASSTHROUGH:
         case VIR_DOMAIN_TPM_TYPE_LAST:
+            break;
         }
     }
 
@@ -1038,6 +1039,7 @@ qemuTPMCanMigrateSharedStorage(virDomainDef *def)
             return QEMU_DOMAIN_TPM_PRIVATE(tpm)->swtpm.can_migrate_shared_storage;
         case VIR_DOMAIN_TPM_TYPE_PASSTHROUGH:
         case VIR_DOMAIN_TPM_TYPE_LAST:
+            break;
         }
     }
     return true;