]> xenbits.xensource.com Git - libvirt.git/commit
daemon: Run virStateCleanup conditionally
authorMichal Privoznik <mprivozn@redhat.com>
Tue, 3 Dec 2013 10:27:22 +0000 (11:27 +0100)
committerMichal Privoznik <mprivozn@redhat.com>
Tue, 3 Dec 2013 13:32:08 +0000 (14:32 +0100)
commita602e90bc1e0743d7e801b730e303674d24fa89f
treee2a33d074d7d89318da5994e79c1f0c661fc5a9e
parent9f6f2fa467e57f41e60835e7e94fd2487c9b0e5e
daemon: Run virStateCleanup conditionally

https://bugzilla.redhat.com/show_bug.cgi?id=1033061

Currently, initialization of drivers is done in a separate thread. This
is done for several reasons: a driver that is initialized may require
running event loop, it may take ages to initialize driver (e.g. due to
autostarting domains). While the thread is spawn and run, the main()
continues its execution. However, if something goes bad, or the event
loop is just exited (e.g. due to a --timeout or SIGINT) we try to
cleanup all the drivers. So we have two threads running Initialize() and
Cleanup() concurrently. This may result in accessing stale pointers -
e.g. netcf driver will free() itself in stateCleanup callback, while the
init thread may come, open a dummy connection in order to autostart some
domains and voilĂ : do_open() iterates over interface drivers and
accesses stale netcf driver.

The fix consists in not running stateCleanup if the init thread is still
running.

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