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

gldownload: fix element description (was "OpenGL uploader")

mxf: Remove mxfdms1.[ch] from the build for now

It's completely unused currently but might be used in the future.

mxfdemux: Add various missing bounds checks, NULL checks and integer overflow protection for error cases

decklinkvideosrc: Free timecodes when done with a frame

timecodestamper: Remove all existing timecode metas if requested to override existing

mpegtssection: Don't free empty streams

Also avoids a useless assertion

mpegtspacketizer: Don't add existing values to group

If the last value is already identical, there is no need in adding it
yet-another-time

kmssink: include drm.h after stdlib.h

As drm.h is internal to libdrm, it expects to have already included
stdlib.h.

kms: include stdlib.h

drm.h does not include all what it needs to compile, in particular
stdlib.h which defines size_t

mpdparser: Don't set variable just to overwrite it again later

We set it to TRUE here, but later we set it to TRUE again anyway if the
parsing actually succeeded at this point. Let's keep the second one.

CID 1374360.

wldisplay: Don't compare a guint with < 0

-1 is the "invalid" value, compare to it directly.

CID 1394490.

audiobuffersplit: New element that splits raw audio buffers into equal-sized buffers

This is useful e.g. if audio buffers should be exactly the duration of a
video frame, or if a audio buffers should never be too large because of
latency constraints.

The element is taking a fractional buffer duration, to allow working
with e.g. 1001/30000 as output duration and it accumulates rounding
errors in the buffer durations and compensates for them by making some
buffers one sample larger than the others.

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

kmssink: remove dependency on libkms

libkms should not be used, because it imposes limitations on the DRM
API, especially regarding bpp and stride. Instead the DRM IOCTL should
be used directly.

Switch from libkms to the IOCTL interface. Set bpp and height for
framebuffer allocation to properly handle planar video formats.

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

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

mimic: remove ancient codec

This was used by MSN messenger in prehistoric times, it's safe
to say no one needs or wants this any more these days. For
decoding old recordings there's still a decoder in ffmpeg.

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

h265parse: Ensure codec_data has the required size when reading number of NAL arrays

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

h264parse: Ensure codec_data has the required size when reading number of SPS

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

gl/contextglx: error out on more invalid display/window combinations

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

gl: add necessary context_egl.h #includes to the source files

gl/memoryegl: remove access to private header gstglcontext_egl.h

It's been removed and thus compiling anything against GstGLMemoryEGL
would error with:

In file included from gstomxvideodec.c:41:0:
usr/include/gstreamer-1.0/gst/gl/egl/gstglmemoryegl.h:32:41: fatal error: gst/gl/egl/gstglcontext_egl.h: No such file or directory
 #include <gst/gl/egl/gstglcontext_egl.h>
                                         ^

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

mpegtspacketizer: Reset values before parsing each section

mpegtspacketizer: Make sure in all cases that we have enough data left for reading a section start

In some cases we ended up parsing sections without.

opencv: forward declare opencv types

Forward declare opencv types to avoid exposing them in the API.

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

interfaces: photography: clean up header indentation

mpdparser: Modify return of the function for loading external resources to void

gst_mpd_client_fetch_on_load_external_resources() never ever return
FALSE due to modified external xml loading functions.

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

mpdparser: Handle invalid external xml link for SegmentList element

Ignore invalid xml link for SegmentList likewise external Period without error.

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

mpdparser: Handle invalid external xml link for AdaptationSet element

Ignore invalid xml link for AdaptationSet likewise external Period without error.

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

mpdparser: Handle invalid external xml link for Period element

Section 5.3.3 in ISO/IEC 23009-1:2014 defines that invalid references
(e.g., invalide URI or cannot be resolved) specified by "@xlink:href" attribute
shall be removed. That means, we should play it without error,
and just ignore the corresponding element.
It's similar to "urn:mpeg:dash:resolve-to-zero:2013".

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

gl: add gstreamer-video to pkg-config path

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

tests: fix lib and cflags order in Makefile.am

pnm: add misssing continuation slash at end of line
vp8parser: no need to link codecparsers twice
dash_demux, mssdemux, hlsdemux: fix lib/flags ordering - local
libs and flags must come first, then base before core before rest

dashdemux: Fix leak in gst_dash_demux_stream_free

Free parsed moof and sync samples with _stream_free().

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

wayland: avoid prefix clash in combined pkg-config check

Adittionally, remove leftover WAYLAND_DRM_ include flags var

Fixes building of ext/wayland/* broken by dfd86de67a8c04f6d1

Related to:

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

vc1parse: Mark pad as needing reconfiguration again if it failed

And return FLUSHING instead of NOT_NEGOTIATED on flushing pads.

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

vtdec: Mark pad as needing reconfiguration again if it failed

And return FLUSHING instead of NOT_NEGOTIATED on flushing pads.

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

videoaggregator: Mark pad as needing reconfiguration again if it failed

And return FLUSHING instead of NOT_NEGOTIATED on flushing pads.

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

ttmlrender: Mark pad as needing reconfiguration again if it failed

And return FLUSHING instead of NOT_NEGOTIATED on flushing pads.

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

assrender: Mark pad as needing reconfiguration again if reconfiguration failed

Also return FLUSHING instead of NOT_NEGOTIATED on flushing pads.

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

iqa: Add to DIST_SUBDIRS

meson: add_global_arguments -> add_project_arguments

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

meson: wayland: wl_scanner is required: false

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

po: Remove remaining reference to videomeasure

decklinkvideosrc: Issue "no signal" warnings

When a frame is found to not have an associated input source (cable
unplugged, wrong mode selected), an element warning will be issued. When
the next frame in the stream is found to have an input source selected
(e.g. cable replugged), an element info will be issued.

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

videomeasure: Remove old unported videomeasure plugin as it is now replaced by IQA

iqa: Error out when trying to compare video stream with different sizes

We do not handle it yet and we should let the user know.

iqa: Add meson build support

Adds a new Image Quality Assessment plugin.

It only offers one metric for now, "dssim", available if
https://github.com/pornel/dssim was installed on the system
at the time the plugin was compiled.

The spearman correlation for dssim against the TID2008 dataset
is 0.81, against 0.70 for the old ssim implementation, and
it runs 15 times faster.

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

gl/tests: fix glimagesink element test on OSX

It was suffering a race starting up the thread that could cause the main
loop to quit before the main loop had started.

applemedia: error out if the uri property is not set

Fixes:

Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '*** +[NSString stringWithUTF8String:]: NULL cString

in the state change test.

meson: wayland: move dep checks to the lib

Moves them closer to where they are used.  We don't want every check in
the main meson.build file.

meson: gl/egl: add missing source file

d42145e8c1ba3bc0445506b92bb7ac04ae98f4dd didn't add the necessary
meson build changes.

gl/egl: remove EGLImage functions from egl context

By adding the necessary GstEGLImage entry points to create a GstEGLImage
from a GstGLMemory.

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

glcontext: add vfunc to retrieve the OpenGL platform version

i.e. the version of EGL, GLX, etc implemented.

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

gl/egl: move get_error_string() into gstegl

So others can use it without #include-ing a private header

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

display/egl: implement getting the EGLDisplay of a specific platform

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

gl/egl: check the feature in the extensions list

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

vmncdec: Sanity-check width/height before using it

We will allocate a screen area of width*height*bpp bytes, however this
calculation can easily overflow if too high width or height are given
inside the stream. Nonetheless we would just assume that enough memory
was allocated, try to fill it and overwrite as much memory as wanted.

Also allocate the screen area filled with zeroes to ensure that we start
with full-black and not any random (or not so random) data.

https://scarybeastsecurity.blogspot.gr/2016/11/0day-poc-risky-design-decisions-in.html

Ideally we should just remove this plugin in favour of the one in
gst-libav, which generally seems to be of better code quality.

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

ttml: Hide behind --enable-experimental

The elements are autoplugged and currently assert on invalid streams.

smoothwave: Remove empty directory

real: Remove empty directory

mpeg1sys: Remove obsolete, empty directory that only contains a meson.build

curl: transfer hangs in case connection was cut between client and server

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

gl/tests: don't use the default framebuffer

Create our own instead as the default framebuffer may require special
fiddling (like having a visible window) to correctly display/be renderable.

Fixes the remaining GL library tests on OS X

glcontext/cocoa: implement empty swap_buffers

Fixes some GL tests on OS X.

gl: remove empty BUGS file

We use bugzilla for bug tracking

meson: Build waylandsink

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

configure: fix build if wayland-client is not available

Simplify the PKG_CHECK_MODULES related to Wayland to avoid the confusion
of NOT_FOUND cases when there are 3 nested checks. Group those 3 checks
together since there are no conditions specific to each one.

Thanks to https://ci.gstreamer.net/ for alerting of the problem.

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

configure: fix build if libdrm is not available

If the NOT_FOUND part of the check PKG_CHECK_MODULES is not written, it
defaults to error. Addind the else clause of this check as
HAVE_WAYLAND="no"

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

glwindow/cocoa: remove unneeded window code in NSWindow

This is all taken care of by GstGLWindowCocoa/GstGLNSView now.

glwindow/cocoa: remove our view from the parent when closing

Otherwise, when the application reuses the same UIView, we were getting
draw notifications on the previous view/layer's which weren't valid anymore
and were referencing pointers that had been freed.

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

gl/caopengllayer: add a debug category

glbufferpool: introduce check for GLMemory allocators

The last missing piece of EGLImage support has been pushed.

Fix some MSVC warnings about const-ness

Some miscellaneous warnings about const-ness, either casting away const
where we know it's safe or removing const where it's not correct.

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

Cast away const from GstMetaInfo in *_get_meta_info() functions

MSVC warns about the const in the implicit argument conversion in the
calls to g_once_init_{enter,leave}. It's OK so explicitly cast it.

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

adaptivedemux: Fix startup SEGMENT seeking and setting for live

Because fragment.timestamp is relative value to period_start,
startup SEGMENT seeking should be pointed to "fragment.timestamp + period_start"

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

gdiscreencapsrc: remove empty gst_gdiscreencapsrc_stop vfunc

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

opencv: move base opencv filter class and utilities to gst-libs

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

fbdevsink: Add to meson build

kmssink: Trivial naming fix in meson for consistency

kmssink: Enable in meson build

dash: Fix breakage in unit test caused by gst-indent

mpdparser: Support multiple Period elements in external xml

External xml could have empty, one or multiple top-level "Period" elements.
Because xml parser cannot parse the multiple top-level elements
(i.e., no root element), we need to wrap a xml in order to make root element.
See also ISO/IEC 23009-1:2014 5.3.2.2

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

msdk: Initial windows build support

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

meson: add build files for msdk

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

msdk: load plugins in h265 and vp8 init

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

msdk: Width and Height need to be integer multiples of 32

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

msdk: add to build

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

msdk: Create plugin for Intel's Media SDK

[scott.d.phillips@intel.com: gst-indent *.c *.h]
[scott.d.phillips@intel.com: link with libva at build time]
[scott.d.phillips@intel.com: remove unused includes]
[scott.d.phillips@intel.com: Update README]

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

mpdparser: wrap unmodified data in protection event

PlayReady being the one of the few DRM formats encoding its data with
base64 it was not consistent to have a special case for this. So the
base64 decoding operation now needs to be done by the protection event
consumer, if needed.

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

mssdemux: wrap unmodified data in protection event

PlayReady being the one of the few DRM formats encoding its data with
base64 it was not consistent to have a special case for this. So the
base64 decoding operation now needs to be done by the protection event
consumer, if needed.

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

dtlscertificate: Fix error checking in RSA_generate_key_ex() usage

Was broken during the port for OpenSSL 1.1.

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

winks: undef windows SDK macros before redefining them

Redefining an already defined macro causes a warning from the msvc
preprocessor

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

Remove 'return' from `void` functions

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

mpegtsdemux: fix operator precedence in SAFE_FOURCC_ARGS

Type cast has higher precedence than bitwise shift, so the third
argument will truncate to 8 bits and then shift right by 8 bits
resulting in constant zero.

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

hlsdemux: Fix NULL pointer dereference when checking if there is a next fragment

Thanks to Aleksandr <tumaleksandr@yandex.ua> for reporting and
suggesting the fix.

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

meson: don't add_global_arguments when being built as a subproject

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

gl: x11-xcb is not a required dependency

Don't fail if it's not found..

kmssink: do not get kms bo pitch for planar formats

The API is ill defined for planar formats in case of dumb BOs,
do not use the reported stride in those cases.

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

kmssink: remove custom gst_kms_sink_get_times

The default get_times() function of the base sink is just fine.

Remove the custom get_times() function, because the default function
already reads the timestamps from the buffers.

Signed-off-by: Michael Tretter <m.tretter@pengutronix.de>
https://bugzilla.gnome.org/show_bug.cgi?id=773473