]> xenbits.xensource.com Git - libvirt.git/commit
rpc: Fix client crash when server drops connection
authorJiri Denemark <jdenemar@redhat.com>
Tue, 26 Mar 2013 14:50:38 +0000 (15:50 +0100)
committerJiri Denemark <jdenemar@redhat.com>
Wed, 27 Mar 2013 08:00:38 +0000 (09:00 +0100)
commitd8d4aa01d8ef61fd26afc52a2dbef6ae7812d5cc
treeab79e1af3336d180f8e74066474b681206792eb7
parenta1fe02f0e942bdd69e8886d8fa76069343207093
rpc: Fix client crash when server drops connection

Despite the comment stating virNetClientIncomingEvent handler should
never be called with either client->haveTheBuck or client->wantClose
set, there is a sequence of events that may lead to both booleans being
true when virNetClientIncomingEvent is called. However, when that
happens, we must not immediately close the socket as there are other
threads waiting for the buck and they would cause SIGSEGV once they are
woken up after the socket was closed. Another thing is we should clear
all remaining calls in the queue after closing the socket.

The situation that can lead to the crash involves three threads, one of
them running event loop and the other two calling libvirt APIs. The
event loop thread detects an event on client->sock and calls
virNetClientIncomingEvent handler. But before the handler gets a chance
to lock client, the other two threads (T1 and T2) start calling some
APIs. T1 gets the buck and detects EOF on client->sock while processing
its RPC call. Since T2 is waiting for its own call, T1 passes the buck
on to it and unlocks client. But before T2 gets the signal, the event
loop thread wakes up, does its job and closes client->sock. The crash
happens when T2 actually wakes up and tries to do its job using a closed
client->sock.
src/rpc/virnetclient.c