[libvirt PATCH] qemu: Avoid segfault when driver initialization fails

Jiri Denemark jdenemar at redhat.com
Tue Jan 26 16:46:26 UTC 2021


On Tue, Jan 26, 2021 at 17:41:54 +0100, Michal Privoznik wrote:
> On 1/26/21 5:08 PM, Jiri Denemark wrote:
> > In case qemuStateInitialize fails for any reason (e.g., a typo in
> > qemu.conf), it properly cleans up after itself and sets qemu_driver back
> > to NULL. A tiny bit later the daemon asks all drivers to shutdown by
> > calling their stateShutdown* APIs. But the implementation of these APIs
> > in QEMU driver expected qemu_driver to be initialized at this point
> > causing a segfault otherwise.
> > 
> > Signed-off-by: Jiri Denemark <jdenemar at redhat.com>
> > ---
> >   src/qemu/qemu_driver.c | 6 ++++++
> >   1 file changed, 6 insertions(+)
> 
> This patch looks familiar :-)
> 
> https://www.redhat.com/archives/libvir-list/2021-January/msg00955.html
> 
> I'm working on alternative approach. Will send shortly.

Oops, I knew something similar was discussed, but I didn't realize it
was the exact same thing... Not to mention I was too lazy to search for
it (mostly because my memory didn't tell me what exactly to search for)
:-)

Jirka




More information about the libvir-list mailing list