dtls: port to OpenSSL 1.1.0

Changes are:

- Use the wrapper functions to access opaque data types.  To preserve
  backward compatibility, define fallback definitions

- Remove the use of idiom "pqueue_size(ssl->d1->sent_messages)", since
  there is no replacement

- Use RSA_generate_key_ex instead of the deprecated RSA_generate_key

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

rawparse: pass flow returns upstream

rawvideoparse wouldn't error out on not-negotiated,
but would just keep on going, because it didn't pass
the flow return value back to the parent class and
thus upstream, so the source wouldnt' stop streaming.

build: Also need to define GST_EXPORTS for Autotools

The change to use GST_EXPORT for symbols under Windows requires
GST_EXPORTS for internal use, and that is also needed under Autotools.
The same thing is done for gstreamer-1.0.dll in -core.

gstgl: Re-enable WINAPI on _MSC_VER

The calling convention may be deprecated, but we still need it for
OpenGL. The build issue was caused by an incorrect syntax being used for
the WINAPI (__stdcall) prototype in function pointers which was accepted
by GCC but is rejected by MSVC.

Just remove win32/common/libgstgl.def

It was always wrong since the symbols being exported in gstgl-1.0.dll
are platform-specific, and the check we do in dist checks it on all
platforms (which usually means Linux) and the list is instead
Linux-specific right now.

Even if we fix that, it can still never be right because it depends on
your configuration even on a specific platform. For instance, when we
start building EGL support on Windows using ANGLE, the symbol list will
change depending on whether that is enabled or not.

We also don't need this anymore since we use GST_EXPORT for all
functions exported on Windows now.

plugins: Use explicit type conversion from enums

MSVC warns about this because it's a C++ compiler, and this actually
results in useful things such as the incorrect 'gboolean' return value
for functions that return GstFlowReturn, so let's do explicit
conversions to reduce the noise and increase its efficacy.

Explicitly define float constants as float

With MSVC, this gives the following warning:

warning C4305: 'function': truncation from 'double' to 'gfloat'

Apparently, MSVC does not figure out what type to use for constants
based on the assignment. This warning is very spammy, so let's try to
fix it.

Fix incorrect return type in several functions

All these should return GstFlowReturn, not gboolean

meson: Don't warn about C4146 with MSVC

The error is:

unary minus operator applied to unsigned type, result still unsigned

This is a commonly-done operation in gstreamer and it's done on purpose.
It's just noise.

gstgl: Add GST_EXPORT to all symbols used on Windows

This is a requirement for those symbols to be exported in gstgl-1.0.dll
when building with the MSVC compiler

meson: Add support for building WGL on Windows

Currently only tested with MSVC.

ext/gl: Don't define boolean on Windows with MSVC

The headers we include already define boolean on Windows with MSVC, and
it leads to a typedef redefinition error with jpeglib.h which tries to
redefine it in jmorecfg.h

gstgl: Porting fixes for MSVC with Meson

GL/gl.h needs windows.h on MSVC

WINAPI should not be used with MSVC. It also causes a build error.

dvb/parsechannels: collapse notes on channel_name encoding conflict

dvb/parsechannels: rename ZAP parser for clarity

This function only does ZAP parsing. Additionally, remove redundant
comment while at it.

dvb: fix gtk-doc syntax for wrongly formatted comments

meson: hls: Only build when any crypto_dep is found

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

meson: directsound: Add ole32 library dependency

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

meson: winscreencap depends on gstvideo

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

meson: Remove gstreamer-check-1.0 dependency

It will later be added under tests/check

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

glsyncmeta: don't glFinish() everytime a sync point is set

At minimum, we only need to glFlush() if we are in a shared GL context
environment.  Move the glFinish() to when the actual wait is requested
which may be never.  Improves the throughput on older GL systems without
GL3/GLES3 and/or fence sync objects.

dash: do not use invalid stream duration

If it is is unknown, consider it infinite

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

dash: properly inherit segmentlist from period

Representation can inherit SegmentList from Period if none is available
from the AdaptationSet

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

meson: gl: add support for building with dispmanx on the rpi

gl/eglimage: Fix passing the destroy function to gst_egl_image_new_wrapped

The function pointer and the user data arguments were swapped in both
uses.

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

meson: Add missing gstvideo dep to segmentclip

In file included from ../subprojects/gst-plugins-base/gst-libs/gst/video/video.h:27:0,
                 from ../subprojects/gst-plugins-bad/gst/segmentclip/gstvideosegmentclip.c:25:
../subprojects/gst-plugins-base/gst-libs/gst/video/video-format.h:27:39: fatal error: gst/video/video-enumtypes.h: No such file or directory
 #include <gst/video/video-enumtypes.h>
                                       ^
compilation terminated.

https://ci.gstreamer.net/job/GStreamer-master-meson/269/console

Implemented gst_video_overlay_set_window_handle()

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

meson: gl: fix detection of glx without gl

We need to check for libGL if we may use desktop OpenGL *or* GLX.

meson: gl: fix incorrect error string

Use the winsys variable for an incorrect winsys provided

qt: Fix failing build on RPI

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

gl/dispmanx: egl_show should resize the window only if there is no render rectangle set

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

gl/dispmanx: Fix set_render_rectangle typo

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

mpegtsmux: fix buffer size mismatch in M2TS mode

In M2TS mode, we need an extra 4 bytes in the buffer, so need
to ensure the buffer can contain these. The allocation site
does not know the mode, so this is done in all cases.

meson: Don't set c_std to gnu99

Use the default for each compiler on every platform instead. This
improves our compatibility with compilers that don't have gnu99 as
a c_std.

dvbsrc: fix 'delivery system'-related messages

* Rephrase tune error to be delsys-neutral
* Refer to the actual check in the 'missing sanity check' warnings
* Use "Delivery system" instead of 'delsys'. The
  latter is OK as a shorthand in the code but not
  even a real word

bayer: fix _transform_caps method to preserve fields in given caps

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

waylandsink: Properly configure internal pool

There was a small omission in the code. It still worked, but the pool
configuration may not have been optimimal.

common: sync to latest revision

Fixes accidental common change when commit patch.

mpegtsdemux: Implement efficient program updates

If the parent bin can handle it, only add/remove the new/gone stream
instead of re-adding/re-moving everything

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

mpegtsdemux: Ensure we always get an upstream stream-id

This was a regression.

We only have a upstream-id via STREAM_START if we were in push-mode.
In pull-mode we need to create one.

Note: It would be good to eventually have that method (copied from
gst_pad_get_stream_id_internal()) public in the future

mpegtsdemux: Fix issues with GST_DISABLE_GST_DEBUG

Some variables were hidden or unused

mpegtsdemux: Switch to using GstStream/GstStreamCollection

For each MpegTSBaseStream, we have a GstStream object which
subclasses can extend with information.

For each program a GstStreamCollection is created with all
GstStream from each stream.

mpegtsbase: Remove duplicated code

Just have one function to check for private section streams

adaptivedemux: fix buffer size print format

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

hlsdemux: fix hls demux stream type inheritance

GstHLSDemuxStream inherits GstAdaptiveDemuxStream, not
GstAdaptiveDemux. Fixes memory corruption issues.

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

dvb/parsechannels: parse guard interval as a fraction

Fixes wrong default failover when parsing libdvbv5 formatted
channel configuration files.

dvb/parsechannels: add prototypes and reorganize for clarity

Update my email address

dvb: drop trailing whitespace and c&p leftovers

dvb/parsechannels: parse DVB-T hierarchy from v5 config file

Avoids relying on the H/W default when the information
can be extracted from the configuration file.

dvb/parsechannels: parse DVB-T lp and hp code rates

mpegts: Also clear packetizer on TIME DISCONT

When dealing with TIME-based input, the incoming stream could have
potentially changed completely.

In order to check whether it did or not, we need to re-check all sections
(PAT, PMT...). If it didn't, we will keep using the existing streams/pad,
and if it did we will act as if there was a program switch.

Fixes HLS streaming with decodebin3/playbin3

adaptivedemux: Calculate values before queue2

In order to calculate the *actual* bitrate for downloading a fragment
we need to take into account the time since we requested the fragment.

Without this, the bitrate calculations (previously reported by queue2)
would be biased since they wouldn't take into account the request latency
(that is the time between the moment we request a specific URI and the
moment we receive the first byte of that request).

Such examples were it would be biased would be high-bandwith but high-latency
networks. If you download 5MB in 500ms, but it takes 200ms to get the first
byte, queue2 would report 80Mbit/s (5Mb in 500ms) , but taking the request
into account it is only 57Mbit/s (5Mb in 700ms).

While this would not cause too much issues if the above fragment represented
a much longer duration (5s of content), it would cause issues with short
ones (say 1s, or when doing keyframe-only requests which are even shorter)
where the code would expect to be able to download up to 80Mbit/s ... whereas
if we take the request time into account it's much lower (and we would
therefore end up doing late requests).

Also calculate the request latency for debugging purposes and further
usage (it could allow us to figure out the maximum request rate for
example).

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

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

opencv: Fix memory leak by gst_caps_to_string

gst_caps_to_string function returned allocated memory.
So, It should be freed using g_free function.

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

glwindow: don't use g_thread_join() to join the navigation thread

Using g_thread_join() in _finalize() handlers may result in a deadlock
joining the current thread when the last reference is held by a signal
handler.

e.g.:

error 'Resource deadlock avoided' during 'pthread_join (pt->system_thread, NULL)'

The backtrace looks like this:
[...]
g_thread_join ()
gst_gl_window_finalize ()
gst_gl_window_x11_finalize ()
g_object_unref ()
g_value_unset ()
g_signal_emit_valist ()
g_signal_emit ()
gst_gl_window_send_mouse_event ()
gst_gl_window_mouse_event_cb ()
g_main_dispatch ()
[..]
g_main_loop_run ()
gst_gl_window_navigation_thread ()
g_thread_proxy ()
start_thread ()
clone ()

gluploadelement: fix leak of upload library object

When only linking the element, the upload object will be created from
_transform_caps() but will never be unreffed as the only case is in _stop().

Add an unref if non-NULL to a new finalize handler for this case.

vkdisplay: hold a weakref on the list of windows

It's possible that the window may have been destroyed when a winsys
event comes in for it.

Fixes an assertion in make -C tests/check generic/states.check

vkdevice: only unref the gstcontext query if non-NULL

Fixes an assertion in make -C tests/check generic/states.check

tests/gl: make sure all GL commands are executed on the GL thread

e.g. the final glGetError() must also be completed on the GL thread

gl/win32: remove egl implementation

It hasn't been used in ages and is dead code.

tsdemux: Add conditions to identify ATSC EAC3

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

dvb/parsechannels: make parsing errors delsys-neutral

DVB is a group of delivery standards but we support
several others. While at this, make message formating
more congruent.

meson: Setup pre-commit hooks when configuring

meson: update version

player: Remove Return documentation for function returning void

Release 1.9.90

Update .po files

po: Update translations

waylandsink: Actually use buffer pool config after setting it up

CID: 1373420

fix for https://bugzilla.gnome.org/show_bug.cgi?id=771871

a2dpsink: Drop some dead-code

Left over from the last cleanup.

CID: 1373418

fdkaacenc: fix accessing freed memory

The buffer data is not always copied in _Fill, and will be
read in _DecodeFrame. We unmap at the end of the function,
whether we get there via failure or early out, and keep a
ref to the buffer to ensure we can use it to unmap the
memory even after _finish_frame is called, as it unrefs
the buffer.

Note that there is an access beyond the allocated buffer,
which is only apparent when playing from souphttpsrc (ie,
not from filesrc). This appears to be a bug in the bit
reading code in libfdkaac AFAICT.

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

fdkaacdec: avoid memory corruption on decoding error

The buffer size is expected to be in multiples of the sample size,
not in bytes.

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

fdkaacenc: fix buffer leak

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

fdkaacenc: set framed=true on src caps

This fixes muxing in MPEG TS.

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

gl/dispmanx: Implements set_render_rectangle to adjust the position of window

We cannot set the x, y coordinate of the video frame at the dispmanx at
this point. We need to teach dispmanx backend to understand about
set_render_rectangle API to draw a video with other UI.

This patch keeps the current behavior which places video frame at the
center of the display if there is no set_render_rectangle call to the
dispmanx window.

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

gl: add necessary padding bytes to all public structs

fdkaacenc: set transmux on the fdkaac lib

Not doing so will fail to decode in a simple fdkaacenc ! fdkaacdec
pipeline, though would work if this goes through a file.

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

fdkaacdec: do not error out of out of sync return

The docs say we should continue feeding in data and decoding

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

fdkaac: fix error with AOT_MP2_AAC_LC removed from libfdkaac API

AOT_MP2_AAC_LC is a "pseudo AOT" which got removed after 0.1.4,
and maps to AOT_AAC_LC.

Remove mpegversion 2 from th caps to match.

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

meson: bluez: fix build

decklink: Update to SDK 10.8

OSX and Linux only, Windows needs someone on Windows to convert the .idl
files to C source code.

fdkaac: fix mixup setting interleaved output

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

player: Only report the initial duration if the query was successful

h265parse: Fix parsing of multiple NALs in the hvcC box

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

bluez: dist new header file

Fixes distcheck.

a2dpsink: Don't try to set device caps on avdtpsink

We can't actually configure the transport, so we should only be working
with whatever we get.

avdtpsink: Chain event handling up to basesink

bluez: Clean up and make a2dpsink functional again

Was crashing, and had a lot of cruft (like a capsfilter and a fakesink)
that was no longer required.

avdtpsrc: Add support for AVRCP metadata

Metadata from AVRCP is emitted as tags, and the duration from AVRCP is
used in queries by avdtpsrc.

openjpegdec: Properly offset and shift for all formats when converting

And while at it, also clean up some code.

jpeg2000parse: Don't pass NULL strings to the colorspace / sampling parsing functions

They assert on NULL.

waylandsink: Don't leak GValues in getcaps

waylandsink: Update our window size on configure event

This is specific to when the waylandsink is not being embedded. In
this patch we pass the render lock to the window so it can safely
call gst_wl_window_set_render_rectangle() with the new size.

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

waylandsink: Remove atomic operation

We already take the render lock from the wlqueue thread in some other
place which indicates that there is no use of this atomic instead of
a proper locking mechanism.

waylandsink: Do not pre-configure proposed pool

Upstream must configure the pool before using it. Pre-configuring
the proposed pool could hide bugs in upstream elements.

waylandsink: Properly draw black border in absence of viewporter

When we don't have a viewporter (scaling support), we can't use the
1x1 scaleup image trick. Instead, we need to allocate a buffer with
the same size as the area that need to have black background.

dx9screencapsrc: throw error for invalid screen index

Currently dx9screencapsrc prints a verbose warning in case the screen
index is out of range for the current number of detected monitors. This
value is then dropped.

However there is no initial indication (beside the console print) if it
worked or not. This may result in capturing an unwanted screen as it
would capture the last set index that was not rejected.

This patch sets the index regardless. Instead, the element throws an
error when it tries to run or getting caps for an invalid index.

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

gldisplay: also free the GWeakRef when removing dead contexts

Otherwise we leak GWeakRef's.

Found with make -C tests/check libs/gstglcontext.valgrind

wayland/Makefile.am: add missing libs

libgstwayland is missing a dependency on $(GST_PLUGINS_BASE_LIBS), so add it.

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