conncheck: set writable callback to socket from TCP active connect
authorJakub Adam <jakub.adam@ktknet.cz>
Thu, 18 Jun 2015 07:05:21 +0000 (09:05 +0200)
committerPhilip Withnall <philip.withnall@collabora.co.uk>
Fri, 3 Jul 2015 13:26:05 +0000 (14:26 +0100)
commite1f748cccacd81cce4db338d0203dc49bc915a30
tree057cf7639bde5a2eeef9463a21d9f3b23a458dbd
parentcd61af3114a51df53619fb0460ace2b3660fc5da
conncheck: set writable callback to socket from TCP active connect

A new socket created in nice_tcp_active_socket_connect() should have its
writable callback set, because it's possible for it to become a base
socket of a peer reflexive candidate, if some is discovered by
connection checks on that TCP active candidate.

Previously, when such prflx candidate became selected, without write_cb
on the socket the agent was never notified about it becoming writable
again after the socket's buffer got filled up. This caused the data flow
to hang permanently.

Reviewed-by: Philip Withnall <philip.withnall@collabora.co.uk>
Reviewed-by: Olivier CrĂȘte <olivier.crete@collabora.com>
http://phabricator.freedesktop.org/T117
agent/agent-priv.h
agent/agent.c
agent/conncheck.c