Right now we don't allow any match in that scenario, but really
if a specific type hasn't been requested it means that any type
is considered acceptable and we should allow matching against
both UEFI and BIOS.

Signed-off-by: Andrea Bolognani <[email protected]>
---
 src/qemu/qemu_firmware.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/qemu/qemu_firmware.c b/src/qemu/qemu_firmware.c
index e0e6a3a557..1d414da601 100644
--- a/src/qemu/qemu_firmware.c
+++ b/src/qemu/qemu_firmware.c
@@ -1104,16 +1104,17 @@ qemuFirmwareMatchDomain(const virDomainDef *def,
          * not present, we can still infer this information from
          * other factors. Specifically, the pflash loader type is
          * only used for UEFI, while the rom loader type can be used
-         * both for UEFI and BIOS */
+         * both for UEFI and BIOS. If the loader type has not been
+         * specified, we keep our options open */
         switch (loader->type) {
         case VIR_DOMAIN_LOADER_TYPE_PFLASH:
             wantUEFI = true;
             break;
         case VIR_DOMAIN_LOADER_TYPE_ROM:
+        case VIR_DOMAIN_LOADER_TYPE_NONE:
             wantUEFI = true;
             wantBIOS = true;
             break;
-        case VIR_DOMAIN_LOADER_TYPE_NONE:
         case VIR_DOMAIN_LOADER_TYPE_LAST:
         default:
             break;
-- 
2.53.0

Reply via email to