oggstream: Use proper type for sample calculation

If we are going to return a (potentially) 64bit integer, don't use
a 32bit one for calculation, otherwise we could end up exceeding
the maximum size of a 32bit int.

oggstream: More fixes for invalid granuleshift

Don't use granuleshift if it wasn't set

discoverer: Don't switch to PLAYING if we saw an error

If we saw an error on the bus, we can't be guaranteed that element will
function properly anyway. Avoids weird state change races also

Revert "discoverer: Avoid race of adding elements while stopping"

This reverts commit 9a8cb299702c6e8c01b5f4a366f7cffc6b13f97a.

The problem is deeper down the stack

rtcpbuffer: fix left shift override

Needs to be cast to the target type

discoverer: Avoid race of adding elements while stopping

This would result in a lot of warnings regarding elements not being
in NULL state when removed, or even leaked elements.

Instead make sure we take the lock and check whether we are processing
or not before allocating or adding anything to the pipeline

videotestsrc: Fix undefined left shift

Cast value to target type

video-converter: Fix undefined left shift

Cast value to target type

oggstream: Fix default granuleshift usage

For stream mappers that don't set a specific granuleshift, it will
have the default value of -1.

Protect the code for that and return the granule value as-is

oggdemux: Protect against invalid granule positions

Only valid values are -1, 0 or positive values. Anything else is
most likely corrupted data streams

codecutils: improve input validation in opus header parsing

Invalid input files do not warrant assertions. Instead
output error messages and let the error bubble up.

sdpmessage: add_attribute accepts NULL value

The attribute can be defined without value regardless session-level
or media-level.
Although `gst_sdp_message_insert_attribute` can be used to set NULL,
it would be easier if `gst_sdp_message_add_attribute` accepts NULL.

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

typefind: Fix out-of-bound read in PNM typefinder

vorbistag: Fix previous comment

We already NULL-ended the string, don't use the bogus cur_size

typefindfunctions: fix off-by-one in webvtt typefinder

We're also checking the byte after the WEBVTT magic.

riff-media: Handle strf_data being NULL

Instead of trying to get the size of a NULL buffer :)

vorbistag: Check whether tag name is valid

In the same way we check before whether the content of the tag
is UTF-8

discoverer: Check return values

The target pad of a ghostpad could have gone (due to shutdown taking
place for example). Check for it's existence before doing anything
with it.

decodebin2: Recheck shutdown behaviour in error case

If we can expose the main chain, recheck whether we are shutting
down or not.

decodebin2 might have been set to READY/NULL during the attempt
to expose, which would cause it to fail ... but it is not a fatal
issue.

discoverer: Uniquely name elements

Helps identifying actual queue/fakesink usage. There seems to be
a race in discoverer, this should help track it down.

oggdemux: Only track time for initialized streams

in push-mode we only can track time (or most operations on streams
for that matter) if the underlying GstOggMap was properly initialized.

oggstream: Add a default GstOggMap

Since the default value of a GstOggPad.map.map was 0 ... we would
end up using wrong functions from mappers() if the stream wasn't
initialized yet.

Instead of that, use a default blank/empty first entry.

rtspconnection: Allow setting a custom accept-certificate function for manually checking a TLS certificate for validity

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

ogmparse: Ensure we don't create bogus fractions

The clamping of the fraction denominator was bogus (it needs to
be >0)

oggdemux: Fix chain leak in push mode

In some corner cases we end up with the building chain not being
properly tracked (and therefore not properly freed).

Add a FIXME so it can later be fixed, but for now just fix the leak

vorbistag: Fix leak in error case

Don't leak the vendor_string on error cases

ogmparse: Error out if we can't figure out the format

In some cases we might not be able to figure out the fixed format
from the header.

Properly error out in those cases.

rtcpbuffer: Add XR type

RTCP XR provides supplements information of the report blocks
from SR and RR. This patch is for downgrading warnings when
XR is detected before implementing entire block types of RFC3611.

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

sdpmessage: do not append NULL value for session-level attr

If an attribute is defined without value, the generated text
should be 'a=key' rather than 'a=key:'. For media-level, it
has already been done.

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

Update disted orc file

For latest orc fixes

ogg: Don't add tags to empty taglist

typefind: Use GST_READ_UINT32 macros

Avoids undefined behaviour with shifting

typefind: Fix out-of-bound memory access

We were not checking the proper amount of available data in several
places

audioconvert: Fix empty mix matrix documentation typo

decodebin3: Allow configure output stream without reassign slot

By select-streams event, current implementation of decodebin3
supports deactivate output stream (i.e., decoder element)
in reassign slot(), but cannot activate any slot without track change.

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

playbin3: Use STREAMS_SELECTED message to update selected stream types

Application might choose only specific type among all available types
using select-streams event. In this case, it is desired that reconfigure
of playsink to clear unused stream path.

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

gst-play: Prevent disabling all streams

This would have bad effects :)

gst-play: Support track change on playbin3

* playbin3 does not support {current,n}-{audio,video,text}
properties, and they were replaced by GstStreams API.
So, GstStreams API and select-stream event should be used
for track change in case of playbin3.
see also https://bugzilla.gnome.org/show_bug.cgi?id=769079

* By using commend line option "--use-playbin3", gst-play will
use playbin3 regardless of "USE_PLAYBIN" env variable.

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

audioconvert: document passing an empty mix-matrix

check: Fix minor leak

oggdemux: Don't forget to reacquire lock when needed

Fixup to ef93130cf04143fcf675b05ce0d19eddfbac4e75

I overlooked the issue. There is a case when the lock is released
and we need to reacquire it

libgstvideo.def: add new definitions

videoencoder: add qos property

This new property control if the encoder base class should gather QoS
stats and if subclasses should use them by dropping late frames.

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

gst-plugins-base: gstaudiochannels: Handled buffer mapping failure

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

oggdemux: Don't drop sticky events

Previous commit was wrong. We should still send all events to the
pad (so that sticky events get attached to it and sent when pad
gets added).

oggdemux: Improve handling of EOS without source pads

We might have a chain to use, but it might not have any active pads

Properly detect that and send an error message on EOS

Revert "videoencoder: flush encoder in transition PAUSED->READY"

This reverts commit 877664a414a466cfcc71c79d28c470722408c9a7.

Revert "videodecoder: flush decoder in transition PAUSED->READY"

This reverts commit 6e9edc3031935ce8d6640d9774a7c093d20d4366.

Revert "audioencoder: flush encoder in transition PAUSED->READY"

This reverts commit 2dcdd13512e788797d03f9990994eecdd339ca0e.

Revert "audiodecoder: flush decoder in transition PAUSED->READY"

This reverts commit e7cf4c058ddd466a797623b5ddf162c263d07059.

win32: update exports file

oggdemux: Error out on EOS if we have no chains to use

There are not active and pending chains, if we get EOS we need to
inform the user via an error message

oggdemux: Don't double lock

The lock was already taken just before this block and is released after

videoencoder: flush encoder in transition PAUSED->READY

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

videoencoder: remove the lock from gst_video_encoder_flush

The lock is already taken before calling the flush method and can lead to
deadlock for some encoders that need to take the same lock from another
thread while flushing

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

videodecoder: flush decoder in transition PAUSED->READY

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

audioencoder: flush encoder in transition PAUSED->READY

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

audiodecoder: flush decoder in transition PAUSED->READY

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

videoencoder: implement QoS

It allows encoders to detect and drop input frames which are already
late to increase the chance of the pipeline to catch up.

The QoS logic and code is directly copied from gstvideodecoder.c.

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

videoencoder test: properly name the encoder variable

The element is an encoder so calling it 'dec' makes things confusing.

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

doc: Add some missing/malformed Since markers

audioconvert: allow empty mix matrix

When an empty mix matrix is passed, audio-channel-mixer
will now generate a (potentially truncated) identity matrix,
this replicates the behaviour of audiomixmatrix in first-channels
mode.

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

Reorder win32 defs

discoverer: output channel positions

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

discoverer: output whether the uri is live

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

API: gst_discoverer_info_get_live

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

API: gst_discoverer_audio_info_get_channel_mask

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

opusenc: Allow unpositioned channels.

The input will be treated as a set of mono channels,
and the channel-mapping-family will be set to 255
(no defined channel meaning)

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

audio-converter: remove unused mix_matrix private field

tests: appsink: fix compiler warning and typo in struct name

elements/appsink.c:624:3: warning: missing braces around initializer
with gcc 4.8.4

rtsp: Start implementing support for RTSP 2.0

Properly handle protocol version in the connection

Add the following headers types:
  * Pipelined-Request
  * Media-Properties
  * Seek-Style
  * Accept-Ranges

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

meson: remove vs_module_defs

GST_EXPORT should handle it.

meson: Add some missing args and dependencies in the gir generation

Use proper GtkDoc notation for NULL/FALSE/TRUE

meson: Add mssing GstBase-1.0 include in the gir generation

audio: Fix unit test after changed GstAudioStreamAlign constructor signature

audio: Add stream align API for getting timestamp at discont and number of samples since discont

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

audio: Add reverse playback support to GstAudioStreamAlign

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

audio: Add helper object for audio discontinuity detection and sample alignment

This is the same code that is in decklinkaudiosrc, audioringbuffer,
audiomixer and various other places. Have it once instead of copying it
everywhere.

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

alsasink: Fix Memory leak in payload not succuss case

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

audioconvert: [API]: expose mix-matrix property.

This obsoletes audiomixmatrix

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

[API]: GST_AUDIO_CONVERTER_OPT_MIX_MATRIX

Taken from audiomixmatrix, credits to Vivia Nikolaidou

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

[API]: gst_audio_channel_mixer_new_with_matrix

+ Refactor previous constructor to call on that new constructor

+ Reimplement is_passthrough to strictly check whether the matrix
  is an identity matrix, comparing channel-masks was incorrect:
  the mixer may be remixing from a list of positions to the same
  list of positions, but ordered differently, and reciprocally,
  the mixer may be remixing from a list of positions to another
  list of positions identically ordered

+ Remove unused tmp field, must have been a refactoring leftover

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

audioconvert: refactor format removal.

remove_format_info was a bit confusing to read, this removes
it in favor of standard gst_caps_map_in_place calls.

This no longer simplifies the resulting caps, but I
consider this should be the job of basetransform.

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

appsink: on drain wait for buffers to be consumed

So that an upstream element can claim all buffers to return to its buffer pool.

Added unit test 'test_query_drain'
  make elements/appsink.check

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

appsink: also clear preroll buffer in _pull_sample

If someone calls gst_app_sink_try_pull_sample they are
probably no longer interested in any preroll samples.

Useful if the user has not registered a preroll appsink callback.

Also added unit test 'test_do_not_care_preroll'
  make elements/appsink.check
that fails without this patch.

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

appsink: unref preroll buffer upon pull

There is no reason for appsink to hang onto the preroll buffer.
If needed, the application can just keep a ref on this buffer
after calling gst_app_sink_try_pull_preroll.

Also added unit test 'test_pull_preroll'
  make elements/appsink.check

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

appsink: rename GstBuffer *preroll to preroll_buffer

priv->preroll can be confused with basesink_class->preroll

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

tests: playbin-complex: skip test that needs oggdemux if it's not available

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

Request minimum buffer even if need_pool is FALSE

When tee is used, it will not request a pool, but still it wants to
know how many buffers are required.

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

video-filter: Support allocation pool with pool object

This is used to indicate upstream the requirement in buffers
while no buffer pool can be provided. In this case, only
configure the pool with caps/size/min/max if we have caps,
which we only parsed when there was no allocation pool.

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

video/affinetransformationmeta: use the provided doc comment for the struct

playbin: ref_sink() sink rather than pointer to sink

Automatic update of common submodule

From 29046b8 to 3f4aa96

videoutilsprivate: fill more video fields.

__gst_video_element_proxy_caps is called by
__gst_video_element_proxy_getcaps with caps set to the caps
allowed downstream. As we didn't set colorimetry or
chroma-site on the resulting caps, upstream considered it
possible to use whatever values it wanted, leading to
not negotiated errors later on.

As the description for that function is:
"Takes caps and copies its video fields to tmpl_caps",
it seems legitimate to set these fields there.

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

appsrc: handle duration query only if the property was set

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

decodebin3/urisourcebin: Switch to actual EOS events internally

Use the intended sequence for re-using elements:
* EOS
* STREAM_START if element is to be re-used

This avoids having elements (such as queue/multiqueue/queue2) not
properly resetting themselves.

When delaying EOS propagation (because we want to wait until all
streams of a group are done for example), we re-trigger them by
first sending the cached STREAM_START and then EOS (which will
cause elements to re-set themselves if needed and accept new
buffers/events).

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

meson: hide symbols by default unless explicitly exported

pbutils: sprinkle more GST_EXPORT

libs: add some more missing GST_EXPORT