videoaggregator: Remove unlock() if set_info() fails

There is not mutex locked here that needs to be unlocked.

sys: Fix caps memory leak from pad template creation

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

gme: Forward other non-EOS events downstream

Allow most events through as-is so at least we don't
warn about missing stream-start. Drop segment and caps,
as those are sent internally.

gst: Don't declare variables inside the for loop header

This is a C99 feature.

dashdemux: Don't declare variables inside the for loop header

This is a C99 feature.

gme: Free decoder instance when going from PAUSED to READY

We only ever cleaned it up before on errors.

msdk: add mjpeg decoder

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

msdk: add mjpeg encoder

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

msdk: Add H.265 decoder

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

msdk: Add H.264 decoder

The decoder only supports system memory output presently.

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

adaptivedemux: Avoid using unset GError

gst_uri_handler_set_uri() doesn't guarantee the GError will be set
if it returns FALSE.

ttmlrender: Fix segfault on _render_text_region()

Fix invalid memory access

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

ttmlparse: Transform caps event

For autoplugging, decodebin/parsebin refers to the caps event.

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

qml: Fix egl being deinitialized on display cleanup

Use the with_egl_display() variant in order to not destroy the
EGLDisplay on destruction.

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

dvbsrc: fix unconditional use of FEC 2/5

It is not defined for < v5 minor 7

dvbsrc: add delivery system to missing modulation check message

A tuning operation can spawn multiple checks. Being
able to differentiate between them makes debugging
easier.

dvb/parsechannels: drop trailing whitespace

player: Don't try to cache the GQuark for GST_PLAYER_ERROR

This is potentially racy (in the unlikely scenario that we get two
first-time calls to gst_player_error_quark() at the same time). This
should not impact anything in terms of performance since it's only on
the error path.

The call itself could just be inlined by making GST_PLAYER_ERROR be
defined to the g_quark_from_static_string() call, but this feels ugly
from an API perspective.

fdkaac: fix the build for older fdkaac versions

Support for 7.1 was added in 0.1.14.

qml/item: also unref the display on destruction

Leaking objects (and a thread!) is never a good idea.

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

tests/examples/qmlsink: scope QApplication/Engine

So they are destroyed before gst_deinit() is run and the leaks tracer
doesn't show false-positives.

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

chromaprint: Fix compilation with chromaprint 1.4

It takes const int16_t * as argument now, while before it was void *. To
make this work with both versions without #ifdef, we omit the const.

dvbsrc: drop needless wrapper around DVB API's dvb_diseqc_master_cmd

Its only purpose was to hold a wait time that was always 0

dvbsrc: drop supperfluous wait in diseqc_send_msg()

It is always zero.

vmncdec: Sanity-check rectangle sizes a bit more thorough

The x/y coordinates could already be bigger than the configured
width/height, and adding the rectangle width/height could cause an
overflow.

dvbsrc: fix unconditional use of SYS_DVBC_ANNEX_C

It is not defined for < v5 minor 6

dvbsrc: fix value for DVB-C annex B field in adapter structure

rawvideoparse: fix typos in property description

dtls: Set openssl's threadid the 1.0.x way

For pre-1.1.x openssl, a callback to set the thread id needs to be
provided to openssl. In 0.9.x the thread id was an unsigned long. In
1.0.x it was expanded to be able to hold a void*. Here we change to use
the 1.0.x API so that the thread id can always hold a GThread*, even on
platforms like msvc x64 where unsigned long is only 32 bits.

All of this is still #ifdef'd out of existence when building with
openssl 1.1.x or later which changed the thread API again, and does not
need a thread id callback.

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

segmentation: Port to GstOpencvVideoFilter base class

grabcut: Port to GstOpencvVideoFilter base class

retinex: Port to GstOpencvVideoFilter base class

This allow reducing slightly the code. It also fixes a bug caused by
in-place buffer being mapped and never unmapped.

Automatic update of common submodule

From d015674 to 39ac2f5

opencv/meson: check presence of required opencv_contrib headers

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

edgedetect: Avoid extra copy of the output image

Simply produce the result into the output image provided by the base
class. This avoid useless copy. This also removes unchecked buffer map
and ensure that GstVideoMeta is respected (for stride mostly).

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

edgedetect: Remove redundant CAPS event handler

There is already a cv_set_caps() implemented, so just do the
entire work there.

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

cvlaplace: Avoid extra copy of the output image

Simply produce the result into the output image provided by the base
class. This avoid useless copy. This also removes unchecked buffer map
and ensure that GstVideoMeta is respected (for stride mostly).

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

cvlaplace: Remove redundant CAPS event handler

There is already a cv_set_caps() implemented, so just do the
entire work there.

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

cvsobel: Avoid extra copy of the output image

Simply produce the result into the output image provided by the base
class. This avoid useless copy. This also removes unchecked buffer map
and ensure that GstVideoMeta is respected (for stride mostly).

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

cvsobel: Remove redundant CAPS event handler

There is already a cv_set_caps() implemented, so just do the
entire work there.

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

opencv: Remove uneeded buffer writability check

When set to in_place, the BaseTransform class will guaranty that buffers
passed to transform_ip() function are writable.

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

gstglupload: relax EGL context check

Check for GST_GL_PLATFORM_EGL enum instead of type GstGLContextEGL.

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

gstglcontext: is_shared should return FALSE if no group

If a sub class of GstGLContext does not create a group
then it currently crashes:

0 g_atomic_int_get (&share->refcount)
1 _context_share_group_is_shared (context->priv->sharegroup)
2 gst_gl_context_is_shared
3 _default_set_sync_gl

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

meson/ttml: Add missing math library dependency

This code call 'round' which may require linking against the math library
on some compilers.

kmssink: post error message if start() fails

It is expected to post an error message in the bus if the device cannot
be started.

kmssink: configure display mode during set_caps

Configure the display mode when setting the negotiated caps instead of
during showing the first frame.

A framebuffer is required to set the mode. Allocate a buffer object
according to the negotiated caps and use it to set the mode. This buffer
object cannot be freed until another page flip happened on the crtc
(i.e., until the first frame is rendered).

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

Signed-off-by: Víctor Manuel Jáquez Leal <vjaquez@igalia.com>

kmssink: add parameter force-modesetting

The force-modesetting parameter forces the kmssink to ignore already
configured display modes, to configure the display mode itself and use
the base plane for output.

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

kmssink: allow only supported resolutions

If the input buffers have a different size than the display, the frames
would have to be scaled or positioned on the display. The kmssink cannot
decide which behaviour would be appropriate for which use case.

In order to avoid scaling or positioning of the input stream, allow only
the supported connector resolutions in the sink caps.

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

Signed-off-by: Víctor Manuel Jáquez Leal <vjaquez@igalia.com>

kmssink: set mode based on framebuffer configuration

Displays usually support multiple modes. Therefore, the kmssink should
not only support the preferred mode, but any mode that is supported by
the display.

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

kmssink: add mode setting and base plane rendering

The kmssink assumed that the mode was already set by another application
and used an overlay plane for displaying the frames.

Use the preferred mode of the monitor and render to the base plane if
the crtc does not have a valid mode.

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

Signed-off-by: Víctor Manuel Jáquez Leal <vjaquez@igalia.com>

interlace: Writing field-order into the src caps

Writing top-field-first vs bottom-field-first into the src caps

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

docs: add aggregator _get_type functions for signals + properties

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

glmemoryegl: remove spurious #if GST_GL_HAVE_DMABUF conditions

8c2118823b3d42840cc6f48cbdc0e1b342f90b80 had some incorrect preprocessor
conditions that aren't actually needed.  Remove them.

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

gst: Fix caps leak

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

opencv: check presence of required opencv_contrib headers

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

opencv: add opencv image format documentation

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

opencv: account for sparse/padded formats when converting caps to cv image type

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

opencvvideofilter: Don't parse the caps twice

The GstVideoFilter already provide caps parsed into GstVideoInfo. Avoid
doing that twice by splitting the helper.

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

opencvfilter: Properly port to GstVideoFilter

This is a subblass of VideoFilter but yet does not use any of it's
features. This also fixes issue in case the incoming images have custom
strides as the VideoMeta is no longer ignored.

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

mssdemux: fix meson build

Follow-up of 73721ad4e9e2d32e1c8b6a3b4aaa98401530e58a.
Forgot to add the fragment parser in the meson build file.

mssdemux: improved live playback support

When a MSS server hosts a live stream the fragments listed in the
manifest usually don't have accurate timestamps and duration, except
for the first fragment, which additionally stores timing information
for the few upcoming fragments. In this scenario it is useless to
periodically fetch and update the manifest and the fragments list can
be incrementally built by parsing the first/current fragment.

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

decklink: Fix compiler warning

gstdecklink.cpp: In member function ‘virtual HRESULT GStreamerDecklinkInputCallback::VideoInputFrameArrived(IDeckLinkVideoInputFrame*, IDeckLinkAudioInputPacket*)’:
gstdecklink.cpp:766:34: error: ‘base_time’ may be used uninitialized in this function [-Werror=maybe-uninitialized]
         capture_time -= base_time;
                                  ^

codecparsers: h264parse: improve documentation commentary

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

codecparsers: h264parse: docs spellcheck

Spell checks, correct capitalization and some rewording to
better fit the terms used in the H.264 spec.

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

schro: #include math.h before schroedinger headers

The schroedinger headers unconditionally #define over C99's rint when
compiling with msvc which messes up the later inclusion of math.h.
Including math.h before schroedinger headers avoids getting syntax
errors in math.h

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

meson: Rework opencv plugin checks

a) Use get_pkgconfig_variable() to get the opencv prefix
b) Place an upper limit on the opencv version
c) Ensure that headers are available

(b) and (c) just copy what the configure.ac checks do.

mpegtssection: Fix off-by-one in PMT parsing

mpegtssection: Don't assert if the given section length is longer than the PMT actually is

Instead error out cleanly and just assert that we didn't read more than
the available data.

decklink: Correctly set top-field-first/bottom-field-first

First of all, all the HD and UHD modes should be top-field-first, as
also returned by the Decklink mode iterator API.

Then we should include the caps field "field-order" in the caps of the
source (not the sink due to negotiation problems with optional fields).

And finally we should set the TFF flag on interlaced buffers that are
top-field-first.

decklinksrc: Add property to drop frames that are marked as having no input signal

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

decklinksrc: Add property to skip the first N seconds of capture

On some hardware the first few frames are bogus and not very useful.
Their timestamps are off, they have no timecodes, or there are spurious
black frames / no-signal frames. After a few frames this stabilizes
though.

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

decklinksrc: Set GAP flag on buffers that are captured without signal

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

decklinksrc: Keep track of a window of capture/stream time mappings and calculate relationship

Based on this we calculate the actual capture time, which should get us
rid of any capturing jitter by averaging it out.

Also add a output-stream-time property which forces the elements to
output the stream time directly instead of doing any conversion to the
pipeline clock. Use with care.

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

decklinksrc: Stop using the "hardware" timestamps and directly use the pipeline clock

The hardware timestamps have no relation to when frames were produced,
only when frames arrived somewhere in the hardware. Especially there is
no guarantee that audio and video will have the same hardware timestamps
although they belong together, and even more important: the rate with
which the hardware timestamps increase is completely unrelated to the
rate with which the frames are captured!

As such we can as well use the pipeline clock directly and stop doing
complicated calculations. Also as a side effect this allows now running
without any pipeline clock, by directly making use of the stream times
as reported by the driver.

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

rawparse: Whitespace cleanup

dataurisrc: remove plugin, moved to core

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

m3u8: Mark uri_join() function as static

We don't really want to export this, especially not without any
namespacing. This can cause problems for static builds.

tsdemux: Post an error message on the bus if we activated a program without pads

Otherwise decodebin and applications are going to wait forever for pads
to appear, which is never going to happen.

Happens on this stream, which probably just need support for it added:
https://samples.mplayerhq.hu/3D/AVC_codec_in_m2ts_not_recognized/AVC_codec_not_recognized_right_video_sample.m2ts

glwindow: move g_main_context_push/pop_thread_default() to run()

Calling g_main_context_push_thread and then g_main_context_invoke()
(used by gst_gl_window_send_message_async()) in the same thread will
cause the invoked function to run immediately instead of being delayed.

This had implications for the creation of the OpenGL context not waiting
until the main loop had completely started up and as a result would
sometimes deadlock in short create/destroy scenarios.

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

glcontext: fix race between creation/shutdown

626bcccff96f624f59c5212b3e21e472240171fd removed some locks that
allowed the main loop quit to occur before the context was fully
created.

2776cef25d2a98668b73272aecfe77e684e6627e attempted to readd them but
missed the scop of the quit() call.

Also remove the use of g_thread_join() as that's not safe to use when
it's possible to lose the last reference from the GL thread.

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

rawbaseparse: Fix output buffer size trimming

For frame->buffer, baseparse is doing that automatically for us. For
frame->output_buffer it doesn't and assumes that the subclass is already
doing that. Consistency!

dash: Fix stripping of space at the beginning/end of durations

The way how strchr() was called here, it could easily read after the end
of the string. Use g_ascii_isspace() instead.

Detected by asan in the unit test.

opencv: Make the OpenCV plugin optional in the meson build

rawparse: Properly align raw audio/video output buffers

That is, aligned to the basic type for audio and to 32 bytes for video.
Fixes crashes if the raw buffers are passed to SIMD processing functions.

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

opencv: Enable in meson build

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

common: use https protocol for common submodule

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

mxfdemux: Post an error message if we got EOS before having any source pads

Otherwise, sending an EOS event has no effect and the application waits
forever for something to happen.

h264parse: Fix indentation

h265parse: Fix assertion when encountering an invalid NALU

Error out properly instead.

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

h264parse: Fix assertion when encountering an invalid NALU

Error out properly instead.

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

mpegtssection: Add more section size checks

The smallest section ever needs to be at least 3 bytes (i.e. just the short
header).
Non-short headers need to be at least 11 bytes long (3 for the minimum header,
5 for the non-short header, and 4 for the CRC).

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

mpegtssection: Fix PAT parsing

Use the estimated number of programs for parsing. Avoids over-reading.

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

hlsdemux: Handle GST_SEEK_TYPE_NONE seek

When one is only updating the "stop" position (i.e. non-flushing seek,
with GST_SEEK_TYPE_NONE on the "start" (or stop in reverse) position),
we only need to store those values instead of moving the current position.

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

ext: Fix various compiler warnings with clang

wldisplay.c:179:15: error: comparison of unsigned enum expression < 0 is always false [-Werror,-Wtautological-compare]
  if (shm_fmt < 0)
      ~~~~~~~ ^ ~

gstsegmentation.cpp:419:40: error: implicit conversion from 'int' to 'char' changes value from 255 to -1 [-Werror,-Wconstant-conversion]
          filter->cvFG->imageData[j] = 255;
                                     ~ ^~~

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

gl/x11: Remove unused static function

mxfdemux: Fix up another size check and prevent allocating too much memory

mxfdemux: Handle zero-sized KLV items better and don't assert on them

mxfdemux: Fix various logic errors in error handling of footer partition parsing

mxfdemux: Don't try parsing empty metadata items

mxfmetadata: Add missing NULL check