rtsp-stream: Fix indentation

rtsp-media: Do not prepare media after media times out

Deferred calls to start_prepare() can be deferred past the point until
which wait_preroll() and by proxy gst_rtsp_media_get_status() is
prepared to wait. Previously there was no lock and no check for this
situation. This meant that a media could be prepared and unprepared
simultaneously by two different threads. Now a lock is in place and a
suitable check is done.

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

rtsp-media: Add property to decide if sending media should be stopped when a client disconnects without TEARDOWN

Without TEARDOWN it might be desireable to keep the media running and continue
sending data to the client, even if the RTSP connection itself is
disconnected.

Only do this for session medias that have only UDP transports. If there's at
least on TCP transport, it will stop working and cause problems when the
connection is disconnected.

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

Back to development

Release 1.7.1

configure: Make -Bsymbolic check work with clang.

Update the -Bsymbolic check with the version glib has. This version
works with clang.

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

rtsp-session-pool: Avoid dollar sign ($) in session ids

Live555 in VLC strips off dollar signs and then gets very confused,
we don't loose too much entropy by just skipping it.

rtsp-server: Add g_autoptr() support to all types

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

rtsp-stream: fixed valgrind error

Fixed the valgrind error in unit test. The UDP source created during
gst_rtsp_stream_join_bin() was not released while destroying the rtp
bin.

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

Automatic update of common submodule

From b319909 to 86e4663

rtsp-client: suspend media during setup request

SETUP request from clients needs to suspend the media to clear the
prerolled buffers. Otherwise it will not affect the prerolled buffer
and the prerolled buffers will be incorrect (for example block-size
from setup request will not affect the prerolled buffer unless the
media is suspended).

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

rtsp-stream: create stream pipeline based on transport

Based on the protocol, create the rtsp stream pipeline. If only TCP or
only UDP is set as the transport protocol, it will not add the extra tee
or queue element to the pipeline. Both these elements will be added, if
it supports both TCP and UDP protocols. This improves the pipeline
performance when one protocol is present.

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

rtsp-stream: Only create RTP sending/receiving rtpbin pads if needed

Adding them when not needed will start some logic inside rtpbin that might be
problematic. Also if e.g. for a sender media we suddenly receive RTP data, we
would start up a rtpjitterbuffer and behave in weird ways.

We still set up the UDP sources for RTP receiving for a sender media to be
able to receive any packets sent by the client for NAT traversal. They will
all go to a fakesink though.

Having an rtpjitterbuffer in the media pipeline will cause the pipeline to be
NO_PREROLL, which will cause deadlocks when seeking the media as it will never
receive ASYNC_DONE after a seek.

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

rtsp-stream: Disable multicast loopback for the multicast udp sources too

On POSIX this setting is for sender sockets, on Windows for receiver sockets.
Previously we were only setting this for sender sockets, which caused looped
back packets to be received on Windows if a multicast transport was used.

examples: Actually use the provided port in the record examples

test-record-auth: Add the option to build in TLS support

test-auth: Use an 'anonymous' user for unauthenticated default

There's a comment on one of the resources that 'user' and 'admin'
shouldn't even be able to see it, but they can if the default
token is 'admin2', since that gives them access anyway.

Add test-record-auth example

rtsp-client: Report RECORD and ANNOUNCE as supported in the OPTIONS

rtsp-server: Change the logic so we don't pop a NULL context

When doing a port scan (e.g. with nmap) the call to GST_RTSP_CHECK()
will sometimes fail. This call is made before any context is pushed
resulting in an attempt to pop a NULL context.

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

rtspserver: Add udp-mcast transport SETUP test

Refactor utility functions in the test file so they can handle
more than UDP and TCP as lower transport.

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

rtsp-stream: Always unref return value of gst_object_get_parent()

Fixes a leak of a GstBin in the udp-mcast case.

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

Automatic update of common submodule

From b99800a to b319909

Use new GST_ENABLE_EXTRA_CHECKS #define

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

Automatic update of common submodule

From 6babecd to b99800a

Update GLib dependency to 2.40.0

stream: listen to sender ssrc signals

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

common: update for new suppression

Makes check-valgrind pass with glib 2.46

rtsp-media: Take reference to media that will be prepared

default_prepare() takes a transfer-none reference GstRTSPMedia object.
Later on a g_idle_source_new() is created and a pointer to the media
object is passed as user data. If the media is freed before the idle
source is dispatched the media object pointer is invalid, but the idle
source callback expects it to still be valid. To fix this a reference to
the media object is taken when registering the source callback function
and a corresponding release of the reference is done when the souce is
destroyed.

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

rtsp-server: Fix memory leaks when context parse fails

When g_option_context_parse fails, context and error variables are not getting free'd
which results in memory leaks. Free'ing the same.

And replacing g_error_free with g_clear_error, which checks if the error being passed
is not NULL and sets the variable to NULL on free'ing.

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

Back to development

Release 1.6.0

Release 1.5.91

stream: fix docs for recently-added get/set_buffer_size API

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

rtsp-media: Don't crash on encrypted RTX SDP

In parse_keymgmt(), don't mutate the input string that's been passed
as const, especially since we might need the original value again if
the same key info applies to multiple streams (RTX, for example).

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

test-mp4: Support filenames with spaces in them. Error out on too few arguments

test-record: Check parameter count and print out help

If no launch pipeline was supplied, print out some help

rtsp-stream: Implement UDP buffer size setting.

Add gst_rtsp_stream_(get|set)_buffer_size and use it to configure the
UDP TX buffer size.

Incorporates a patch by Hyunjun Ko <zzoon.ko@samsung.com>
Fixes https://bugzilla.gnome.org/show_bug.cgi?id=749095

rtsp-media: Fix small typo causing gtk-doc to complain

Release 1.5.90

media-factory: get port number through gst_rtsp_url_get_port

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

media-test: Removing unnecessary assertion

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

Document that source keeps a ref on server until it's destroyed

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

media-test: Test for multiple dynamic payload

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

media: Only add fakesink once per pipeline

The intention is to prevent going PLAYING state before pads are created.
If there was mutilple dynamic payload, it would leak few fakesink and
actually prevent from ever reaching playing state.

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

Revert "rtsp-media: Only add 1 fakesink per pipeline"

This reverts commit 22bf61f16c1210bb458fc3f53642179a0211104f.

rtsp-media: Only add 1 fakesink per pipeline

There should be only one fakesink per pipeline, not per dynpay. This
would lead to element naming clash.

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