From e43263601fe27feb8d005668528f35f75428b8c6 Mon Sep 17 00:00:00 2001 From: Michal Privoznik Date: Tue, 6 Feb 2018 12:18:40 +0100 Subject: [PATCH] conf: Check for user aliases duplicates only https://bugzilla.redhat.com/show_bug.cgi?id=1553162 When validating a device XML config we check if user provided alias is unique. We do this by maintaining a hash table of device aliases as we iterated over all devices defined for the domain. However, it may happen that what appears as two devices in domain XML is in fact just one interface in hypervisor. We can assume libvirt generated aliases to be unique and thus really check user provided ones only. Signed-off-by: Michal Privoznik --- 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 b98b1ca428..04a6ee77af 100644 --- a/src/conf/domain_conf.c +++ b/src/conf/domain_conf.c @@ -5573,7 +5573,7 @@ virDomainDeviceDefValidateAliasesIterator(virDomainDefPtr def, struct virDomainDefValidateAliasesData *data = opaque; const char *alias = info->alias; - if (!alias) + if (!alias || !virDomainDeviceAliasIsUserAlias(alias)) return 0; /* Some crazy backcompat for consoles. */