rtsp-media: assertion error due to wrong condition check

In media to caps function, reserved_keys array is being used for variable i,
leading to GLib-CRITICAL **: g_ascii_strcasecmp: assertion 's1 != NULL' failed
changed it to variable j

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

rtsp-media: Strip keys from the fmtp that we use internally in our caps

Skip keys from the fmtp, which we already use ourselves for the
caps. Some software is adding random things like clock-rate into
the fmtp, and we would otherwise here set a string-typed clock-rate
in the caps... and thus fail to create valid RTP caps

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

threadpool: Fix possible warning in gst_rtsp_thread_pool_cleanup()

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

Automatic update of common submodule

From f74b2df to 9aed1d7

Back to development

Release 1.5.2

rtsp-client: allow application to decide what requirements are supported

Add "check-requirements" signal and vfunc to allow application
(and subclasses) to check the requirements.

Based on patch from Hyunjun Ko <zzoon.ko@samsung.com>

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

Automatic update of common submodule

From 6015d26 to f74b2df

rtsp-media: Always use real payloader when creating streams

A bin that contains the real payloader might be used as payloader. In this
case we have to get the real payloader for the various properties it provides.

Example use cases for this are bins that payload some media and then have
additional elements that add metadata or RTP extension headers to the stream.

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

test-netclock: Use gst_pipeline_set_latency() to set a high-enough, equal latency for all receivers

test-netclock: Use new ntp-time-source property on rtpbin

Select the clock time to be used as NTP time source. This allows proper
synchronization between receivers, independent of sharing base times, and just
requires them to use the same clock.

test-netclock: Setting the same base time on sender and receiver is not necessary

It's going to be fixed up by rtpbin when using ntp-sync=TRUE

rtsp-stream: add description for gst_rtsp_stream_request_aux_sender

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

docs: add missing types

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

docs: add missing apis

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

test-netclock-client: Use ntp-sync=TRUE and buffer-mode=SYNC for proper synchronization

GstRTSPAuth: Add client certificate authentication support

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

test-netclock-client: Use new GstClock API to wait for clock synchronization

test-netclock-client: Use a GMainLoop and playbin's source-setup signal

A mainloop is needed to get glimagesink to display something on OSX, and
the source-setup signal just makes things a little bit easier.

Automatic update of common submodule

From d9a3353 to 6015d26

Automatic update of common submodule

From d37af32 to d9a3353

Automatic update of common submodule

From 21ba2e5 to d37af32

Automatic update of common submodule

From c408583 to 21ba2e5

docs: remove variables that we define in the snippet from common

This is syncing our Makefile.am with upstream gtkdoc.

Automatic update of common submodule

From 44a3517 to c408583

Back to development

Release 1.5.1

rtsp-client: No flush during Teardown.

When calling gst_rtsp_watch_write_data in gstrtspconnection.c and
backlog is empty it can happen that just a part of a message will be
sent and rest is in backlog queue. If then flush during teardown
just a part of message will be sent.This can lead to client miss
teardown response since it expect to get the last part of message.

The flushing during teardown was introduced to fix a deadlock that now
is fixed more generally in handle_request by temporary  setting backlog
size to unlimited.

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

tests: Use AM_TESTS_ENVIRONMENT

Needed by the new automake test runner and the
current version of the common submodule.

rtsp-server: Use single-include rtsp header to make sure we get all definitions

rtsp-media: Mark some more functions static

rtsp-media: Only unblock the media in suspend() when actually changing the state

Otherwise we're going to lose a few packets for live streams during DESCRIBE.

examples: Use AVPF profile for the RTX example

rtsp-sdp: Only add RTX to the SDP when using a feedback profile

rtsp-stream: get valid clock-rate from last-sample

clock-rate in last-sample's caps is integer, not unsigned.
To get this value properly, variable needs to be type-casted to int.

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

autogen.sh: only run autopoint if gettext requested in configure.ac

Not just because there happens to be a po directory.

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

Revert "configure.ac: uncomment gettext version setup"

This reverts commit 1545d8fef7065081079172ec264a0061039ac075.

We don't need a gettext setup here and there's no po
directory either, so no reason why autopoint would be
run in the first place.

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

Fix timeout function signatures across tests and examples

tests: define GST_CHECK_TEST_ENVIRONMENT_BEACON

Make sure the test environment is set up.

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

configure: bump automake requirement to 1.14 and autoconf to 2.69

This is only required for builds from git, people can still
build tarballs if they only have older autotools.

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

configure.ac: uncomment gettext version setup

Fixes autogen.sh. It would run autopoint, which would complain
that it could not find the gettext version in configure.ac.

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

test-video-rtx: set exact payload type to PCMA payloader

Setting wrong payload type causes failure to do retransmission through audio stream

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

rtsp-stream: fix to get valid each stream data for request-aux-sender signal

Because of duplicated g_signal_connect for request-aux-sender signal,
wrong stream pointer is passed to the signal handler.
Instead of passing each stream, pass stream array and get the relevant stream.

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

Update autogen.sh to latest version from common

Fixes build after aclocal_check etc. helpers have been removed.

Automatic update of common submodule

From bc76a8b to c8fb372

rtsp-stream: Limit the queues to 1 buffer

We only need them to be able to pre-roll, queueing up more data here
is only going to harm latency and memory usage.

rtsp-stream: Update comment and ASCII art to the latest code

We have a queue in front of the udpsink too to prevent the pipeline from
locking up.

rtsp-media: Properly return first rtptime

Instead we where returning first GstBuffer timestamp. This would result
in clock skew and unwanted behaviour in RTSP playback.

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

rtsp-stream: Don't leave buffer mapped

If the seq is NULL, the RTP buffer was left mapped. We should always
unmap the buffer.

Fix typo in README

Fix double semicolons

rtsp-stream: Get the seqnum-base and other information from the last buffer in the sink

This gives more accurate values than asking the payloader. There might be
queueing happening between the payloader and the sink.

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

rtsp-media: Don't seek for PLAY if the position will not change

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

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));
  ^