rtsp-media: Don't include payload type in the caps for framesize

When the sdp media attribute framesize are converted to caps
the <payload> should not be included.

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

Based on the patch for rtspsrc by Linus Svensson <linussn@axis.com>

rtsp-sdp: add payload type to the sdp framesize attribute

The sdp framesize attribute is desribed in RFC6064. It is specified
for payloading of H263 and has the following form
a=framesize:<payload type> <width>-<height>. The <width>-<height> part
should be added to the caps in a payloader and the <payload type> should
be added by the rtsp-server.

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

examples: test-uri: fix tainted variable

Insignificant but this keeps Coverity happy.

CID #1268404

examples: Add a simple example of network synch for live streams.

An example server and client that works for synchronising live streams
only - as it can't support pause/play.

rtsp-media-factory: Add functions to set/get the media gtype

Allow specifying the GType of a GstRtspMedia subclass to create
as a simpler way to get the factory to create a custom
GstRtspMedia sub-class, without subclassing GstRtspMediaFactory.

rtsp-media: fix double unlock in _get_buffer_size()

Fixes an abort when calling gst_rtsp_media_get_buffer_size()
because of double g_mutex_unlock () usage.

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

rtsp-session: Use monotonic time for RTSP session timeout

Changed RTSP session timeout handling to monotonic time
and deprecating the API for current system time.

This fixes timeouts when the system time changes.

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

rtsp-client: Only error out in PLAY if seeking actually failed

If the media was just not seekable, we continue from whatever position we are
and let the client decide if that is what is wanted or not.

Only if the actual seek failed, we can't really recover and should error out.

rtsp-stream: Add necessary queues between tee and multiudpsink

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

rtsp-media: If seeking fails, don't wait forever for the media to preroll again

Instead error out properly the same way as if the SEEKING query already
failed.

rtsp-stream: minor code formatting fix

rtsp-media: fix logic for collect_streams

Fix the logic of gst_rtsp_media_collect_streams() so after looping collecting
all streams it knows if it got any, and can check if the transport mode is OK.

CID #1268400

rtsp-media: Don't set the transport mode based on what elements we find

Just print a warning if the one that was set before disagrees with what
elements we found. It must already be set to something before as this
function is called after we received the SDP from ANNOUNCE in RECORD mode,
and we would reject ANNOUNCE if the RECORD flag was not set.

tests: rtspserver: rename shadowed variable

We have two different 'sink' variables here,
rename one of them for clarity.

rtsp-client: fix awkward if clause

examples: test-uri: improve uri argument handling and accept file names

Print an error if the argument passed is not a URI and can't
be converted into one, or no arguments have been provided.

examples: test-uri: don't remove mount point after 10 seconds

It's very irritating when trying to test stuff repeatedly
and serves no real purpose other than showing that it can
be done.

examples: add new test-record to .gitignore

rtsp-media: Use flags to distinguish between PLAY and RECORD media

test-record: Set latency for playback-style example to 2s instead of 200ms

tests: add some unit tests for ANNOUNCE and RECORD

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

rtsp-client: fix a couple of leaks in handle_announce

rtsp-media: Expose latency setting for setting the rtpbin latency

test-record: Use GOptionContext to parse the server port and take the pipeline from the commandline

rtsp-stream: Put the timestamp of receival of the initial packet over TCP on the first buffer

Add initial support for RECORD

We currently only support media that is RECORD or PLAY only, not both at once.

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

rtsp-stream: RTCP and RTP transport cache cookies seperated

RTCP packets were not sent because the same tr_cache_cookie was used for
both RTP and RTCP. So only one of the tr_cache lists were populated
depending on which one was sent first. If the tr_cache list is not
populated then no packets can be sent. Most often this happened to be
RTCP. Now seperate RTCP and RTP transport cache cookies are added which
resulted in both the tr_cache_lists to be populated regardless of which
one was sent first.

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

rtsp-stream: fix false compiler warning

rtsp-stream.c:3034: error: ‘visited’ may be used uninitialized in this function

rtsp-client: log interleaved data received

rtsp-client: fix unintentional fallthrough to debug warning when receiving interleaved data

rtsp-client: If we have a single-stream media and SETUP contains no control, use the one and only stream

rtsp-client: Use a random session ID in the SDP

RFC4566 Section 5.2 says that it should make the username, session id,
nettype, addrtype and unicast address tuple globally unique. Always using
1188340656180883 is not going to guarantee that: https://xkcd.com/221/

Instead let's create a 64 bit random number, which at least brings us
closer to the goal of global uniqueness.

https://tools.ietf.org/html/rfc4566#section-5.2

examples: Don't call gst_init() and gst_get_option_group()

The latter calls the former at the appropriate time.

rtsp-client: Drop trailing \0 of RTSP DATA messages

We add a trailing \0 in GstRTSPConnection to make parsing of
string message bodies easier (e.g. the SDP from DESCRIBE) but
for actual data this means we have to drop it or otherwise
create invalid data.

rtsp-stream: Have one copy of the transports cache for RTP and RTCP each

Fixes crash when two threads access handle_new_sample() at the same
time, one for RTP, one for RTCP.

Otherwise, when iterating over the transports cache, it might be modified by
another thread at the same time if the transports cookie has changed.

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

rtsp-stream: Set format=TIME on our app sources for TCP

Revert "rtsp-session-pool: Make sure session IDs are properly URI-escaped"

This reverts commit 935e8f852d050b4939f1d0f44b38e9b55a2fbe36.

RFC 2326 states that session IDs may consist of alphanumeric as well as
the safe characters $-_.+ -- N.B. the percent character is not allowed.

Previously the session ID was URI-escaped, this meant that any character
which was not alphanumeric or any of the characters +-._~ would be
percent encoded. While the RFC (surprisingly) mentions that linear white
space in session IDs should be URI-escaped, it does not say anything
about other characters. Moreover no white space is allowed in the
session ID. Finally the percent character which is the result of
URI-escaping is not allowed in a session ID.

So there is no reason to do any URI-escaping, and now it is removed.

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

Automatic update of common submodule

From f2c6b95 to bc76a8b

Fix 'make check' from top-level directory

examples: Add command-line parsing and take a 'port' argument

This allows users to run multiple servers on different ports for testing.

Only done for examples that actually take arguments and hence are capable of
outputting different streams for each instance on each port.

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

rtsp-client: Add a send_message default signal handler

This allows subclasses to easily hook into the response sending
mechanism without doing everything from a signal, which seems
awkward from subclasses.

Automatic update of common submodule

From ef1ffdc to f2c6b95

configure: add --disable-examples switch

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

examples: add a retransmisison example implementing RFC4588

Currently only SSRC-multiplexed rtx streams are supported

rtsp-stream: Fix some minor memory leaks

rtsp-media: Some minor cleanup

rtsp-stream: Fix compiler warnings

rtsp-stream.c:1351:3: error: non-void function 'gst_rtsp_stream_get_retransmission_time' should return a value [-Wreturn-type]
  g_return_if_fail (GST_IS_RTSP_STREAM (stream));
  ^

rtsp-stream.c:1384:3: error: non-void function 'gst_rtsp_stream_get_retransmission_pt' should return a value [-Wreturn-type]
  g_return_if_fail (GST_IS_RTSP_STREAM (stream));
  ^

media: implement ssrc-multiplexed retransmission support

based off RFC 4588 and the server-rtpaux example in -good

rtsp: Ref transports in hash table.

Also ref streams for transports.
This solves a crash when reciving a rtcp after teardown but before
client finalize.

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

Automatic update of common submodule

From 7bb2bce to ef1ffdc

client: refactor cleanup of cached media

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