]> xenbits.xensource.com Git - libvirt.git/commit
qemuxml2argvtest: Don't overwrite driver stateDir
authorMichal Privoznik <mprivozn@redhat.com>
Fri, 10 Mar 2017 08:55:43 +0000 (09:55 +0100)
committerMichal Privoznik <mprivozn@redhat.com>
Fri, 10 Mar 2017 08:55:43 +0000 (09:55 +0100)
commit887ffbce43144314d730a6a3cdd7396318a148ce
tree180c926a9ed7d92bc33f570da1895501b057f580
parent8af68ea47830b8d32907dc50c6ca4869d14bb862
qemuxml2argvtest: Don't overwrite driver stateDir

This is a very historic artefact. Back in the old days of
830ba76c3e when we had macros to add arguments onto qemu command
line (!) we thought it was a good idea to let qemu write out the
PID file. So we passed -pidfile $stateDir/$domName onto the
command line. Thus, in order for tests to work we needed stable
stateDir in the qemu driver. Unfortunately, after 16efa11aa696
where stateDir is mkdtemp()-d, this approach lead to a leak of
temp dir.

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
tests/qemuxml2argvtest.c