pbutils: fix some GIR annotations

Mostly related to out and array parameters

video: fix some GIR annotations

Mostly related to out and array parameters

audio: fix some GIR annotations

Mostly related to out and array parameters

rtsp: fix some GIR annotations

Mostly related to out and array parameters.

rtp: fix some GIR annotations

Mostly related to out and array parameters.

audioaggregator: fix filtered getcaps

In the situation described in
https://bugzilla.gnome.org/show_bug.cgi?id=795397,

downstream_caps consists of two structures, the first with
the preferred rate, if at all possible (44100), the second
containing the full range of allowed rates, as audioresample
correctly tries to negotiate passthrough caps.

As audioaggregator cannot perform rate conversion, it wants
to return a fixated rate in its getcaps implementation,
however it previously directly used the first structure in
the caps allowed downstream, without taking the filter into
consideration, to determine the rate to fixate to.

With this, we first intersect our downstream caps with the
filter, in order not to fixate to an unsupported rate.

tests: videodecoder: Fix a minor c&p mistake

This completes what commit 0de0f3b2c should have already done. :)

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

tests: videodecoder: Fix a minor c&p mistake

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

video: Safe guard info->finfo when mapping a frame

Not sure how that slept through but that case is possible and leads
to segfaults in any code path of the function right now.

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

encoding-profile: Handle escaped '\:' in caps describing profiles

Otherwise it won't be possible to specify some profiles such as

    video/x-h264,profile=(string)high-4:4:4

With this patch, we can do

    video/x-h264,profile=(string)high-4\:4\:4

meson: gl: fix 'invalid keyword argument' meson warnings

Required is not a valid kwarg for cc.has_header()

rtpbasedepayload: condition the sending of gap events

The default implementation for packet loss handling previously
always sent a gap event.

While this is correct as long as we know the packet that was
lost was actually a media packet, with ULPFEC this becomes
a bit more complicated, as we do not know whether the packet
that was lost was a FEC packet, in which case it is better
to not actually send any gap events in the default implementation.

Some payloaders can be more clever about, for example VP8 can
use the picture-id, and the M and S bits to determine whether
the missing packet was inside an encoded frame or outside,
and thus whether if it was a media packet or a FEC packet,
which is why ulpfecdec still lets these lost events go through,
though stripping them of their seqnum, and appending a new
"might-have-been-fec" field to them.

This is all a bit terrible, but necessary to have ULPFEC
integrate properly with the rest of our RTP stack.

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

appsink: Reuse sample object in pull_sample if possible

Simple optimization to reduce memory allocations.

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

parsebin: Post STREAM_COLLECTION on EVENT_STREAM_COLLECTION

Otherwise decodebin won't get notified about STREAM_COLLECTION comming
from the sources and thus will never get informored about it. Without
being informed about the stream collection decodebin won't be able to
select any streams. It ends up not creating any output for the streams
defined from outside parserbin.

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

egl: fix build when using RPi EGL

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

meson: move some dependency checks to the top level

The newly-added build definitions for test/icles relied
on dependencies that were only defined when the examples
are enabled, thus breaking meson build -Ddisable_examples=true

video: Don't corrupt caption GstMeta

the meta initialization function is provided *after* the base implementation
fields have been set so do *NOT* reset them otherwise it would result
in corrupted GstMeta.

Instead explicitely set our fields to the default values we actually want.

Automatic update of common submodule

From f0c2dc9 to ed78bee

gl: Define default value for GST_GL_HAVE_WINDOW_GBM

Thus, silent compiler's warning:
"GST_GL_HAVE_WINDOW_GBM" is not defined, evaluates to 0 [-Wundef]

tag: fix some GIR annotations

Mostly related to out and array parameters.

tests: make getaddrinfo suppression more generic

So it works for me on debian sid.

valgrind supps: ignore leaks where pango is the first frame

valgrind supps: ignore intentional leak in GL test

valgrind supps: conditional moves in nvidia-glcore

Not much we can do here

valgrind supps: getaddrinfo leaks when resolving addresses

https://lists.debian.org/debian-glibc/2016/03/msg00243.html

audioaggregator: unref converted buffer after gst_buffer_replace

alsamidisrc: unref buffer_list before early return

tests/audiomixer: fix caps leak

valgrind supps: ignore bash leaks

gst_install_plugins_* can instantiate bash

urisourcebin: Avoid unreffing a pad we are not owning

expose_output_pad takes ownership of the pad.

uridecodebin3: free_play_items when READY_TO_PAUSED failed.

We will never go through the PAUSED_TO_READY transition if
that is the case, and thus never free the play items.

playbin3: fix leak of recursive mutex

valgrind suppressions: ignore XInitThreads leaks

As we load that symbol dynamically, valgrind gets confused
when it leaks and reports the leak against an unrelated library
and an unknown (??) symbol.

To address that, put the loading and calling of that symbol
in a separate function, and ignore any malloc leak happening
in that function.

valgrind supps: ignore leak in XextCreateExtension

Searching for that specific function in other suppression files
on the Internet shows a few projects ignoring it

eg: https://github.com/XCSoar/XCSoar/blob/master/valgrind.suppressions

vorbisenc: do not map input buffer in WRITE mode

meson: port tests/icles

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

audiovisualizer: Only fixate pixel-aspect-ratio if the field exists

It's optional.

appsrc: Documentation fix in gst_app_src_push_sample_internal

Make it clear that it doesn't take ownership of the sample.

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

audiovisualizer: Fixate pixel-aspect-ratio to the closest value to 1/1

playback: Add support for CEA 608/708 CC overlay elements

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

video: Add support for VANC and Closed Caption

This commits add common elements for Ancillary Data and Closed
Caption support in GStreamer:

* A VBI (Video Blanking Interval) parser that supports detection
  and extraction of Ancillary data according to the SMPTE S291M
  specification. Currently supports the v210 and UYVY video
  formats.

* A new GstMeta for Closed Caption : GstVideoCaptionMeta. This
  supports the two types of CC : CEA-608 and CEA-708, along with
  the 4 different ways they can be transported (other systems
  are super-set of those).

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

pbutils: Add Closed Caption caps description

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

audiobasesrc: posting errors should be always be safe

Don't try to signal an error in the ringbuffer if it hasn't been
allocated yet.

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

audioringbuffer: Don't spam INFO for every buffer

This makes GST_DEBUG=4 outputs too spammy, and such frequent messages
are meant to go into DEBUG or TRACE anyway.

tests: Enable tests for videodecoder

The tests pass fine here so don't see any reason to keep them disabled.

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

encodebin: Always respect encoding profile preset factory name

And fail if it is not present.

encodebin: Inform about missing elements in the "text" error field

Instead of the debug field, that information should be directly
exposed to end users.

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

video: Fix/Add section documentation

docs/libs: The big spring cleanup

* Explicitely specify which headers aren't to be included in gtkdoc-scan
  This is essentially all the headers that are not installed and only
  for internal/local usage. This also includes the orc-generated headers.
* Remove all symbols/sections that are no longer present (due to accurately
  scanning only the headers we need).
* Add or expose sections which weren't previously exposed
* Make sure the "unified" library headers (ex: gst/video/video.h) are used
  everywhere applicable. Only use the specific headers where applicable
  (such as the GL-implementation-specific objects)
* Add all documentation which was not previously exposed in the right sections
* Update 'types' file to get as many runtime information as possible

This brings down the number of unused symbols to 15 (from over 300).

libs: Documentation cleanup

* Fix wrong naming, wrong types and typos
* Add missing sections
* Add missing documentation for entries
* Explicitely mark private structure entries
* Remove items that never existed

tools: Fix discoverer deprecation

The only information provided by "misc" was the missing plugins which
is already handled with another API

libs: Add deprecation guard

Those functions have been marked as deprecated

check: Remove old structures from ABI check

Just in case someone still tries to run it on 32bit osx

videorate: In reverse playback mode, don't output the first buffer with ts=segment.stop

Instead go backwards before segment.stop based on the framerate or the
next buffers end timestamp. Otherwise the first buffer will usually be
dropped because outside the segment.

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

gl: pick up GstVideo-1.0.gir from local build dir

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

tools: play: fix leaving STDIN in non-blocking mode after exit

gst-play-1.0 sets STDIN to non-blocking mode to have the input
characters read as soon as they arrive.

However, when gst_play_kb_set_key_handler() gets called from
restore_terminal() it forgets to restore the STDIN blocking status.

This can result in broken behavior for cli command executed in the same
terminal after gst-play-1.0 exited.

It turns out that putting STDIN in non-blocking mode is not even the
proper way to achieve the desired effect, instead VMIN and VTIME in
struct termios should be set to 0.

Let's do that, and don't mess with the STDIN blocking mode now that it's
not necessary.

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

encoding-profile: fix gi annotation for gst_encoding_profile_find

gl: Disable glmixerbin for the time being too

Otherwise we have one copy in gst-plugins-bad and one (unused) here,
which makes static linking unhappy.

encodebin: Use gst_object_unref() instead of g_object_unref()

... for better debugging (e.g., tracing refcount)

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

audio-aggregator: Check return values

And copy over already-parsed information

CID #1427140

meson: libs: use gnome.mkenums_simple() to generate enumtypes files

This way we no longer need custom wrapper scripts or template files.

gl/gbm: Initialize window handle (= gbm surface) like other window systems

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

viv-fb: Include gstglfuncs.h to fix cross compilation errors

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

dmabufallocator: Fix build if LINUX_DMA_BUF_H is missing

allocators: Add DMABuf synchronization

When doing CPU Access, some architecture may require caches to be
synchronize before use. Otherwise, some visual artifact may be
visible, as the CPU modification may still resides in cache.

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

video: Set correct value in g-i annotations for tile related mask constants

video: Include gstvideoaffinetransformationmeta.h in video.h

pbutils: Include gstaudiovisualizer.h in pbutils.h

Back to development

Release 1.14.0

Update docs

oggstream: protect against out-of-bounds read

We need at least 17 bytes of data for a valid flac header

oss-fuzz #6974

tests: audiomixer: remove unistd.h include

Not needed and breaks the build with MVSC.

audiostreamalign: Mark the whole type as new in 1.14

pkg-config: Add orc-0.4 to Requires.private of gstreamer-audio and gstreamer-video

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

Release 1.13.91

Update docs

meson: install new audio-prelude.h

docs: update api decorators to ignore

video: GST_EXPORT -> GST_VIDEO_API

We need different export decorators for the different libs.
For now no actual change though, just rename before the release,
and add prelude headers to define the new decorator to GST_EXPORT.

tag: GST_EXPORT -> GST_TAG_API

We need different export decorators for the different libs.
For now no actual change though, just rename before the release,
and add prelude headers to define the new decorator to GST_EXPORT.

sdp: GST_EXPORT -> GST_SDP_API

We need different export decorators for the different libs.
For now no actual change though, just rename before the release,
and add prelude headers to define the new decorator to GST_EXPORT.

rtsp: GST_EXPORT -> GST_RTSP_API

We need different export decorators for the different libs.
For now no actual change though, just rename before the release,
and add prelude headers to define the new decorator to GST_EXPORT.

rtp: GST_EXPORT -> GST_RTP_API

We need different export decorators for the different libs.
For now no actual change though, just rename before the release,
and add prelude headers to define the new decorator to GST_EXPORT.

riff: GST_EXPORT -> GST_RIFF_API

We need different export decorators for the different libs.
For now no actual change though, just rename before the release,
and add prelude headers to define the new decorator to GST_EXPORT.

pbutils: GST_EXPORT -> GST_PBUTILS_API

We need different export decorators for the different libs.
For now no actual change though, just rename before the release,
and add prelude headers to define the new decorator to GST_EXPORT.

gl: GST_EXPORT -> GST_GL_API

We need different export decorators for the different libs.
For now no actual change though, just rename before the release,
and add prelude headers to define the new decorator to GST_EXPORT.

fft: GST_EXPORT -> GST_FFT_API

We need different export decorators for the different libs.
For now no actual change though, just rename before the release,
and add prelude headers to define the new decorator to GST_EXPORT.

audio: GST_EXPORT -> GST_AUDIO_API

We need different export decorators for the different libs.
For now no actual change though, just rename before the release,
and add prelude headers to define the new decorator to GST_EXPORT.

app: GST_EXPORT -> GST_APP_API

We need different export decorators for the different libs.
For now no actual change though, just rename before the release,
and add prelude headers to define the new decorator to GST_EXPORT.

allocators: GST_EXPORT -> GST_ALLOCATORS_API

We need different export decorators for the different libs.
For now no actual change though, just rename before the release,
and add prelude headers to define the new decorator to GST_EXPORT.

audioaggregator: fix channel-mask negotiation

When outputting more than two channels, a channel-mask has to be
specified in the output caps.

We follow the same heuristic as other cases, when downstream
does not specify a channel-mask, we use that of the first
configured pad, and if there was none we generate a fallback
mask.

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

playback: Remove subtitle-encoding property on uridecodebin3

For now, until we found a more future-proof solution

docs: video: fix typo and add since marker

docs: libs: add another missing symbol

docs: add video region of interest add/get parameter api to docs

audio: Add audioaggregator.h in audio.h

docs: add GstPhysMemoryAllocator to docs

Revert "alsasrc: lock calls to snd_pcm_delay() with mutex as in alsasink"

This reverts commit 4f88125b3ddc15367d528e850cc27739f5bb27fa.

This seems to have undesirable side-effects and needs more
investigation first.

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

gl: only enable gbm backend if gudev is available

gudev/gudev.h is included unconditionally, so only enable
the gbm backend if gudev was actually found. This also
matches the meson build behaviour.

Should fix build on GNOME SDK builder.