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

Allocate memory to stun_mutex_ptr and stun_signal_ptr

Cache GSocketAddress in UdpBsdSocket

Creating a GSocketAddress takes a measurable amount of time, so let's cache it
as it doesn't change once a socket is created.

Multiple fixes for new test

* Make STUN thread drop STUN packets using a global variable
* Completely refactor the way the STUN thread exits
  - Now exit by setting exit_stun_thread as TRUE and sending
    dummy data
  - Remove the old signal mechanism
* Don't set unnecessary socket options
* Lock stun_mutex_ptr before signalling STUN response

Fixes to dribble mode test as per suggestions

- Use LEFT_AGENT and RIGHT_AGENT instead of calling GINT_TO_POINTER everytime
- Add padding (0's in our case) before the actual payload and make sure the
  reciever checks for this padding. This ensures no STUN packets got through
- Free GSList's properly in cb_agent_new_candidate
- Use proper calls to g_signal_handlers_disconnect_by_func

Minor bug fixes and enhancements to the test

Update .gitignore for new test

Use %lu for long unsigned int when calling printf

Fix test-dribble

The ragent state can be either NICE_COMPONENT_CONNECTED or NICE_COMPONENT_READY

Update documentation to reflect the dribble mode feature

Added new tests for dribble mode in libnice

4 tests in total :

  * Common to all :
    - Gather lagent and ragent candidates
    - Delay STUN response for lagent
    - Make sure connchecks succeed and data is sent across
      to lagent

  * standard_test :
    - Set remote candidates for lagent while lagent is still
      gathering it's own candidates

  * bad_credentials_test :
    - Set incorrect credentials for both lagent and ragent
    - Make sure connchecks fail
    - Signal STUN response and set correct credentials

  * bad_candidate_test :
    - Set a bad remote candidate for lagent
    - Make sure connchecks fail
    - Signal STUN response and set correct remote candidates

  * new_candidate_test :
    - As soon as a new local candidate is found set it as
      the remote candidate for the other agent

Add conn_check_add_for_local_candidate to add the newly found local candidate to the conncheck list

conn_check_add_for_local_candidate works exactly like conn_check_add_for_candidate
except that it starts connchecks for newly found local candidates with existing
remote candidates

Refactor priv_add_local_candidate_pruned to accept a NiceAgent and a stream_id
in order to call conn_check_add_for_local_candidate for newly gathered local candidate

Dribble Mode : Do not check whether we are still gathering our own candidates

version 0.1.2.1

version 0.1.2

Recognize a matching check if it's an INVALID message (error with no error-code)