From: Cole Robinson <crobinso@redhat.com>
Date: Tue, 31 Mar 2026 15:39:58 +0000 (-0400)
Subject: qemu: remove qemuDomainOpenFile() TODO comment
X-Git-Tag: v12.3.0-rc1~48
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=a97bb897b5fc9f42c1be50fac1c3192950f995fd;p=thirdparty%2Flibvirt.git

qemu: remove qemuDomainOpenFile() TODO comment

qemuDomainOpenFile() only acts on the 'dac' driver, where 'label'
and 'imagelabel' are always identical (see virSecurityDACGenLabel()).
So there's nothing TODO here

Reviewed-by: Peter Krempa <pkrempa@redhat.com>
Signed-off-by: Cole Robinson <crobinso@redhat.com>
---

diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c
index f494c6469b..91d57ee60b 100644
--- a/src/qemu/qemu_domain.c
+++ b/src/qemu/qemu_domain.c
@@ -10690,7 +10690,6 @@ qemuDomainOpenFile(virQEMUDriverConfig *cfg,
     bool dynamicOwnership = cfg->dynamicOwnership;
     virSecurityLabelDef *seclabel;
 
-    /* TODO: Take imagelabel into account? */
     if (def &&
         (seclabel = virDomainDefGetSecurityLabelDef(def, "dac")) != NULL &&
         seclabel->label != NULL &&