v1: https://listman.redhat.com/archives/libvir-list/2023-April/239441.html

Address v1's cleanup bugs by using g_autofree and the concurrency bugs by
forking a subprocess before chdir(). It works!

I have been working on the other solution of putting paths in /var/run, from
https://listman.redhat.com/archives/libvir-list/2023-April/239536.html, but
so my attempts have ballooned out of control. I would like to offer this
improved version of my original patch in the meantime. Plus there's the
unclear impact of the other strategy on upgrades: 
https://listman.redhat.com/archives/libvir-list/2023-April/239470.html,
which means even once that's implemented it will take a long time to vet.
The chdir() approach cuts right to the point.

I'm hoping you'll accept both approaches as complementary and worth adopting.

Original issue: https://gitlab.com/libvirt/libvirt/-/issues/466

Cheers!

Nick Guenther (1):
  qemu: Allow sockets in long or deep paths.

 src/qemu/qemu_command.c | 72 ++++++++++++++++++++++++++++++++---------
 1 file changed, 57 insertions(+), 15 deletions(-)

-- 
2.34.1

Reply via email to