diff options
| author | Stefan Hajnoczi | 2013-03-27 10:10:46 +0100 |
|---|---|---|
| committer | Luiz Capitulino | 2013-04-02 17:47:37 +0200 |
| commit | 9b938c7262e403f5467110609cb20ef1ae6e9df2 (patch) | |
| tree | a4d062e324c9eaa8a6d9432af65d1c5e40dd05ff /include | |
| parent | qemu-socket: set passed fd non-blocking in socket_connect() (diff) | |
| download | qemu-9b938c7262e403f5467110609cb20ef1ae6e9df2.tar.gz qemu-9b938c7262e403f5467110609cb20ef1ae6e9df2.tar.xz qemu-9b938c7262e403f5467110609cb20ef1ae6e9df2.zip | |
chardev: clear O_NONBLOCK on SCM_RIGHTS file descriptors
When we receive a file descriptor over a UNIX domain socket the
O_NONBLOCK flag is preserved. Clear the O_NONBLOCK flag and rely on
QEMU file descriptor users like migration, SPICE, VNC, block layer, and
others to set non-blocking only when necessary.
This change ensures we don't accidentally expose O_NONBLOCK in the QMP
API. QMP clients should not need to get the non-blocking state
"correct".
A recent real-world example was when libvirt passed a non-blocking TCP
socket for migration where we expected a blocking socket. The source
QEMU produced a corrupted migration stream since its code did not cope
with non-blocking sockets.
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Luiz Capitulino <lcapitulino@redhat.com>
Diffstat (limited to 'include')
0 files changed, 0 insertions, 0 deletions