threaded-example: Make sure the foundation string is NULL terminated

Version 0.1.6.1

Version 0.1.6

Fix udp-turn returning wrong message length in socket_recv_messages.

When using optimal path where recv message has only one buffer, the
buffer length was set to the size of the buffer, not its length, which was
causing the wrong message length to be returned causing the agent to
discard STUN messages because their size doesn't match received data
length.

agent: Add a missing forward declaration

agent: Simplify gathering-done stage of UPnP handling

This introduces no functional changes, but consolidates and documents
the code a bit more.

agent: Tidy up upnp_mapping handling

Use a NULL check to determine if upnp_mapping is an empty list (rather
than counting every element). Use g_slist_free_full() where appropriate.

This introduces no functional changes.

agent: Document units for NiceAgent::upnp-timeout property

It’s in milliseconds.

stunagent: Only declare the long term key valid if the username/realm coudl be read

example: Prevent possible non-NULL terminated string

address: Correctly identify ipv6 link local

The test for fe80::/10 always returned FALSE.

add nice_agent_forget_relays()

component: Clear turn local candidates when clearing turn servers

But keep the turn connection that's currently being used if it is
the selected candidate. Also clear the TURN candidate refresh.

agent: Make the TurnServer structure refcounted

Instead of just blindling assuming that all of the pointers are valid
until the end.

discovery: Use g_slist_free_full for candidate refresh list

discovery: Use g_slist_free_full for discovery list

discovery: Remove never used and leaked msn_turn_* fields in CandidateDiscovery

discovery: Remove never used and leaked msn_turn_* fields in CandidateRefresh

agent: Factor out creating the TurnServer structure

discovery: Remove duplicated element between CandidateRefresh and NiceCandidate

discovery: Store NiceCandidate in CandidateRefresh

component: Make detach_source close the socket

component: Make is possible to remove a socket

Change the SocketSource handling to make it possible
to remove a source, not only to add one.

factor out: component_clean_turn_servers()

stream: No need to pass RNG to stream_restart()

It can now take it from the agent.

conncheck: Only print debug message if actually freeing something

conncheck: Don't fail a stream that has on-going discovery

agent: Add nice_agent_restart_stream() to restart a single stream

stream: Clear connchecks on restart inside stream

conncheck: Use g_slist_free_full() for conncheck list

Skips one iterator over g_slist_foreach() + g_slist_free(). It is also easier
to read.

conncheck: Make conn_check_free_item() static

It is never used outside of this file.

conncheck: Only stop conncheck timer if no stream has pending conn checks

conncheck: Remove useless return value on conn_check_prune_stream

conncheck: Factor out stopping connectivity checks

component: Remove return value from *_restart() functions, they can't fail

component: socket has been renamed nicesock

Add more debug info when adding relays

Protect test-dribble against read->connected->ready state changes

Return the correct originating IP on TURN

If doing a recv on UDP-TURN, the 'from' address was kept set to
the turn server which causes connchecks to fail with "mismatched address"
error. This fixes it

add nice_component_state_to_string()

Nice to make debug logs more friendly to read.

pseudotcp: Fix partial acknowledgement of segments

If an incoming ACK acknowledges part of a pending SSegment, correctly
increment the base offset of that SSegment before attempting to transmit
it. Otherwise, the wrong data will be transmitted and the offset passed
to packet() will be incorrect.

pseudotcp: Fix an assertion on an invalid calculated RTT

If a packet has an invalid tsecr field, invalid (e.g. negative) RTTs may
be calculated from it. If so, discard the packet as invalid.

tests: Add a test-pseudotcp-fuzzy test for fuzzing PseudoTcpSocket

As documented at the top of the file, this creates two PseudoTcpSockets
in a loopback configuration, and mutates the raw packets which are sent
between them, hoping to cause assertion failures and illegal memory
accesses in the pseudotcp code.

Best run under Valgrind or GDB.

agent: Fix leaks if a component is removed during a recv() call

If the error handling block in the middle of the recv() loop is hit,
various things were previously leaked (context, cancellable_source) and
the return values were potentially incorrect.

Fix the break out of the error handling block to clean up correctly.

stun: Restore argument type to stun_usage_bind_process(), it's used outside libnice

agent: Factor out a tiny bit of common code

agent: Limit printed length of uname and password in debug output

At this point in the code, neither the username nor password are
nul-terminated, so printing them in the debug output as nul-terminated
strings can easily result in rubbish being printed after them.

Fix this by specifying the string length in the printf() format string.
Finally my terminal will no longer ask me to install N’Ko fonts when
running the libnice tests.

agent: Fix a leak in nice_output_stream_write() with a NULL cancellable

If @cancellable is NULL in a call to nice_output_stream_write(), the
WriteData struct is created with a reference count of 4, but only two
operations are scheduled which will result in its reference count being
decremented. The third operation is only scheduled if @cancellable is
non-NULL (and the final reference is dropped unconditionally at the end
of the function).

Fix this by properly implementing reference counting for WriteData,
rather than hard-coding the expected number of references in a fragile
and unmaintainable way.

tcp-bsd: Don't clear error that hasn't been set

agent: Don't return the socket to the app in any reliable case

No need to strdup foundation in new_selected_pair since we queue signal

pseudotcp: Also cause Writable callback to be emitted if can_send() says no

The GOutputStream based systems can sometimes call this, decide the stream is
not writable and then poll for it. We have to make sure that poll is woken up.

Fix TCP TURN for Google compatibility. Message length was sent uninitialized

Small cleaning of tcp-bsd.c

Fix possible segfault in stun_message_validate_buffer_length()

build: Check for [s]size_t before redefining them on MinGW

MinGW defines size_t and ssize_t for us, so we should not
unconditionally redefine them in stun/win32_common.h. Add an
AC_CHECK_TYPES configure check to avoid this.

agent: Define IN6_ARE_ADDR_EQUAL macro when building under MinGW

Older versions of MinGW don’t define IN6_ARE_ADDR_EQUAL, as normally
defined in netinet/in.h. Conditionally define it in address.c to fix the
build on MinGW.

MinGW has added support for the macro here:
https://github.com/mirror/mingw-w64/commit/0f4899473c4ba2e34fa447b1931a04e38c1f105e

Based on a patch originally by Luciana Fujii Pontello
<luciana@fujii.eti.br>.

pseudotcp: Validate option lengths

Remove all traces of compile time marshallers

agent: Use generic marshaller

Initialize variables to reduce unexpected behavior in case something goes wrong

Rename variables that shadow global definitions

stun: Use struct sockaddr_storage * for any pointer that we write to

Some platforms have a larger alignment requirement for struct sockaddr_* than
for plain struct sockaddr, in that case, we need to make sure that the incoming pointer
has the right alignement before taking it in.

stun tests: Fix a bunch of issues found by more aggressive GCC warnings

Don't mix gsize vs size_t, seems like they're not always the same size

candidate: Use explicit 64-bit constants to make 32-bit gcc happy

pseudotcp: Don't mix "long" and guint, one is signed, the other is unsigned

So use guint64 everywhere, this is never supposed to be negative.

tcp-turn: Don't access output message after freeing it

turn: Don't overwrite source, destroy it first

turn: Don't access list node after freeing it

component: Set dummy callback from cancellable GSource

agent: Make it possible to set the relays after gathering

This allows finding the relays asynchronously while trying to connect
with other methods.

agent: Prevent nice_agent_set_port_range() from being called after a gather

The port range is only used when gathering, print a critical if it is
called afterwards as it will have no effect.

agent: Prevent gather from being called multiple times

This can cause odd behaviors as it is not a idempotent function.

agent: Refactor nice_agent_set_relay_info()

No behavior change is introduced

turn: Put timeouts on the specific main context

turn: Don't try to process empty messages

agent: Bail out early on empty message

This can happen if a lower layer has consumed the message, bail out earlier in
that case.

agent: Don't overwrite the size of the GInputBuffer, only set the global size

pseudotcp: Also accept NiceInputMessage array with a single message

test-fullmode: Don't use gmainloop when not required

There are more than one callback that could stop it

test-fullmode: Do #if instead of #ifdef as we set it to 0

test-fullmode: Only wait for data on component 1

tcp-bsd: Use default main context if none specified

turn-tcp: Don't corrupt messages

agent: Free every parameter

Fixes off-by-one error

agent: Create a custom pointer GType to recognize the stream ids

agent: Improve documentation for NiceAgent::new-candidate

Note that it’s a new _local_ candidate.

NEWS: Remove superfluous empty lines

Version 0.1.5.1

Version 0.1.5

configure: Remove -Wunsafe-loop-optimizations, it causes false warnings

tests: Add private header to tarball

tests: Don't verify the message content with not in reliable mode, because its unreliable

agent: Enforce limits on the size buffers, because the retval is signed

doc: Update doc generation

iostream: De-gtkdoc the comments, as it's not public API

agent: Remove unused #include

net/if_arp.h doesn’t exist on OS X, and hence causes cross-compilation
to fail. It doesn’t appear to be needed for any of the functions
currently used in interfaces.c anyway.

agent: Ensure the pending signal queue is cleared on dispose

agent: Fix a potential re-entrancy situation

If we emit signals from component_io_cb() in the middle of a read, it’s
possible that one of those signals will be picked up by the client to
perform another read. A likely candidate (and what was triggering the
re-entrancy here) would be component-state-changed. Since signals are
emitted synchronously, the second read would being inside the first, and
trigger the anti-re-entrancy assertion.