This matches the behavior of qemuDomainGetImageIds() which we are about to convert to. I think it's safe to depend on the security driver to have already validated this for us. Signed-off-by: Cole Robinson <crobinso@redhat.com> --- src/qemu/qemu_domain.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c index 6695f32c01..a0f41c436e 100644 --- a/src/qemu/qemu_domain.c +++ b/src/qemu/qemu_domain.c @@ -10703,9 +10703,8 @@ qemuDomainOpenFile(virQEMUDriverConfig *cfg, if (def && (seclabel = virDomainDefGetSecurityLabelDef(def, "dac")) != NULL && - seclabel->label != NULL && - (virParseOwnershipIds(seclabel->label, &user, &group) < 0)) - return -EINVAL; + seclabel->label != NULL) + virParseOwnershipIds(seclabel->label, &user, &group); return virQEMUFileOpenAs(user, group, dynamicOwnership, path, oflags, needUnlink); -- 2.53.0