From: Michal Privoznik <mpriv...@redhat.com>

In one instance, after virDomainDefGetSecurityLabelDef(SECURITY_DAC_NAME)
was called, we take the returned secdef and compare secdef->model against
SECURITY_DAC_NAME. This makes no sense because
virDomainDefGetSecurityLabelDef() has already done this comparison.

Signed-off-by: Michal Privoznik <mpriv...@redhat.com>
---
 src/security/security_dac.c | 8 --------
 1 file changed, 8 deletions(-)

diff --git a/src/security/security_dac.c b/src/security/security_dac.c
index 2210117f12..8fb26168ca 100644
--- a/src/security/security_dac.c
+++ b/src/security/security_dac.c
@@ -2404,14 +2404,6 @@ virSecurityDACGenLabel(virSecurityManager *mgr,
         return rc;
     }
 
-    if (seclabel->model
-        && STRNEQ(seclabel->model, SECURITY_DAC_NAME)) {
-        virReportError(VIR_ERR_INTERNAL_ERROR,
-                       _("security label model %1$s is not supported with 
selinux"),
-                       seclabel->model);
-            return rc;
-    }
-
     switch ((virDomainSeclabelType)seclabel->type) {
     case VIR_DOMAIN_SECLABEL_STATIC:
         if (seclabel->label == NULL) {
-- 
2.49.1

Reply via email to