tests: Fix strict aliasing of sockaddr structures

Casting from one struct sockaddr type to another breaks C’s strict
aliasing rules (variables of different types cannot alias). Fix this
cleanly by using unions of struct sockaddrs to convert between the
types (i.e. type-punning).

I wish sockaddr didn’t have to be this painful.

See:
http://gcc.gnu.org/onlinedocs/gcc-4.4.1/gcc/Optimize-Options.html#Type_002dpunning

stun: Make the MD5 code strict-aliasing correct

build: Update .gitignore file

pseudotcp: Add methods to query the state of the send and recv buffers

pseudotcp: Implement window scaling for PseudoTCP.

Syncs with libjingle SVN rev 77

tests: Add more asserts to pseudotcp test

tests: Use right type when taking to stdio f*()  C APIs

tests: Close pseudosocket as soon as it's done sending

tests: Fix racy destruction of pseudotcp object

tests: No need to call adjust_clock in readable

This can only be called after we received a packet, where we already
adjust the clock

tests: Use one struct for data and buffer

pseudotcp: Only consolidate recv buffer if it's getting buffer

Should improve receive performance a bit.

Syncs with libjingle SVN rev 76

pseudotcp: Make it possible to disable the Nagle Algorithm

Syncs with libjingle SVN rev 56

agent: Don't make relayed sockets available in nice_agent_get_selected_socket()

Also document that fact.

agent: Note that nice_agent_set_relay_info() can be called many times

Add to the documentation to note that it can be called several times to
set details for several relay servers during discovery.

build: Don’t unconditionally enable -Werror for non-release builds

Do not unconditionally enable -Werror, since that breaks the build for
people who have stricter compilation warnings enabled.

Instead, add an --enable-compile-warnings configure flag which enables
-Werror when passed as --enable-compile-warnings=error. This mimics the
flag in gnome-common, for consistency. But we enable errors by default
on non-released versions.

agent: Read all socket data in nice_agent_g_source_cb

stun: Add printf function attribute

This shuts a compiler warning up and allows for format string checking
of debug messages.

stun: Fix format specifier for a size_t variable

This fix may not be entirely cross-platform, and I have been unable to
test whether it is; making it cross-platform is made more difficult by
the fact that the STUN code doesn’t use GLib.

The PRIuPTR macro was defined in POSIX:2004, and later in C99.
http://pubs.opengroup.org/onlinepubs/009696799/basedefs/inttypes.h.html

build: Update .gitignore file

build: Remove auto-generated INSTALL file

It gets automatically generated by autotools, so just gets in the way if
checked into git.

configure: Verify that the compiler understands warning cflags

Older compilers don't understand all of the flags

agent: Add warning on incorrect API usage

stun: Indications are never authentication when using long term auth

turn: Send queued TURN messages to the turn server

It was trying to send queued messages to the final destination instead
of going through the TURN server

conncheck: Avoid reading null username

examples: Remove g_type_init() call for recent GLib versions

stun: Fix a use of a function with an aggregate return value

div() has an aggregate return, which GCC doesn’t like, although this
seems like a pretty pointless warning because div_t is the same size as
a pointer on 64-bit platforms (probably) and hardly going to cause
performance problems by passing it by value.

Anyway, it seems easier to simplify the code by using explicit / and %
operators inline, than it does to add pragmas and shut the warning up.

stun: Explicitly avoid a memcpy() from NULL

If stun_message_append_bytes() is called through
stun_message_append_flag(), data will be NULL and len will be 0. This
will result in a memcpy(ptr, NULL, 0) call. This probably won’t do any
harm (since any reasonable memcpy() implementation will immediately
return if (len == 0)), but the standard allows for memcpy() to explode
if (data == NULL), regardless of the value of len.

In order to be conformant, and to shut up the scan-build static analysis
warning about it, only do the memcpy() if (len > 0).

stun: Remove unused variables

agent: Avoid memcpy()ing garbage between username strings

If (username_len > uname_len), memcpy() would fall off the end of the
uname string and copy (username_len - uname_len) bytes of garbage into
username.

Theoretical bug: not actually attempted to be demonstrated in the wild.

agent: Fix an old-style function declaration

agent: Avoid a potential NULL pointer dereference

password could be NULL (priv_get_password() can return NULL), but
g_base64_decode() aborts if it receives NULL for the encoded text.

agent: Fix format specifiers in debug messages

Mostly problems with the specifier for gsize.

agent: Add missing format specifiers to debug messages

agent: Add printf function attribute

This shuts a compiler warning up and allows for format string checking
of debug messages. (See future commits for the fixes this has resulted
in.)

Fix strict aliasing of sockaddr structures

Casting from one struct sockaddr type to another breaks C’s strict
aliasing rules (variables of different types cannot alias). Fix this
cleanly by using unions of struct sockaddrs to convert between the
types (i.e. type-punning).

I wish sockaddr didn’t have to be this painful.

See:
http://gcc.gnu.org/onlinedocs/gcc-4.4.1/gcc/Optimize-Options.html#Type_002dpunning

Add missing ‘default’ cases to switches

This shuts GCC’s -Wswitch-default warning, and makes the code flow a
little more explicit.

This introduces no functional changes.

examples: Remove redundant non-NULL checks

The return value of g_strsplit() can never be NULL.

Fix variable shadowing

build: Update .gitignore file

build: Delete ar-lib in maintainer-clean build target

build: Update AC_CONFIG_HEADERS instead of AC_CONFIG_HEADER

The latter is deprecated.

agent: Clarify valid range of stream and component IDs

Valid IDs will never be 0, which is convenient.

agent: Add API to retrieve the selected GSocket

This should be used only when the protocol has it's own keepalive mechanism

agent: Getting the local candidates returns a copy, fix the doc

Add special case to make it compile on Solaris

Silence aclocal warning by changing the order of the macros

build: Add GUPnP libraries to Libs.private in pkg-config file

They’re private dependencies which are needed on the linker command line when
linking statically against libnice.

https://bugs.freedesktop.org/show_bug.cgi?id=70965

build: Update configure.ac according to autoupdate

https://bugs.freedesktop.org/show_bug.cgi?id=70965

build: Add AM_PROG_AR to configure.ac

This shuts up an automake warning about needing it when using a non-POSIX
archiver.

https://bugs.freedesktop.org/show_bug.cgi?id=70965

build: Add ‘-I m4’ to ACLOCAL_AMFLAGS

This shuts up an automake warning and ensures that in-tree m4 macros are found
by aclocal.

https://bugs.freedesktop.org/show_bug.cgi?id=70965

autogen.sh: Honor NOCONFIGURE=1

This is adapted from telepathy-logger's autogen.sh.  See
http://people.gnome.org/~walters/docs/build-api.txt
for rationale behind NOCONFIGURE=1.

Fix unitialized variable in nice_agent_parse_remote_candidate_sdp

Refresh TURN CreatePermission at receiving data in case TURN path is asymetrical.

Set the minimum GLib version to 2.30 to disable deprecated warnings on newer GLib versions and make sure the maximum allowed GLib version is 2.32

Also actually require 2.30

Open socket with random port. Use random port to avoid TURN error 437 Allocation Mismatch

win32: Add new 0.1.4 symbols to the win32 .def file as well

version 0.1.4.1

Version 0.1.4

Fix documentation

Add build instructions to the README

Remove debugging print

examples: Make input non blocking to allow exit when remote hangs up

Do not unref the mainloop until the thread has joined since it could be still using it

Export the new nice_agent_get_selected_pair API

Do not unref the GThread since g_thread_join takes the reference. Also rename the variable.

Change the m= line in the SDP to use ICE/SDP dummy type instead of RTP/AVP

Add new parsing API, for stream and candidates

Add example that uses the new SDP parsing API

Adding new SDP parsing and generation API.
This adds nice_agent_set_stream_name, nice_agent_get_stream_name,
nice_agent_generate_local_sdp, nice_agent_parse_remote_sdp.

Fix/clean threaded-example

Refactor nice_agent_set_stream_tos to use agent_find_stream internal API

Add a similar simple example using threads

example: Exit on Ctrl-D and send EOS signal to the other end

Change license for simple-example

Add a simple example to showcase the API thanks to Bryce Allen

Cleanup documentation

Add nice_agent_get_selected_pair API to NiceAgent

Add nice_address_ip_version API to NiceAddress

Fix username validation in OC2007 mode

Wrong 'inbound' argument was passed into priv_validate_username()

Add win32 directory to EXTRA_DIST

Add nice_agent_set_port_range to the list of exported API functions

Manage multiple TURN CreatePermission in parallel

agent: display NiceComponentState as strings in debug message

https://bugs.freedesktop.org/show_bug.cgi?id=49219

Fix C preprocessor bug

Fix a minor bug in the dribble mode test

If for some reason the opening of the socket is delayed, or simply takes
too much time, the test will fail. This patch ensures that the test will
only proceed once the socket is open.

version 0.1.3.1

version 0.1.3

Compile the plugin for both GStreamer 0.10 and 1.0

Revert this patch to drop GStreamer 0.10 support

nicesrc: Use gst_buffer_fill

The plugin name is not a string anymore

Check gstreamer 1.0 not 0.11

GST_FLOW_WRONG_STATE -> GST_FLOW_FLUSHING

Use GstPushSrc as a base class

Port GStreamer src/sink to 0.11

Fix gaddr initialization

Only define _FORTIFY_SOURCE if optimisation is enabled

Merge remote-tracking branch 'shadeslayer/dribble_mode'

Also free the GCond and the GMutex