agent: Don’t clear the PseudoTcpSocket on error

Instead, keep the closed object around. This allows differentiation
between the states: TCP support was never initialised; and TCP support
was initialised but is now closed.

pseudotcp: Fix a typo in some documentation

pseudotcp: Factor out state change functions for ESTABLISHED and CLOSED

Both state changes need some follow-up code executed immediately after,
which was duplicated in a number of places. Factor that out. This
introduces no behavioural changes.

pseudotcp: Split out some state checks

This clarifies the code a little, and does not introduce functional
changes.

pseudotcp: Tidy up buffer size handling in queue_connect_message()

This removes some hard-coded offsets. It does not introduce any
functional changes.

pseudotcp: Store segment flags in SSegment

Rather than reconstructing the flag state when sending a segment, just
store the flags explicitly. This does not introduce any behavioural
changes.

pseudotcp: Allow the ‘current’ time to be set

This is needed for the upcoming new test suite for pseudo-TCP. It
shouldn’t be used in normal code — only in tests.

Ideally, the pseudo-TCP code should originally never have called
g_get_monotonic_time() itself, and should have always taken a time
parameter from the caller; then it would be more testable.
Unfortunately, API guarantees prevent this from being changed now.

pseudotcp: Add socket state to debug messages

This should make debugging a little clearer.

pseudotcp: Put TCP flags in an enum

The on-the-wire flags (FIN, RST, ACK, etc.) should be in an enum to
clarify the code a little. This introduces no functional changes.

pseudotcp: Put TCP options in an enum

This tidies things up a little. No functional changes.

tests: Improve debug output of pseudotcp tests slightly

tests: Add setlocale() call to test-pseudotcp.c

This ensures UTF-8 output is printed correctly.

tests: Add a logging domain for the unit tests

This clarifies the log output a little.

socket: Gracefully return from send() if the socket is closed

agent: Ensure Component.selected_pair is cleaned when freeing candidates

When freeing candidates (component_free_socket_sources()), the sockets
which back Component.selected_pair.[local|remote] are closed and their
addresses destroyed. Component.selected_pair should be cleared as well
to allow calling code to condition on (Component.selected_pair == NULL)
to see if it’s invalid.

agent: Factor out common clean up code in Component

This introduces no functional changes.

agent: Tidy up a few variable references

This simplifies the code a little. No functional changes.

agent: Rearchitect message handling to use GErrors for EWOULDBLOCK

Previously, an EWOULDBLOCK return value from the low-level socket calls
(including PseudoTcpSocket) would be represented by a zero number of
bytes (or messages) read by the agent. This conflicts with the use of
zero to represent end of stream (EOS) for pseudo-TCP connections, where
the sender has indicated that they are not going to send any more bytes.

So, now use GError (G_IO_ERROR_WOULD_BLOCK) to represent EWOULDBLOCK,
just like the GSocket functions. Zero is reserved exclusively for if:
 • the number of requested bytes/messages is zero; or
 • reliable mode is enabled and EOS is reached.

This does change the documented behaviour of the NiceAgent send/recv
API, but only by allowing a new behaviour (returning zero) rather than
by changing an existing one, so it should be OK.

pseudotcp: Fix the type of an argument to apply_option()

We’re dealing with unsigned bytes here, not chars. This will become
important when adding new TcpOptions numbered from 254 downwards.

pseudotcp: Add pseudo_tcp_socket_is_closed_remotely()

New convenience API to check if the remote end of a pseudo-TCP socket
has been closed (but not necessarily the local end). This is currently a
stub implementation, but will be used fully once pseudo-TCP FIN–ACK
support lands.

pseudotcp: Add pseudo_tcp_socket_is_closed()

New convenience API to check if the socket is in state TCP_CLOSED.

pseudotcp: Factor out state changes and validate them

By validating state changes, we can verify that the implementation
conforms to the RFC 793 state machine at runtime. This will become more
important when FIN–ACK support is added, adding more states.

agent: Factor out peer reflexive priority calculation

agent: Add "keepalive-conncheck" property to make keepalives be conncheck

This is to use keepalives to check if the connection is still ongoing. If
they don't get a reply, then we can assume that the connection has died.

This needs to not happen in the case of ICE-TCP.

nicesink: Check for signal present before removing it

The signal is only connected when setting an agent, if no agent
has been set, there will be no signal.

stun: Fix definition of ssize_t on Windows

ssize_t should definitely be signed, not unsigned.

build: Add README.win32 to EXTRA_DIST

As spotted by B Andrei on the mailing list.

socket: Remove dead code

At that point, (len > 0), since there are conditions above handling the
(len == 0) and (len < 0) cases, both of which break from the loop.

Coverity issue: #29325

nicesink: Block until component is writable if in reliable mode

Fix bug in the ice-tcp unit test

conncheck: Go to connected only if the component was connected or ready

Otherwise, it claimed it was connected when the component was in the FAILED
state, when it should really have gone to connecting.

outputstream: Fix ref leak

component: Fix ref leak

agent: Expand documentation for nice_interfaces_get_ip_for_interface()

Set candidate pairs from discovered peer-reflx tcp-passive candidates into DISCOVERED state so it doesn't fail if the other side doesn't reply to our binding request

Fix retransmission timers for reliable sockets
The retransmissions would timeout after 600ms which caused the TCP connections
to timeout before it's able to connect. TCP sockets need to timeout
after 8 seconds instead.

Revert "agent: Separate reliability from ice-tcp vs ice-udp"

This reverts commit 18e5dff4f25b12522e857c13d3ef3bdb40212246.

Change test-icetcp to be more robust against ready->connected->ready state changes

Cast the current time for pseudotcp timeout calculation into guint32
The pseudo_tcp_socket_get_next_clock returns the next clock in terms
of g_get_monotonic_clock / 1000, however, it gets cropped to guint32.
When we try to find the interval, by substracting the current time as
a guint64, it will return a negative value, which prevents an integer
overflow by setting the timeout to 0. This causes the notify_clock
timeout to be called all the time by the mainloop, taking 100% of CPU.

agent: Print libnice debug messages when setting G_MESSAGES_DEBUG appropriately

This way all the debug can be controlled from one place.

Also disable -Wsuggest-attribute=format with -Werror because gcc 4.8.3
erroneously suggested it.

pseudotcp: Use different g_log domain for pseudotcp

Also add one for pseudotcp-verbose

debug: Make debug messages ready for g_log()

Send thing one line at a time, so no explicit \n

Also make it possible to set a log handler explicitly

Fix timer for TURN CreatePermission.
The TURN CreatePermission is a list of multiple permissions but the timer
is created and overwrites the old one, so some of them don't get triggered
at the right time.
This patch was suggested by Livio Madaro. We create a timer for the minimal
amount of time, and trigger the retransmissions on the permissions that
timed out, then reschedule for the next retransmissions.

Check error code 438 for TURN Nonce expired

Do not access component if it was destroyed when calling the io callback
When calling component_emit_io_callback, the agent mutex will be unlocked before
the callback is called, this can cause another thread to remove the
stream and thus destroying the component. We must verify that the
source was not destroyed after calling component_emit_io_callback
and avoid accessing the component in that case.
Bug report by Stephan Thamm in the mailing list.

Use sockaddr for stun_usage_bind_process in unit test

agent: Use AC_CHECK_HEADERS to check for netdb.h

Rather than assuming it is not available on Windows.

interfaces: Add missing brackets

agent: Fix infinite loop when the peer closes a TCP connection remotely

tcp-passive: Relay the tcp writable callback up

Otherwise, the caller gets a callback from an unexpected source.

agent: Fix an old-style function definition in the Windows code

agent: Add debugging when parsing addresses

agent: Switch to GetAdapterAddresses() to find local IPs on Windows

The old GetIpAddrTable() function only returned IPv4 addresses. In order
to support IPv6 addresses, we need to use GetAdapterAddresses(), which
is available from Windows XP or Server 2003 onwards.

agent: Use getaddrinfo() to parse addresses from strings

Rather than using inet_pton(), which doesn’t support IPv6 link-local
scope IDs, use getaddrinfo(). This is supported on all platforms
(POSIX.1-2001, and Windows winsock).

agent: Use getnameinfo() instead of inet_ntop() to stringify addresses

inet_ntop() drops IPv6 link-local scope IDs on the floor, which means
libnice doesn’t generate any IPv6 candidates for link-local connections,
meaning that it will theoretically fail to negotiate a connection on an
IPv6-only network where the two peers are local.

Fix that by using getnameinfo() to convert sockaddrs to strings, which
preserves the scope ID.

There are no portability concerns — getnameinfo() is POSIX.1-2001, and
isn’t used on Windows (although it is supported, surprisingly).

outputstream: Always try sending

Otherwise we're never woken up when it's again possible to send

outputstream: Fix reference leak

agent: Factor our writability signal

introspection: Rename gir file to match library version

socket: Avoid g_main_context_ref(NULL) calls using the default context

If (agent->main_context == NULL) the sockets are created with a NULL
main context, which they then try to ref. That does not end well.

examples: Remove unnecessary dynamic type casts

We know the agent is a GObject — there is no need to check it at
runtime. This also improves Tartan’s static analysis of signal
connections.

agent: Add a missing precondition assertion

Spotted by Tartan.

docs: Remove some private symbols from the documentation sections file

agent: Add a missing field documentation comment

agent: Fix typos in documentation

build: Add the lcov directory to .gitignore

agent: Add missing #includes to candidate.h

agent: Mark nice_agent_attach_recv() as (skip)

This removes it from the introspected bindings. It is unintrospectable
because it specifies a callback (@func) which persists after the
nice_agent_attach_recv() call returns. This is only supported by
gobject-introspection if a full closure with GDestroyNotify callback for
the user is passed in too.

agent: Fix a typo in a documentation comment

agent: Add missing (element-type) annotations to gtk-doc comments

These make the GSLists usable from introspection.

agent: Fix badly formatted gtk-doc comments

build: Add GObject introspection support

This bumps our autoconf requirement to 2.62 (from 2.59c) and disables
automake portability warnings.

It does not introduce a hard dependency on gobject-introspection; but if
available, it requires version 1.30.0. Support can be enabled with
--enable-introspection, and is required to make distcheck.

The generated introspection file has not been checked thoroughly, and
may expose internal symbols. A lot of new annotations are needed to make
many of the symbols usable.

nice: Expose UPnP support in nice.pc.in

It might be useful for dependent packages to check whether libnice was
compiled with UPnP support, so expose that as a variable in the
pkg-config file, set to ‘true’ if supported, and the empty string
otherwise.

nice: Auto-substitute pkg-config requirements in nice.pc.in

The requirements in the pkg-config file were outdated and incomplete. It
makes a lot more sense to automatically substitute them in from the
checks in configure.ac.

interfaces: License was meant to be the same as the rest

Require automake 1.12

We use macros from 1.12 anyway.

agent: Separate reliability from ice-tcp vs ice-udp

We want ICE-TCP to not have reliable behavior unless the agent
is reliable otherwise it will confuse existing VoIP applications.

agent: Only deal with pseudotcp if actually using PseudoTCP

PseudoTCP is only used if both the pseudotcp socket exists and the
underlying transport is unreliable.

discovery: Make it less likely that a locally assigned remote foundation clashes with the remote one.

discovery: Compare turn server when assigning foundations

The RFC says that "for reflexive and relayed candidates, the STUN or TURN servers
used to obtain them have the same IP address."

discovery: Use nice_address_equal_no_port()

discovery: Compare tranports (TCP vs UDP) while assigning remote foundations

address: add nice_address_equal_no_port() to compare addresses ignoring ports

agent: Add new-selected-pair-full

This signal include the whole candidate as the foundation is ambiguous.

signal

TODO: Update to match current state

gitignore: Add test-driver from recent automake

agent: Add new-candidate-full and new-remote-candidate-full signals

These signals include the actual candidate instead of just the foundation
which is not unique.

candidate: Add Boxed type

stunmessage: Accept NULL terminated buffers for fast speed validation

discovery: Differentiate socket creation failure from other errors

If the socket can't be bound, then trying another port makes sense,
otherwise it doesn't.

Add TCP server reflexive candidates

For TCP server reflexive candidates, since STUN can't work on TCP
candidates, we will use STUN on UDP or TCP and once we get an
externally mapped address, we will create a server reflexive candidate
for all local TCP candidates that share the same base address, and use
the same mapped port as the base addresse's port.

Signal reliable-transport-writable when ice-tcp socket is writable

Add a nice_socket_can_send and nice_socket_set_writable_callback APIs

Call g_socket_check_connect_result before g_socket_is_connected

If we don't call check_connect_result on an outbound connection,
then is_connected will always return FALSE. That's why we check
both conditions to make sure g_socket_is_connected returns the
correct result, otherwise we end up closing valid connections

Bind to a local random port for tcp-bsd

Change argument names for tcp-bsd

Set local address of TCP-BSD socket

nice_tcp_bsd_socket_new: rename addr -> remote_addr

Add a bytestream-tcp property that is read-only for now