introspection: add missing allow-none annotations

https://bugzilla.gnome.org/show_bug.cgi?id=730952

introspection: add (nullable) annotations to return values

https://bugzilla.gnome.org/show_bug.cgi?id=730952

gi: improve annotations

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=730953

signals: use generic marshal function

Use the generic C marshal function.
Use more explicit type instead of G_TYPE_POINTER

context: add type macro

sdp: hide key length defines

They don't have a namespace.

Back to development

Release 1.3.3

mikey: add different key length parameters

Add encryption and authentication key length parameters to MIKEY. For
the encoders, the key lengths are obtained from the cipher and auth
algorithms set in the caps. For the decoders, they are obtained while
parsing the key management from the client.

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=730472

stream tests: Make sure we get right multicast address from stream

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=731577

client: ref the context until rtsp watch is alive

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=731569

client: Destroy the rtsp watch after connection close

media: fix confusing comment

rtsp-session: Timeout in header.

Adding the possbilty to always have timout in header.
This is configurabe with setting "timeout-always-visible".

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=728264

Back to development

Release 1.3.2

Automatic update of common submodule

From 211fa5f to 1f5d3c3

client: store TCP ports in transport

Store the TCP ports in the transport when we are doing RTSP over TCP.
This way, we can easily get to the ports from the transport.

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=729776

stream: add signals for new RTP/RTCP encoders

New signals to allow the user to configure the dynamically created
encoders.

https://bugzilla.gnome.org/show_bug.cgi?id=730228

media: Make suspend()/unsuspend() virtual

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=730109

client: fix send-message signal marshaller

Use generic marshalling for the send-message signal. It has
two POINTER arguments, not just one.

https://bugzilla.gnome.org/show_bug.cgi?id=729900

tests: add and remove pads only once

In this test we simulate a dynamic pad by watching the caps event.
Because of renegotiation in the base payloader now, this caps is sent
multiple times but we can only deal with 1 invocation, use a variable to
only 'add and remove' the pad once.

tests: add unit test for correct handling of Require headers

https://bugzilla.gnome.org/show_bug.cgi?id=729426

rtsp-client: handle Require headers and respond with OPTION_NOT_SUPPORTED

Servers must handle Require headers and must report a failure
if they don't handle any of the Required options, see RFC 2326,
section 12.32: https://tools.ietf.org/html/rfc2326#page-54

https://bugzilla.gnome.org/show_bug.cgi?id=729426

Back to development

Release 1.3.1

Automatic update of common submodule

From bcb1518 to 211fa5f

Update .gitignore

tests: fix memory leak in sessionmedia unit test

client: emit a signal before sending a message

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=728970

client: pass context to send_message

Pass the current context to send_message, we will need it later.

client: fix typo in comment

media: Do not stop thread twice if default_prepare() fails

client: set the watch to flushing before going to NULL

First set the watch to flushing so that we unblock any current and
future attempt to send data on the watch, Then set the pipeline to
NULL.

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=728153

rtsp-session-pool: Fixes annotation

Fixes annotation for gst_rtsp_session_pool_create() and memory leaks
in the sessionpool test.

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=728060

media: make media_prepare virtual

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=728029

media: stop the thread in more error cases

media: allow NULL as the thread

Use the default context whan passing a NULL thread.

rtsp-client: indent cleanup

Coverity was moaning about unreachable code, and I think it was just
confused by { being before the label. We'll see if it pops up again.

Coverity 1197705

client: Add drop-backlog property

When we have too many messages queued for a client (currently hardcoded
to 100) we overflow and drop the messages. Add a drop-backlog property
to control this behaviour. Setting this property to FALSE will retry
to send the messages to the client by waiting for more room in the
backlog.

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=725898

client: support for POST before GET when setting up a tunnel

client: remove watch of the second client after http tunnel setup

The second client will be freed after the HTTP tunnel has been set up.
Make sure it's RTSP watch is never dispatched again.

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=727488

media: Make media_prepare() fail if port allocation fails

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=727376

media test: cleanup the thread pool in tests

rtsp-media: Unblock blocked streams in unprepare

The streams will be blocked when a live media is prepared.
The streams should be unblocked in gst_rtsp_media_unprepare.

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=727231

media: release the state lock when going to NULL

Set our state to UNPREPARING and release the state-lock before
setting the pipeline to the NULL state. This way, any pad-added
callback will be able to take the state-lock and check that we are now
unpreparing instead of deadlocking.

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=727102

media: protect status with lock

Make sure we only update the status with the lock.

rtsp: update for MIKEY API changes

client: parse the mikey response from the client

Parse the mikey response from the client and update the policy for
each SSRC.

stream: add method to set crypto info

Make a method to configure the crypto information of a stream.
Set udpsrc in READY instead of PAUSED so that we can configure caps
later.

client: cleanup error paths

media: fix docs

test: enable SRTP only on RTSPS

We only want to enable SRTP when doing rtsp over TLS so that we can
exchange the keys in a secure way.

test: print an error on failure

stream: add SRTP support

Install srtp encoder and decoder elements in rtpbin
Add MIKEY in SDP

tests: Add unit tests for sessionpool

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=726470

tests: Improve code coverage of rtsp-threadpool tests

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=726873

tests: Improve code coverage for rtsp-session-media

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=726940

gobject-introspection: Add annotations to support language bindings

In addition a few cosmetic changes:

 * Adjust the order of arguments
 * Fix typo: occured -> occurred
 * Fix indentation after Return:-clauses

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=726941

rtsp-stream: Don't mix IPv4 and IPv6 addresses

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=726362

stream: take caps after the session manager

Take the caps for the SDP after they leave the rtpbin so that we can
also get the properties added by rtpbin elements.

stream: release lock while pushing out packets

Keep a cache of the transports and use this to iterate the transport
while pushing packets. This allows us to release the lock early.

See https://bugzilla.gnome.org/show_bug.cgi?id=725898

rtsp-client: vmethod for modifying tunnel GET response

Add a vmethod tunnel_http_response where the response to the HTTP GET
for tunneled connections can be modified.

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=725879

sdp: make 1 media line per profile

If we have multiple profiles (AVP or AVPF) for a stream, make one m=
line in the SDP for each profile. The client is then supposed to pick
one of the profiles in the SETUP request. Because the m= lines have the
same pt, the client also knows that only 1 option is possible.

factory: add profile property and pass to media and streams

sdp: pass multicast connection for multicast-only stream

Pass the multicast address of the stream in the connection info in the
SDP so that clients try a multicast connection first.
Only allow multicast connections in the test-multicast example. Also
increase the TTL a little.

.gitignore: Ignore gcov intermediate files

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=725484

stream: release some locks in error cases

docs: Enable and fix gtk-doc warnings

 * Makefile: Enable gtk-doc warnings, like the rest of GStreamer
 * addresspool/mediafactory: Add missing annotation colon
 * stream: Annotate return value

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=725528

Automatic update of common submodule

From fe1672e to bcb1518

Automatic update of common submodule

From 1a07da9 to fe1672e

examples: use LDADD for libs instead of LDFLAGS

configure: make sure releases are in .doap file

examples: test-cgroups: don't put code with side effects into g_assert()

The g_assert() might get compiled out with the right
compiler/preprocessor flags.

examples: add cgroup test binary to .gitignore

examples: fix cgroup test build

Fixes build failure caused by compiler warning:
test-cgroups.c:82:35: error: no previous prototype for ‘gst_rtsp_cgroup_pool_get_type’ [-Werror=missing-prototypes]

.gitignore: ignore temp files created in the course of 'make check'

rtsp-media: don't loose frames handling new PLAY request

If client supplied a range check if the range specifies the start point.
If not, then do an accurate seek to the current position. If a start
point was specified do do a key unit seek to make sure the streaming
starts with decodeable frames.

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=724611

Revert "media: only flush when setting a new start position"

This reverts commit f67fc23aab59f28796bebf130504ff46ccb97b0a.

We need to do the flush in all cases, demuxer block currently for
non-flushing seeks.

media: only flush when setting a new start position

Only flush the pipeline when we change the start position with
a seek.

See https://bugzilla.gnome.org/show_bug.cgi?id=724611

stream: set ttl-mc before adding the socket

Set ttl-mc before adding the socket. Otherwise the value ttl-mc will
never be set on socket.

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=724531

media: stop thread if media is already prepared

in gst_rtsp_media_prepare() the thread is not used if media is already
prepared (e.g. media shared) so we want to stop the thread. otherwise, a
leak occurs.

https://bugzilla.gnome.org/show_bug.cgi?id=724182

build: Ship gst-rtsp-server.doap file

tests: Fix another compiler warning with gcc

rtsp-server: Fix lots of compiler warnings with clang

configure: Synchronise with the configure scripts of the other modules

configure: Update version to 1.3.0.1 and require GStreamer 1.3.0

Revert "rtsp-server: support build against last stable release"

This reverts commit 099a10f61f11413ad0ada8ee0b7b7ad1210b1b2f.

Let us require 1.2.3 now, which is going to be released in a few
minutes.

session: improve RTP-Info

Ignore streams that can't generate RTP-Info instead of failing.
Don't return the empty string when all streams are unconfigured but
return NULL so that we don't generate and empty RTP-Info header.
Improve docs a little.

Don't free rtpinfo GString when it is NULL

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=723554

media: only set keyframe flag when modifying start

Only set the keyframe flag when we modify the start position. The
keyframe flag should probably be ignored when no change is requested but
until we can claim this is all documented properly and all demuxer
implement this, avoid setting the flag.

See also https://bugzilla.gnome.org/show_bug.cgi?id=723075

thread-pool: Unref source after mainloop has quit to avoid races in GLib

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=723741

stream: handle NULL seqnum and rtptime arguments

thread-pool: Unref reused threads in gst_rtsp_thread_stop()

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=723519

stream: add fallback for missing stats property

Use a fallback when the payloader does not have a stats property

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=723554

Automatic update of common submodule

From f7bc1c3 to 1a07da9

stream: don't leak stats structure

Don't leak the stats structure and deal with NULL stats.

stream: Get rtpinfo properties atomically from payloader

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=722844

media: refactor state change functions and signals

Make functions to set the target state and the pipeline state and emit
the signals from those functions.

media: add signal to notify of pending state changes