]> xenbits.xensource.com Git - people/liuw/libxenctrl-split/libvirt.git/commit
stream: Check for stream EOF
authorMichal Privoznik <mprivozn@redhat.com>
Tue, 10 Jan 2012 15:57:30 +0000 (16:57 +0100)
committerMichal Privoznik <mprivozn@redhat.com>
Thu, 12 Jan 2012 11:02:40 +0000 (12:02 +0100)
commit833b901cb711a9db148b4fe39e658411f82f3467
tree98366a9829653435b7410011ee11a692ad2eff89
parent74e701f8e55eddcff1e48f0e76444c464369e676
stream: Check for stream EOF

If client stream does not have any data to sink and neither received
EOF, a dummy packet is sent to the daemon signalising client is ready to
sink some data. However, after we added event loop to client a race may
occur:

Thread 1 calls virNetClientStreamRecvPacket and since no data are cached
nor stream has EOF, it decides to send dummy packet to server which will
sent some data in turn. However, during this decision and actual message
exchange with server -

Thread 2 receives last stream data from server. Therefore an EOF is set
on stream and if there is a call waiting (which is not yet) it is woken
up. However, Thread 1 haven't sent anything so far, so there is no call
to be woken up. So this thread sent dummy packet to daemon, which
ignores that as no stream is associated with such packet and therefore
no reply will ever come.

This race causes client to hang indefinitely.
src/rpc/virnetclient.c
src/rpc/virnetclient.h
src/rpc/virnetclientstream.c
src/rpc/virnetclientstream.h