From 401f9087bc2e4697d1f6018145d668eac1630d4d Mon Sep 17 00:00:00 2001 From: Felipe Borges Date: Mar 24 2020 10:34:46 +0000 Subject: libvirt-6.1.0-2.fc32 Check for disk type correctly in virDomainDiskTranslateSourcePool Signed-off-by: Felipe Borges --- diff --git a/check-for-disk-type-correctly.patch b/check-for-disk-type-correctly.patch new file mode 100644 index 0000000..1ee4a3d --- /dev/null +++ b/check-for-disk-type-correctly.patch @@ -0,0 +1,39 @@ +From 3918dbd84e4951b43f93fbf50ef52be00274850c Mon Sep 17 00:00:00 2001 +From: Michal Privoznik +Date: Mon, 9 Mar 2020 16:40:57 +0100 +Subject: [PATCH] virDomainDiskTranslateSourcePool: Check for disk type + correctly + +When rewriting the virDomainDiskTranslateSourcePool() function in +v6.1.0-rc1~184 a typo was introduced. Previously, we allowed +startup policy only for those volumes which translated to +VIR_STORAGE_TYPE_FILE. But starting with the referenced commit, +the value we checked for was changed to VIR_STORAGE_VOL_FILE +which comes from a different enum and has a different value too. +This is wrong, because virStorageSourceGetActualType() returns a +value from the original enum. + +Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1811728 + +Signed-off-by: Michal Privoznik +Reviewed-by: Peter Krempa +--- + src/conf/domain_conf.c | 2 +- + 1 file changed, 1 insertion(+), 1 deletion(-) + +diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c +index d8471acd2d..d2d97daf80 100644 +--- a/src/conf/domain_conf.c ++++ b/src/conf/domain_conf.c +@@ -31856,7 +31856,7 @@ virDomainDiskTranslateSourcePool(virDomainDiskDefPtr def) + } + + if (def->startupPolicy != 0 && +- virStorageSourceGetActualType(def->src) != VIR_STORAGE_VOL_FILE) { ++ virStorageSourceGetActualType(def->src) != VIR_STORAGE_TYPE_FILE) { + virReportError(VIR_ERR_XML_ERROR, "%s", + _("'startupPolicy' is only valid for " + "'file' type volume")); +-- +2.25.1 + diff --git a/libvirt.spec b/libvirt.spec index 783ec67..c9f33aa 100644 --- a/libvirt.spec +++ b/libvirt.spec @@ -218,7 +218,7 @@ Summary: Library providing a simple virtualization API Name: libvirt Version: 6.1.0 -Release: 1%{?dist} +Release: 2%{?dist} License: LGPLv2+ URL: https://libvirt.org/ @@ -226,6 +226,7 @@ URL: https://libvirt.org/ %define mainturl stable_updates/ %endif Source: https://libvirt.org/sources/%{?mainturl}libvirt-%{version}.tar.xz +Patch0001: check-for-disk-type-correctly.patch Requires: libvirt-daemon = %{version}-%{release} Requires: libvirt-daemon-config-network = %{version}-%{release} @@ -1973,6 +1974,9 @@ exit 0 %changelog +* Tue Mar 24 2020 Felipe Borges - 6.1.0-2 +- Check for disk type correctly in virDomainDiskTranslateSourcePool + * Wed Mar 04 2020 Cole Robinson - 6.1.0-1 - Update to version 6.1.0