On Thu, Apr 02, 2026 at 11:12:32 -0400, Cole Robinson via Devel wrote:
Fixed to abide domain seclabel model='dac' override
Signed-off-by: Cole Robinson <crobinso@redhat.com> --- src/qemu/qemu_tpm.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/src/qemu/qemu_tpm.c b/src/qemu/qemu_tpm.c index 660410bcba..dacceac678 100644 --- a/src/qemu/qemu_tpm.c +++ b/src/qemu/qemu_tpm.c @@ -1316,16 +1316,20 @@ qemuExtTPMPrepareHost(virQEMUDriver *driver, { g_autoptr(virQEMUDriverConfig) cfg = virQEMUDriverGetConfig(driver); g_autofree char *shortName = virDomainDefGetShortName(def); + uid_t uid; + gid_t gid;
'gid' is not used
if (!shortName) return -1;
+ qemuDomainGetImageIds(cfg, def, NULL, NULL, &uid, &gid);
and this tolerates any of the last two args being NULL.
+ return qemuTPMEmulatorPrepareHost(tpm, cfg->swtpmLogDir, cfg->swtpm_user, cfg->swtpm_group, cfg->swtpmStateDir, - cfg->user, + uid, shortName); }
With 'gid' removed: Reviewed-by: Peter Krempa <pkrempa@redhat.com>