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

rtsp-server: support build against last stable release

Until 1.2.3 is out with the new get_type function and we
can require that.

stream: fix compilation

stream: add property to configure profiles

client: let stream check supported transport

Delegate the check if a transport is allowed to the stream.

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

stream: add method to check supported transport

Add a method to check if a transport is supported

configure.ac: Only check for gstreamer-check, not check

We include check in gstreamer-check since quite some time now.

stream: return clock-rate from get_rtpinfo

And use it to correct the rtptime to the requested start-time.

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

session-media: calculate start-time

stream: also return the running-time

Return the running-time in the rtpinfo as well.

session-media: let the session-media make the RTPInfo

Add method to create the RTPInfo for a stream-transport.
Add method to create the RTPInfo for all stream-transports in a
session-media.
Use the session-media RTPInfo code in client. This allows us to refactor
another method to link the TCP callbacks.

mount-points: sort sequence before g_sequence_lookup

* gst/rtsp-server/rtsp-mount-points.c (gst_rtsp_mount_points_remove_factory):
  sort sequence if dirty, otherwise lookup will fail.

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

configure: rename package from gst-rtsp to gst-rtsp-server

To match git module name and avoid confusion with the
rtsp lib in gst-plugins-base and rtsp plugin in -good.

configure: bump core/base/good requirement to 1.2.0

Bump to released stable version and make implicit
requirements explicit.

Fix broken gettext setup which is not used anyway

Automatic update of common submodule

From dbedaa0 to d48bed3

media: add setup_sdp vmethod

gst/rtsp-server/rtsp-media.[ch]: added setup_sdp vmethod and public
gst_rtsp_media_setup_sdp.

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

rtsp-stream: Check return value of sscanf

streamid is only valid if sscanf matched something.

rtsp-client: Fix iteration

Wouldn't even enter the code block otherwise (i++ was used as the check
and not the postfix).

client: add vmethod to configure media and streams

Implement a vmethod that can be used to configure the media and the
streams based on the current context. Handle the blocksize handling in
the default handler.

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

Make git ignore more unit test binaries

rtsp-server: add padding to many public structures

Not mini objects though, since they are not subclassable
anyway, nor kept on the stack or inlined in a structure.

media: add new create_rtpbin vmethod

* gst/rtsp-server/rtsp-media.[ch]: add new create_rtpbin vmethod.

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

tests: fix memory leak, free test's thread pool

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

stream-transport: free url in finalize

media: also do state change in suspended state

media: also handle prepare and range in suspended state

When we are suspended, we are already prepared.
We can get the range in the suspended state.

check: add test for uri in setup

Added unit tests for the new functionality in GstRTSPStreamTransport.

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

client: store setup uri and use in PLAY response

Store the uri used when doing the setup and use that in the PLAY
response.

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

stream-transport: add method to get/set url

client: suspend after SDP and unsuspend before PLAYING

Based on patches by Ognyan Tonchev <ognyan@axis.com>

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

media: add suspend modes

Add support for different suspend modes. The stream is suspended right after
producing the SDP and after PAUSE. Different suspend modes are available that
affect the state of the pipeline. NONE leaves the pipeline state unchanged and
is the current and old behaviour, PAUSE will set the pipeline to the PAUSED
state and RESET will bring the pipeline to the NULL state.
A stream is also unsuspended when it goes back to PLAYING, for RESET streams,
this means that the pipeline needs to be prerolled again.

Base on patches by Ognyan Tonchev <ognyan@axis.com>

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

media: start live streams in blocked state

Start live streams in the blocked state and make them preroll using the
messages. This ensure that no data is played by the sink until we explicitly
unblock the stream right before going to PLAYING.

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

media: refactor starting and waiting for preroll

Based on patches from Ognyan Tonchev <ognyan@axis.com>

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

stream: add API to block streams

Add an API to block on the streams and make it post a message.

Based on patch by Ognyan Tonchev <ognyan@axis.com>

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

docs: Specify the override file

Even if it's empty (for now) it avoids make distcheck complaining

media: move default implementations to where they are used

media: take the right lock in gst_rtsp_media_set_pipeline_state()

We need to take the state_lock when calling this method.

media: handle add-added on non-bins too

Handle dynamic payloaders that are not bins, as used in the unit-test.

rtsp-media/-factory: Fix request pad name comments

These must be escaped for gtk-doc to parse the comments without warnings.

rtsp-media: remove transports if media is in error status

* gst/rtsp-server/rtsp-media.c (gst_rtsp_media_set_state): if we are
  trying to change to GST_STATE_NULL and media is in error status, we
  remove all transports.

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

rtsp-media: use element metadata to find payloader

Use the element metadata to find the payloader instead of checking
for the base class.

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

rtsp-stream: add getter for payload type

* gst/rtsp-server/rtsp-stream.c: add new method gst_rtsp_stream_get_pt.

* gst/rtsp-server/rtsp-media.c (pad_added_cb): find real payloader
  element and create the stream with this one instead of the dynpay%d
  element.

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

rtsp-*: Refer to NULL as a constant in comments

Plus one typo fix.

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

rtsp-*: Fix type name typos in comments

  * rtsp-auth: Refer to GstRTSPToken, not GstRTSPtoken
  * rtsp-auth: Refer to part of constant name as text
  * rtsp-auth/-permissions/-token: Refer to Permissions not Permission
  * rtsp-session-media: Fix GstRTSPSessionMedia typo
  * rtsp-stream: Fix typo when refering to GstBin

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

docs: Improve documentation

  * Include annotation-glossary to quiet gtk-doc
  * Rename remaining ClientState -> Context
  * Rename object hierarchy file
  * Remove stale chapter references
  * Add missing function and object references
  * Include missing GstRTSPAddressPoolResult

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

rtsp-server: sprinkle some allow-none annotations for g-i

stream: add method to filter transports

Add a method to safely iterate and collect the stream transports

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

rtsp: allow NULL func in filters

Passing a null function make the filters return a list of
refcounted objects.

address-pool: fix address increment

Use a guint instead of guint8 to increment the address. It's still not
completely correct because a guint might not be able to hold the complete
address range, but that's an enhacement for later.
Add unit test to test improved behaviour.

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

client: allow absolute path in requests

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

client: make make_path_from_uri a vmethod

stream: Add functions to get rtp and rtcp sockets

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

context: defing a GType for the context

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

Fixed several GIR warnings

auth: small typos

tests: Add unit tests for token

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

token: Validate args for gst_rtsp_token_is_allowed

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

token: Fix bug when creating empty token

We always want to have a valid GstStructure in the token.

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

thread-pool: avoid race in shutdown

If we call g_main_loop_quit before the thread has entered g_main_loop_run, we
don't actually stop the mainloop ever. Solve this race by adding an idle source
to the mainloop that calls the _quit. This way we immediately exit the mainloop
if quit was called before we started it.

tests: Add unit tests for permissions

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

tests: Test mediafactory permissions

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

permissions: Fix refcounting when adding/removing roles

Previously a role that was removed was unreffed twice, and when
replacing an existing role the replaced role was freed while still being
referenced. Both bugs are now fixed.

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

tests: Check gst_rtsp_url_parse return value

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