oggdemux: do not dereference NULL pad in warning message

Coverity 1197695

video-event: Update the running times in the force-keyunit events from the pad offsets

decodebin: In adaptive streaming mode, only have a fixed buffer limit for the non-buffering multiqueue

sdp: guard against address parse errors.

adder: rework the logic to check if eos has to be sent.

Checking the size available was incorrect, and the infos
for per-pad EOS are available.

Same logic as audiomixer.

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

audioringbuffer: parse channels field from compressed audio caps

Also parse channels as an optional field in the caps for compressed
audio formats.

subtitleoverlay: Consider all caps for overlays, not just the first.

Check all supported caps on the overlay video pad, not just the
first of (possibly) many.

tools: update gst-play-1.0 man page

videodecoder: do not deactivate the bufferpool, just unref

Videodecoder does late renegotiation, it will wait for the next
buffer before renegotiating its caps and bufferpool. It might happen
that downstream element switched from passthrough to non-passthrough
and sent a reconfigure upstream (that caused this renegotiation).
This downstream element will ask the video sink below for the bufferpool
with an allocation query and will get the same bufferpool that
videodecoder is holding, too.

When renegotiating, if videodecoder deactivates its bufferpool it
might be deactivating the bufferpool that some element downstream
is using and cause the pipeline to fail.

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

audiobasesink: clip start samples to match clipped start time

Clock slaving can clip start time to zero, giving us a shorted
duration than we originally got. To keep in sync, we must then
discard the samples falling before that zero timestamp.

This possibly fixes random distortion caused by constant PA
underflows which are never resynced.

mikey: Fix the KEMAC payload

The KEMAC payload actually needs to have subpayloads and the key should
go into the KEY_DATA subpayload. Add support for subpayloads and
implement the KEY_DATA payload.
Add some pointers to the conversion functions that allow us to add
encryption and decryption later.

playbin: Drop reference to any source element in NULL state

Drop the reference instead of waiting for either finalize(), or
for a new source when reused. Everyone else already forgot about
the old source.

rtspconnection: Added gst_rtsp_watch_set_flushing to list.

Added gst_rtsp_watch_set_flushing to list in file
libgstrtsp.def

videodecoder: Always drain the decoder after a discont group in reverse playback mode

videodecoder: Flush the decoder once per discont group, not once per keyframe

videodecoder: Handle reverse playback with multiple GOPs per discont group properly

baseparse will reverse each GOP for us already, so the segment events can
be after our keyframe. Make sure to get it and all other relevant sticky
events before starting to decode.

videodecoder: Log event types of events that are pushed downstream

videodecoder: In reverse playback mode we need to finish the subclass after passing all frames to it

rtspconnection: add flush method

Add a method to set/unset the flushing state that makes _wait_backlog()
unlock.

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

ximagesink: only extrapolate alpha mask for 32-bit depth

Instead of passing bogus alpha mask values when there's no alpha.

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

mikey: fix return values of g_return_*

rtsptransport: UDP is also default for SAVP and AVPF

docs: add MIKEY docs

mikey: add MIKEY parsing helpers

MIKEY is defined in RFC 3830 and is used to exchange SRTP encryption
parameters between a sender and a receiver in a secure way.
This library implements a subset of the features, enough to implement
RFC 4567, using MIKEY in SDP and RTSP.

rtspconnection: Fix minor memory leaks in error handling

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

rtspconnection: Fix connection_poll()

* Only check for conditions we are interested in.
* Makes no sense to specify G_IO_ERR and G_IO_HUP in condition, they
  will always be reported if they are true.
* Do not create timed source if timeout is NULL.
* Correctly wait for sources to be dispatched, context_iteration() is
  not guaranteed to always block even if set to do so.

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

rtpbasepayload: add pt and ssrc to stats

tests: decodebin: port old decodebin2 test for parser and decoder linking

They were in the old decodebin2.c tests file and were never ported.
Now we can get rid of decodebin2.c

playback: Add video-/audio-filter properties

This provides an audio-filter and video-filter property to allow
applications to set filter elements/bins. The idea is that these will
e
applied if possible -- for non-raw sinks, the filters will be skipped.

If the application wishes to force the application of the filters, this
can be done by setting the new flag introduced on playsink -
GST_PLAY_FLAG_FORCE_FILTERS.

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

Revert "playback: Add video-/audio-filter properties"

This reverts commit fb8fdedb4f4649aa33700bbc720131c1678df49f.

playback: Add video-/audio-filter properties

This provides an audio-filter and video-filter property to allow
applications to set filter elements/bins. The idea is that these will be
applied if possible -- for non-raw sinks, the filters will be skipped.

If the application wishes to force the application of the filters, this
can be done by setting the new flag introduced on playsink -
GST_PLAY_FLAG_FORCE_FILTERS.

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

rtspconnection: Silence a compiler warning

Cast the argument into (const char *) on W32, as winsock2 expects it.

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

playsink: Fix documentation for what the audio chain looks like

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

docs: update plugin docs and remove old properties and signals

Re-generate .args and .signals file from scratch so that
old signals that no longer exist (such as the 'new-decoded-pad'
signal on decodebin) no longer show up in the documentation.

adder: set a group-id on the stream-start event

Set a default group-id to fix a warning printed by the sink.

Add new header file

oggmux: implement vp8 granulepos function

Add an extra function to the oggstream map to inform it about
the incoming buffers. This way oggmux can keep a count on the
vp8 invisible frames and calculate the granulepos correctly.

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

oggmux: create vp8 header data if not provided in caps

vp8 stream header shouldn't be assumed to be provided in caps always
as this would repeat the same code in all demuxers/encoders. Instead,
make oggmux generate them if they are not supplied.

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

rtspconnection: gst_rtsp_watch_wait_backlog

New method that wait until there is room in backlog queue.

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

rtspconnection: GstRTSPWatch func for tunnel GET response

Add a callback in GstRTSPWatch where the response to HTTP GET for
tunneled connections can be modified.

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

rtspdefs: add RFC 4567 headers and status code

This new Header and status code is used for SRTP

decodebin: Buffer up to 5 seconds in multiqueue buffering mode

2 seconds might be too small for some container formats, e.g.
MPEGTS with some video codec and AAC/ADTS audio with 700ms
long buffers. The video branch of multiqueue can run full while
the audio branch is completely empty, especially because there
are usually more queues downstream on the audio branch.

decodebin: Keep the number of buffers after an adaptive streaming demuxer lower

Usually these buffers are multiple seconds large, and having a maximum
of 5 buffers in the multiqueue there can use a lot of memory. Lower
this to 2 for adaptive streaming demuxers.

decodebin: Simplify adaptive streaming demuxer code a bit

pango: demote debug WARNING to LOG for variable framerate video input

No need why we need to warn about that, it's perfectly allowed.

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

tests: add textoverlay passthrough with composition feature unit tests

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

pango: basetextoverlay: handle video/x-raw(ANY) if downstream supports the GstVideoOverlayCompositionMeta API

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

video-overlay-composition: add GST_CAPS_FEATURE_META_GST_VIDEO_OVERLAY_COMPOSITION

docs: Removing GnomeVFS left bits

gnomevfs was removed time ago but there are still some left bits.

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

typefindfunctions: lower H.263 typefinder max probability

The typefinder returns LIKELY for as little as one possible
sync and no bad sync (not even taking into account how much
data was looked at for that). It's generally just not fit
for purpose, so should just not return anything like LIKELY
at all ever, even more so since it only recognises one out
of ten H263 files, and likes to mis-detect mp3s as H263.

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

rtspconnection: Call closed() when GET is closed in tunneled mode

This patch adds read source on the write socket in tunneled
mode and we get a callback when client disconnects the GET
channel.

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

videoformat: Remove duplicate/incorrect section

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

docs: Add annotations for return values

Rephrase and clarify some return value descriptions

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

docs: Fix argument and annotation typos

 * colorbalance: Fix misspelled annotation
 * rtsp: Replace incorrectly documented function argument
 * sdp: Escape @ character to avoid gtk-doc warning
 * video-*: Add missing annotation colon
 * videodecoder/video-color: Fix function argument typos
 * videoutils: Remove unknown annotation field

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

.gitignore: Ignore gcov intermediate files

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

Automatic update of common submodule

From fe1672e to bcb1518

playbin: improve autoplug_query_caps return

Makes autoplug_query_caps return
downstream_caps + intersect_first(filter_caps, element_caps)

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

Automatic update of common submodule

From 1a07da9 to fe1672e

rtsp: fix build with older GLib versions

The gio/gnetworking.h header is only available since glib 2.36

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

rtspconnection: Add missing include

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

playsinkconvertbin: improve gst_play_sink_convert_bin_getcaps return

If we have the peer caps and a caps filter, return peer_caps +
intersect_first (filter, converter_caps) instead of
intersect_first (filter, peer_caps + converter_caps) and preservers
downstream caps preference order.

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

tests: Refactor RTP basepayloading test into pay/depay parts

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

rtpbasepayload: Let caps event also configure seqnum-offset

Previously the sequence number kept track of by GstRTPBasePayload would
only be set when going from READY to PAUSED state. This meant that a
downstream element that attempted to configure a basepayloader by
setting seqnum-offset e.g. in its sinkpad's caps template would have
trouble configuring the basepayloader. The reason was that the caps
event which arrives with the desired value for seqnum-offset did not
arrive at the basepayloader until caps negotiation took place,
significantly later than the transition from READY to PAUSED.

The result after this patch is that the default value for the
seqnum-offset property, or later set values for this property, will take
effect when going from READY to PAUSED like before. In addition the an
arriving caps event will also affect the basepayloaders configured
sequence number as the event arrives.

rtpbasepayload: Fix payload type property boundary value

The payload type field in an RTP packet header is 7 bits wide, hence the
boundary values ought to be 0x00 and 0x7f, not the previously stated
values 0x00 and 0x80.

rtpbasedepayload: Fix typos in comments

docs: add GstVideoPool to docs

decodebin: If we have a demuxer without dynamic srcpads, just assume no-more-pads

Otherwise we will wait until the multiqueue after the demuxer will
overrun, which is clearly not needed then.

decodebin: Also make sure to not duplicate an element factory after a group

If we are using an adaptive stream demuxer, which outputs a non-container
stream, we are putting another multiqueue after the *parser* following
the adaptive stream demuxer. We do not want to add another instance of
the same parser right after this multiqueue.

decodebin: During pre-rolling always use the auto-preroll limits on multiqueues

Even if we're buffering in the multiqueues.

decodebin: Pass through the seekability information when setting multiqueue limits

decodebin: During exposing of pads don't set the multiqueue limits multiple times to different values

Instead just set them once in the very end to the correct values.

decodebin: Only enable multiqueue buffering once we're pre-rolled

Otherwise we will emit buffering messages not just from the last
multiqueue but also from previous multiqueues... confusing the
application with different percentages during pre-rolling.

decodebin: Make sure that we always have a second multiqueue for adaptive streaming demuxers

For adaptive streaming demuxer we insert a multiqueue after
this demuxer. This multiqueue will get one fragment per buffer.
Now for the case where we have a container stream inside these
buffers, another demuxer will be plugged and after this second
demuxer there will be a second multiqueue. This second multiqueue
will get smaller buffers and will be the one emitting buffering
messages.
If we don't have a container stream inside the fragment buffers,
we'll insert a multiqueue below right after the next element after
the adaptive streaming demuxer. This is going to be a parser or
decoder, and will output smaller buffers.

uridecodebin: Always use buffering in multiqueue for adaptive streams

uridecodebin: Only add a queue2 for buffering for non-adaptive streaming streams

uridecodebin: pass on the buffering property for adaptive streams

Adaptive streams should download its data inside the demuxer, so
we want to use multiqueue's buffering messages to control the
pipeline flow and avoid losing sync if download rates are low;

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

tests: add new unit tests to .gitignore

rtspconnection: New unit test

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

rtspconnection: Remove read child source when POST is disconnected

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

defs: update for new rtspconnection symbols

oggdemux: allow file to go until the end in push mode

When seeking back to original state after duration seeks, let
upstream know that we want the whole file, including the last
byte that wasn't requested on the duration seeks.

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

oggdemux: remove unused instance variable event

It is never set to anything

rtspconnection: allow specifying a certificate database

Two new functions have been added,
gst_rtsp_connection_set_tls_database() and
gst_rtsp_connection_get_tls_database(). The certificate database will be
used when a certificate can't be verified with the default database.

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

rtspconnection: get rid of superfluous whitespaces

encodebin: simplify tests

Also use the profile helper for the ogg profile here.

video: Fix NV12_64Z32 default offset and size

This was a regression introduced by f52fd7a68, where we started using
the stride to encode the dimensions in tiles. This patch simply updates
offset and size calculation as described in the documentation,
part-mediatype-video-raw.txt.

playbin: Keep inputselector around until we release its pads

Otherwise there's an interesting race condition when we destroy
the inputselector (actually it will be destroyed later when its state
change message gets destroyed) and afterwards release its sinkpad.

This is the code path when the last channel is removed from the
input selector.

Gave this warning sometimes, for chained oggs or whenever else
we change decode groups:
GStreamer-CRITICAL **: Padname '':sink_0 does not belong to element inputselector0 when removing

audioconvert: never do mixing for 1->1 channel conversions

MONO and NONE position are the same, for example, but in
general there isn't much to do here for such a conversion.

Fixes problem in audioconvert, which would end up using
a mixmatrix when converting between different mono format
because it thinks MONO positioning is different from
unpositioned channels, which is not the case in this
special case. The mixmatrix would end up being 0.0 so
audioconvert would convert to silence samples.

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

audio: map channels=1,channel-mask=0 to MONO instead of NONE

Fixes problem in audioconvert, which would end up using
a mixmatrix when converting between different mono format
because it thinks MONO positioning is different from
unpositioned channels, which is not the case in this
special case. The mixmatrix would end up being 0.0 so
audioconvert would convert to silence samples.

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

encodebin: refactor tests

Add a new test to demo how to get missing plugin message.
Split some tests that unneccesarily munge unrelated checks into one test.

playsink: Only remove the complete text chain if the text pad goes away

If the text pads does not go away we just set the overlay to silent, which
allows us to immediately re-enable subs later again. However before this
change we also released the streamsynchronizer text pads, which deadlocked
because there was still dataflow going on. Just do this only if we remove
the complete chain.

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

tools: gst-play: add volume control

oggmux: properly flush when seeking at the beginning

Reset all internal status when collect pads forwards a flush-stop
from the pads to be able to start the stream again.

uridecodebin: Don't leak pad references

tests: Don't build disabled plugins' check tests

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

playbin: First try to get the pad's current caps, then query caps

The caps query might give us ANY caps while the pad has fixed caps
configured currently.

playbin: Fix memory leak in autoplugging code

We should not leak element factories ideally.

playbin: Fix memory leak in unit test

subtitleoverlay: Remove unused function

audiosrc: Fix typo in docs

We read *from* the audio device, not to it.