tests: Remove FIXME

The session leak is now fixed, lets remove those FIXME comments.

tests: Test to setup two sessions on one connection

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

tests: Test setup with tcp transport

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

client: Configure transport after creating session media

The default implementation of configure_client_transport() in
rtsp-client uses the session media when it chooses channels for
interleaved traffic.

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

client: Stop caching media in client when doing setup

If the media has been managed by a session media, it should not be
cached in the client any longer. The GstRTSPSessionMedia object is now
responsible for unpreparing the GstRTSPMedia object using
gst_rtsp_media_unprepare(). Unprepare the media when finalizing the
session media.

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

rtsp-stream: unref srtp decoder when leaving bin

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

rtsp-client: mikey memory leaks

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

Automatic update of common submodule

From 84d06cd to 7bb2bce

Parallelise 'make check-valgrind'

Automatic update of common submodule

From a8c8939 to 84d06cd

Automatic update of common submodule

From 36388a1 to a8c8939

rtsp-media: deactivate media when shutting down from paused

This was only done when going directly from playing.

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

rtsp-client: add stream transport to context

We add the stream transport to the context so we can get the configured
client stream transport in the setup request signal.

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

stream: release lock even not all transports have been removed

We don't want to keep the lock even we return FALSE because not all the
transports have been removed. This could lead into a deadlock.

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

rtsp-sdp: Rename clock-base and seqnum-base to timestamp-offset and seqnum-offset

These were renamed in GstRTPBasePayload in 1.0

client: set session media to NULL without the lock

We need to set session medias to NULL without the client lock otherwise
we can end up in a deadlock if another thread is waiting for the lock
and media unprepare is also waiting for that thread to end.

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

rtsp-media: Set state to UNPREPARING in all cases

media: set state to unpreparing when unprepare is initiated

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

rtsp-client: Remove backlog limit while processings requests

If the backlog limit is kept two cases of deadlocks may be
encountered when streaming over TCP. Without the backlog
limit this deadlocks can not happen, at the expence of
memory usage.

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

rtsp-client: do not free main context before rtsp watch

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

tests: Extend unit test timeout to accomodate for valgrind

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

rtsp-*: Treat sending packets to clients as keepalive

As long as gst-rtsp-server can successfully send RTP/RTCP data to
clients then the client must be reading. This change makes the server
timeout the connection if the client stops reading.

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

rtsp-client: Allow backlog to grow while expiring session

Allow the send backlog in the RTSP watch to grow to unlimited size while
attempting to bring the media pipeline to NULL due to a session
expiring.  Without this change the appsink element cannot change state
because it is blocked while rendering data in the new_sample callback.
This callback will block until it has successfully put the data into the
send backlog. There is a chance that the send backlog is full at this
point which means that the callback may block for a long time, possibly
forever. Therefore the media pipeline may also be prevented from
changing state for a long time.

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

rtsp-client: Make old compilers happy

rtsp-client.c:2553:50: error: cast to pointer from integer of different size [-Werror=int-to-pointer-cast]

Just in case that guint8 doesn't fit in a pointer. Just in case ...

client: raise the backlog limits before pausing

We need to raise the backlog limits before pausing the pipeline or else
the appsink might be blocking in the render method in wait_backlog() and
we would deadlock waiting for paused.

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

client: make define for the WATCH_BACKLOG

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

client: simplify session transport handling

link/unlink of the transport in a session was done to keep track of all
TCP transports and to send RTP/RTCP data to the streams. We can simplify
that by putting all the TCP transports in a hashtable indexed with the
channel number.

We also don't need to link/unlink the transports when we pause/resume
the streams. The same effect is already achieved when we pause/play the
media. Indeed, when we pause the media, the transport is removed from
the media and the callbacks will not be called anymore.

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

stream-transport: make method to handle received data

Make a method to handle the data received on a channel. It sends the
data to the stream of the transport on the RTP or RTCP pads based on
the channel number.

test: add example of dumping RTCP reports

rtsp-media: Make sure that sequence numbers are monotonic after pause

The sequence number is not monotonic for RTP packets after pause. The
reason is basepayloader generates a randon sequence number when the
pipeline goes from ready to pause. With this fix generation of sequence
number will be monotonic when going from pause to play request.

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

rtsp-client: Protect saved clients watch with a mutex

Fixes a crash when close() is called while merging clients
in handle_tunnel(). In that case close() would destroy the
watch while it is still being used in handle_tunnel().

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

rtsp-stream: Remove the multicast group udp sources when removing from the bin

rtsp-media: Query position and stop time only on the RTP parts of the pipeline

The RTCP parts, in specific the RTCP udpsinks, are not flushed when
seeking and will always continue counting the time. This leads to
the NPT after a backwards seek to be something completely different
to the actual seek position.

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

examples: fix another reference leak

gst_rtsp_media_get_element() returns a new ref.

examples: unref element after usage

gst_bin_get_by_name_recurse_up() returns an element
reference that must be unreffed after usage.

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

signals: Fix copy-pasto in target-state signal offset

Makefile: Add usage of build-checks step

Allows building checks without running them

rtsp-stream: Listen on the multicast group for RTP/RTCP packets

When a UDP multicast transport is used it is expected that the server listens
for RTP and RTCP packets on the multicast group with the corresponding port.
Without this we will never get RTCP packets from clients in multicast mode.

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

Back to development

Release 1.4.0

media: correct misspelled words in description

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

Release 1.3.91

docs: update docs

server: implement client REMOVE filter

client: expose _close() method

Expose a previously internal close method to close the client
connection.

session-pool: signal session-removed outside of the lock

Release the lock before emiting the session-removed signal.

filter: Release lock in filter functions

Release the object lock before calling the filter functions. We need to
keep a cookie to detect when the list changed during the filter
callback. We also keep a hashtable to make sure we only call the filter
function once for each object in case of concurrent modification.

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

client: check if watch is set in handle_teardown()

The unit tests run without a watch

client tests: send teardown to cleanup session

server tests: send teardown to cleanup session

client: keep ref to client for the session removed handler

This extra ref will be dropped when all client sessions have been
removed. A session is removed when a client sends teardown, closes its
endpoint of the TCP connection or the sessions expires.

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

client: manage media in session as a last step

Once we manage a media in a session, we can't unmanage it anymore
without destroying it. Therefore, first check everything before we
manage the media, otherwise if something is wrong we have no way to
unmanage the media.
If we created a new session and something went wrong, remove the session
again. Fixes a leak in the unit test.

examples: print 'stream ready at url' for mp4 and ogg example

rtsp: fix for MIKEY api change

client: free watch context only once

The watch context is freed when the source is destroyed. Avoids
a CRITICAL when we try to unref the context twice.

client: fix build

client: protect sessions with lock

Protect the list of sessions with the lock.

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

Client: keep a ref to the session

Don't just keep a weak ref to the session objects but use a hard ref. We
will be notified when a session is removed from the pool (expired) with
the new session-removed signal.
Don't automatically close the RTSP connection when all the sessions of
a client are removed, a client can continue to operate and it can create
a new session if it wants. If you want to remove the client from the
server, you have to use gst_rtsp_server_client_filter() now.

Based on patch from Ognyan Tonchev <ognyan.tonchev at axis.com>

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

session-pool: add session-removed signal

Add a signal to be notified when a session is removed from the pool.

Make rtsp-server.h a single-include header, use it for G-I

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

Release 1.3.90

stream: crypto can be NULL

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.