amcvideodec/enc: Correctly check for no PTS on input buffers

MediaCodec gives us a presentation timestamp of 0 if it does not know
anything, but GStreamer gives us GST_CLOCK_TIME_NONE. Don't mix up these
two.

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

dashdemux: Handle current SIDX before current position in normal mode like in keyunit-only trick mode

Otherwise we'll get into an infinite loop here. Now this is still not
correct and will cause a clean error, but at least it won't hang forever
anymore.

gldisplay: fix list leak

Windows aren't always removed in time, and it turns out to be
very, very hard to remove a window in a way that's not racy and
not deadlocky. Since the window itself doesn't leak, freeing
the list on object destruction is enough.

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

mxfmux: fix index entry leak

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

gstaggregator: fix event use after free

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

mpegts: fix invalid memory access in language descriptor

Also add a couple sanity asserts/returns

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

glupload: fix GValue leak

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

meson: Only run decklink cpp test when building it

Otherwise it gets run on Windows with MSVC where we don't build
Decklink and then it causes an error.

webrtcechoprobe: return from _read() early if the probe is not configured yet

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

tsparse: Convert PCR values immediately

Doing lazy conversion of PCR values doesn't work right
when a PCR discont is encountered. Instead, convert PCR
values to the continuous timestamp domain as soon as we
encounter them and store that instead.

applemedia: Fix video texture cache build issue on MacOS

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

applemedia: ensure all textures are released before texturecache is released

It was previously possible for videotexturecache to be finalized before all of
its textures. Finalizing outstanding textures in this circumstance leads
to a crash. This patch ensure resources are freed in the proper order.

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

adaptivemutex: Fix double mutex unlock

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

meson: update version

Release 1.10.4

Update .po files

bluez: Chain up to basesrc query instead of override pad query function

Overriding the pad query function completely overrides all the default
query handling implemented in basesrc, including caps etc. The correct
thing to do is just override the basesrc query vfunc and then chain up
for the queries we don't handle.

waylandsink: consider buffer size when copying to wl shm pool

When the sink receives a buffer that is neither a wl_shm one nor a
dmabuf one, this buffer is copied to an internal wl_shm buffer before
being sent to the display.
In that case, the actual size of the received buffer (which may differ
from the one negotiated in the caps) must be used.

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

hlsdemux: Fix GHashTable leak

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

rawbaseparse: Drop incomplete frames at EOS

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

This would ideally be solved in baseparse but that requires further
thought at this point, and in the meantime it would be good to have
rawbaseparse not assert on this but handle it gracefully instead.

Builds for MacOS

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

applemedia: free videotexturecache texture in gl thread

The cached texture was treated as user_data passed to GstGLBaseMemory
and freed with a GDestroyNotify function. However, this data must
be treated specially: it must be destroyed in the GL thread.

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

GstGLSinkBin: fixed sink property leak

No need to keep an extra reference to sink since an indirect one is added by gst_bin_add()

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

gl/window: attach the GMaincontext in window struct to GSource for X11 window

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

amcvideoenc: Set timestamps on header buffers too

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

amcvideoenc: Encoder output is generally not properly parsed

Don't claim it is and let h264parse and other parsers do their job.

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

amcvideoenc: Encoded H264 is generally not AU aligned

Claiming that it is, can cause h264parse to skip some parsing steps and
the output stays unaligned.

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

vkwindow: fix build error on i386

Vulkan handles are either pointers (64-bit) or 64-bit integer
handles (32-bit). Cast the type in failure cases.

vkwindow.c:229:561: error: return makes integer from pointer without a cast [-Werror=int-conversion]
vkwindow.c:231:194: error: return makes integer from pointer without a cast [-Werror=int-conversion]

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

mpegdemux: Add extra length checks to TS scanning.

Add some missing size checks to the timestamp scanning
fast path.

mpegdemux: Add stream-format to the H.264 caps.

H.264 in MPEG-PS is always byte-stream

psdemux: Rewrite PSM parsing using GstByteReader

Avoid possible buffer overflows and ignore invalid PSM packets better
by using GstByteReader.

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

adaptivedemux: Handle SEEK event only once

send_event() of parent class (i.e., GstBinClass) iterates srcpads
to send SEEK event. And performing it per srcpad is inefficient.
So, let's drop duplicated SEEK event by checking seqnum

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

Release 1.10.3

Update .po files

po: Update translations

motioncells: fix mingw build

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

applemedia: Fail cleanly instead of crashing if obtaining a GL display failed

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

tsdemux: Don't leak pad name and template for Opus

It's set again (the same way) further below, overwriting the previously
allocated memory without freeing.

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

mxfdemux: Set stream tags to NULL after unreffing

Otherwise we might try to unref them a second time a little bit later.

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

tsdemux: Set caps/tags in the GstStream object before pushing the stream-start event

Otherwise downstream will get a more or less empty GstStream although we
already know a lot about it at this point.

pkg-config: Directly link the library with -l instead of the .la file

Which might not exist, like on most modern Linux distributions.

dvdspu: Handle vob display rect starting on an odd line

DVDs always have subpictures that start on an even Y
coordinate, but gstspu does more generic vobsubs these
days, so handle ones that start on an odd vertical position.

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

glslstage: define GL constants if needed instead of using #ifdef

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

gl/examples/gtk: fixed compilation on systems without GL_GEOMETRY_SHADER

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

gl: remove unneeded gst_gl_buffer_alloc from header file

There's no implementation anymore!

glcontext/egl: don't overwrite set GError

Doing so is an error and will cause a glib warning to be printed.

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

androidmedia: Silently skip COLOR_FormatAndroidOpaque when converting to caps

This is special and handled in the decoder when doing rendering to a
surface. Printing a warning for this is just unnecessary noise

uvch264src: Fix wrong argument order to force-key-unit event

CID 1373421

androidmedia: some files are missing in the release tarball

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

dash: Define debug category used in the isoff code for the test too now

dash: Include debug of gstisoff on dashdemux category

Move debug category of gstisoff from default to dashdemux

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

meson: build decklink plugin

autoconvert: fix a reconfigure event leak

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

hlsdemux: Extract ID3 tags, strip them from the output and provide them as tags

They often don't only contain the PCR information but also other
metadata, like title. Give this information to the pipeline.

Also strip the tags from the stream as we a) already parsed them now and
b) decoders don't like these tags to happen in the middle of the stream
(i.e. the start of each fragment) and tagdemux only can strip them off
the beginning and end.

tsdemux: Don't leak a taglist if a taglist has already been created

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

mirror: Fix documentation for the mirror mode

Using the wrong enum there causes it to be randomly mixed with the docs
of the other enums.

https://bugs.debian.org/848297

amcaudiodec: Set "is-adts" to 1 for ADTS AAC

Otherwise it fails to decode.

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

videoaggregator: Remove unlock() if set_info() fails

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

gl/window: remove use of main_context_push/pop_thread_default()

No-one's using/depending on it (it would have criticalled and not worked)
and it's causing more problems than it's solving.  Store the GMainContext
in the public struct instead for subclasses to optionally use instead of
relying on the push/pop state to be correct.

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

gl/effects: use non-PBO GLMemory for internal textures

middle textures in gleffects do not need to use GstGLMemoryPBO as they
aren't transfering data to/from the GPU. This will cost too much DMA
memory and cause performance issue. Change the allocator to use non-PBO
GstGLMemory.

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

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.

dvbsrc: fix unconditional use of FEC 2/5

It is not defined for < v5 minor 7

adaptivedemux: Avoid using unset GError

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

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.

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

gst: Fix caps leak

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

Release 1.10.2

Update .po files

po: Update translations

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.

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

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.

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.

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

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

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

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

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