diff options
| author | Dan Winship <danw@gnome.org> | 2011-08-27 09:59:02 -0400 |
|---|---|---|
| committer | Dan Winship <danw@gnome.org> | 2011-08-27 12:28:03 -0400 |
| commit | cef679d004b9fe89ae08519861e235818dd57b46 (patch) | |
| tree | 9fc4baa0ae0827f3b0c04f4031fa6d2381e1a45c /gio/gsocket.c | |
| parent | 60f23ecbbc7d65eb5188875dc2c50a5f793a82a4 (diff) | |
GSocket: fix GIOCondition on timed-out socket operation
The docs for g_socket_set_timeout() claimed that if an async operation
timed out, the GIOCondition passed to the source callback would be
G_IO_IN or G_IO_OUT (thus prompting the caller to call
g_socket_receive/send and get a G_IO_ERROR_TIMED_OUT), but in fact it
ended up being 0, and gio/tests/socket.c was erroneously testing for
that instead of the correct value. Fix this.
Diffstat (limited to 'gio/gsocket.c')
| -rw-r--r-- | gio/gsocket.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/gio/gsocket.c b/gio/gsocket.c index f1ba5dd25..77d2d2c81 100644 --- a/gio/gsocket.c +++ b/gio/gsocket.c @@ -2439,7 +2439,6 @@ socket_source_prepare (GSource *source, if (*timeout < 0) { socket_source->socket->priv->timed_out = TRUE; - socket_source->pollfd.revents = socket_source->condition & (G_IO_IN | G_IO_OUT); *timeout = 0; return TRUE; } @@ -2476,6 +2475,8 @@ socket_source_dispatch (GSource *source, #ifdef G_OS_WIN32 socket_source->pollfd.revents = update_condition (socket_source->socket); #endif + if (socket_source->socket->priv->timed_out) + socket_source->pollfd.revents |= socket_source->condition & (G_IO_IN | G_IO_OUT); return (*func) (socket_source->socket, socket_source->pollfd.revents & socket_source->condition, |