platform/upstream/gstreamer.git
2 years agoRelease 1.21.1
Tim-Philipp Müller [Tue, 4 Oct 2022 00:14:01 +0000 (01:14 +0100)]
Release 1.21.1

2 years agoUpdate ChangeLogs for 1.21.1
Tim-Philipp Müller [Tue, 4 Oct 2022 00:13:59 +0000 (01:13 +0100)]
Update ChangeLogs for 1.21.1

2 years agoplaysink: Hold a reference to the soft volume element
Jan Schmidt [Fri, 30 Sep 2022 16:33:49 +0000 (02:33 +1000)]
playsink: Hold a reference to the soft volume element

Always hold a reference to the soft volume element
provided by the playsinkaudioconvert bin helper, the
same as when volume is provided by a sink element,
or the soft volume element gets unreffed too soon.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3108>

2 years agoxvimagesink: Don't leak XvShmImage
Jan Schmidt [Fri, 30 Sep 2022 18:51:21 +0000 (04:51 +1000)]
xvimagesink: Don't leak XvShmImage

If allocating an XvShmImage generates an X error, but
still returns some allocated memory, make sure to free it
so it doesn't leak.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3100>

2 years agoxvimagesink: Zero initialize mask array
Jan Schmidt [Fri, 30 Sep 2022 18:49:16 +0000 (04:49 +1000)]
xvimagesink: Zero initialize mask array

Clear the stack array before setting bits to pass to
XISelectEvents(). Fixes spurious crash from an X error:

X Error of failed request:  BadValue (integer parameter out of range for operation)

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3100>

2 years agoxvimagesink: Don't leak temporary
Jan Schmidt [Fri, 30 Sep 2022 18:47:31 +0000 (04:47 +1000)]
xvimagesink: Don't leak temporary

Use a stack variable instead of a temporarily
malloced variable that wasn't being freed
properly. Fixes a small leak of a GstXvTouchDevice

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3100>

2 years agodecodebin3: Make sure event is writable before modifying
Jan Schmidt [Fri, 30 Sep 2022 18:41:06 +0000 (04:41 +1000)]
decodebin3: Make sure event is writable before modifying

Make sure we're operating on a private copy of an event when
modifying it.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3107>

2 years agourisourcebin: Make sure event is writable before modifying.
Jan Schmidt [Fri, 30 Sep 2022 18:40:09 +0000 (04:40 +1000)]
urisourcebin: Make sure event is writable before modifying.

Make sure we're operating on a private copy of an event before
modifying it.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3107>

2 years agodecodebin3: Don't lose a ref on EOS event
Jan Schmidt [Fri, 30 Sep 2022 18:37:05 +0000 (04:37 +1000)]
decodebin3: Don't lose a ref on EOS event

Make sure not to give away the ref on the final EOS
event for which the probe handler is returning GST_PAD_PROBE_REMOVE
when pushing the event manually.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3107>

2 years agoqueue2: Hold the lock when modifying sinkresult
Edward Hervey [Mon, 3 Oct 2022 09:16:25 +0000 (11:16 +0200)]
queue2: Hold the lock when modifying sinkresult

As it's done elsewhere. Avoids a potential race of the field being modified in
the meantime.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3110>

2 years agoqtdemux: Don't stop task when resetting
Edward Hervey [Wed, 27 Jul 2022 09:19:50 +0000 (11:19 +0200)]
qtdemux: Don't stop task when resetting

This is a regression that was introduced in
cca2f555d14b5751f7f9d466b66127544dad5138 (yes, 9 years ago).

The only place where a demuxer streaming thread should be stopped is when the
sinkpad is deactivated from pull mode (i.e. PAUSED->READY).

Attempting to stop the task in this function would cause this to happen when a
FLUSH_STOP or STREAM_START event is received... which can cause deadlocks.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3109>

2 years agoavauddec: fix unnecessary reconfiguration if the audio layout isn't specified
Tim-Philipp Müller [Thu, 29 Sep 2022 11:42:21 +0000 (12:42 +0100)]
avauddec: fix unnecessary reconfiguration if the audio layout isn't specified

It would constantly want to renegotiate (and spam the debug log) even
though the channel layout hasn't actually changed. We use the same
fallback in gst_ffmpegauddec_negotiate() already.

This happens with WMA files for example.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3103>

2 years agosplitmuxsrc: don't consider unlinked pads when deactivating part
Mathieu Duponchelle [Fri, 30 Sep 2022 16:57:01 +0000 (18:57 +0200)]
splitmuxsrc: don't consider unlinked pads when deactivating part

If splitmuxsrc exposes multiple pads, but only one is linked, part pads
will never see an EOS event. This shouldn't prevent the part from being
eventually deactivated.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3099>

2 years agopbutils: descriptions: add meta/x-klv
Tim-Philipp Müller [Thu, 29 Sep 2022 23:10:25 +0000 (00:10 +0100)]
pbutils: descriptions: add meta/x-klv

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3094>

2 years agopbutils: descriptions: fix gst_pb_utils_get_caps_description_flags()
Tim-Philipp Müller [Thu, 29 Sep 2022 23:07:07 +0000 (00:07 +0100)]
pbutils: descriptions: fix gst_pb_utils_get_caps_description_flags()

And add a little unit test.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3094>

2 years agoxmptag: Call gst_tag_register_musicbrainz_tags during init
James Cowgill [Thu, 29 Sep 2022 08:39:15 +0000 (09:39 +0100)]
xmptag: Call gst_tag_register_musicbrainz_tags during init

We need to call this to register the MusixBrainz tags before we use
them in an XMP schema.

Fixes this critical when attempting to run jpegparse on a JPEG
containing MusicBrainz XMP tags:

  GStreamer-CRITICAL **: 20:41:07.885: gst_tag_get_type: assertion 'info != NULL' failed

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3092>

2 years agotest-effect-switch: Fix some memory leaks and make effect element ownership clearer
Sebastian Dröge [Thu, 29 Sep 2022 11:36:38 +0000 (14:36 +0300)]
test-effect-switch: Fix some memory leaks and make effect element ownership clearer

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3091>

2 years agogaussblur: Fix memory leak when setting caps multiple times
Sebastian Dröge [Thu, 29 Sep 2022 11:34:31 +0000 (14:34 +0300)]
gaussblur: Fix memory leak when setting caps multiple times

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3091>

2 years agov4l2codecs: Warn user when no media devices are found
Christopher Obbard [Tue, 27 Sep 2022 17:17:24 +0000 (18:17 +0100)]
v4l2codecs: Warn user when no media devices are found

Currently if the user is not able to access the devices under /dev/media*,
either due to no media devices present on the system or simply no permission
to access the device, v4l2codecs initialises with no features or debug messages.

Since calling `GST_DEBUG="v4l2*:7" gst-inspect-1.0 v4l2codecs` is a typical way
to diagnose why element(s) failed to enumerate, we should be more verbose here
when the user is not able to access any /dev/media* device. So print a simple
debug message in this case to aid debugging.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3088>

2 years agobuffer: drop parent meta in deep copy/foreach_metadata
Philipp Zabel [Wed, 28 Mar 2018 15:54:15 +0000 (17:54 +0200)]
buffer: drop parent meta in deep copy/foreach_metadata

The purpose of a deep buffer copy is to be able to release the source
buffer and all its dependencies. Attaching the parent buffer meta to
the newly created deep copy needlessly keeps holding a reference to the
parent buffer.

The issue this solves is the fact you need to allocate more
buffers, as you have free buffers being held for no reason. In the good
cases it will use more memory, in the bad case it will stall your
pipeline (since codecs often need a minimum number of buffers to
actually work).

Fixes #283

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2928>

2 years agotsdemux: Don't trigger a program change when falling back to ignore-pcr behaviour
Mart Raudsepp [Tue, 20 Sep 2022 20:31:45 +0000 (23:31 +0300)]
tsdemux: Don't trigger a program change when falling back to ignore-pcr behaviour

Since commit a79a756b79aa1675e we could change to ignore-pcr automatically at 500ms
into a live stream when no PCR is seen by then. However the stream counting in
program change detection was wrongly considering ignore-pcr programs to have a
separate PCR PID, even though we are actually ignoring the PCR PID completely,
resulting in an erroneous program switch getting triggered from the different
stream count. This in turn would send an EOS and switch out the pads for what
actually is still the same program, while we intended to simply apply a
workaround for broken encoders.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3060>

2 years agogst-docs: update macOS new project instructions
Aleix Conchillo Flaqué [Fri, 23 Sep 2022 21:46:33 +0000 (14:46 -0700)]
gst-docs: update macOS new project instructions

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3074>

2 years agomeson: Do not modify default value of -Dtools for subprojects
Nirbheek Chauhan [Tue, 12 Apr 2022 19:03:03 +0000 (00:33 +0530)]
meson: Do not modify default value of -Dtools for subprojects

See: https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/1136
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2166>

2 years agomeson: Remove some dead code
Nirbheek Chauhan [Tue, 12 Apr 2022 19:03:26 +0000 (00:33 +0530)]
meson: Remove some dead code

We no longer create any symlinks.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2166>

2 years agoexamples: Support multiple video streams in JS webrtc sendrecv
Nirbheek Chauhan [Mon, 26 Sep 2022 11:23:43 +0000 (16:53 +0530)]
examples: Support multiple video streams in JS webrtc sendrecv

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3079>

2 years agortspsrc: Fix usage of IPv6 connections in SETUP
Nirbheek Chauhan [Tue, 1 Mar 2022 11:00:10 +0000 (16:30 +0530)]
rtspsrc: Fix usage of IPv6 connections in SETUP

If the SETUP request returns an IPv6 server address in the Transport
field, we would generate an incorrect URI, and multiudpsink would fail
to initialize:

```
     rtspsrc gstrtspsrc.c:9780:dump_key_value:<source>    key: 'Transport', value: 'RTP/AVP;unicast;source=fe80::dc27:25ff:fe5e:bd13:8080;client_port=62696-62697;server_port=4000-4001'
...
     rtspsrc gstrtspsrc.c:4595:gst_rtspsrc_stream_configure_udp_sinks:<source> configure RTP UDP sink for fe80::dc27:25ff:fe5e:bd13:8080:4000
...
multiudpsink gstmultiudpsink.c:1229:gst_multiudpsink_configure_client:<udpsink0> error: Invalid address family (got 23)
```

We can't look at stream->is_ipv6 because we can't rely on the server
returning the right value there. In the issue reported about this,
server reported itself as `KuP RTSP Server/0.1`, and the SDP was:

```
c=IN IP4
m=video 54608 RTP/AVP 96
a=rtpmap:96 H264/90000
```

So we need to parse the string value and figure out the family
ourselves.

Fixes https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/1058

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1819>

2 years agomeson: Remove builtin variable datadir from pkgconfig variables
Nirbheek Chauhan [Tue, 27 Sep 2022 14:16:28 +0000 (19:46 +0530)]
meson: Remove builtin variable datadir from pkgconfig variables

Fixes warning with meson 0.62:

gst-plugins-bad| subprojects/gst-plugins-bad/meson.build:546: WARNING:
Project targets '>= 0.62' but uses feature deprecated since '0.62.0':
pkgconfig.generate variable for builtin directories. They will be
automatically included when referenced

and more.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3086>

2 years agodecodebin3: Fix memory issues with active selection list
Edward Hervey [Tue, 27 Sep 2022 11:41:37 +0000 (13:41 +0200)]
decodebin3: Fix memory issues with active selection list

This had a couple of issues:
* The backing strings (from GstStream) could disappear
* The actual list wasn't properly reset/freed when decodebin3 was re-used

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3083>

2 years agoqtdemux: guard against timestamp calculation overflow in gap event loop
Tim-Philipp Müller [Tue, 27 Sep 2022 12:56:54 +0000 (13:56 +0100)]
qtdemux: guard against timestamp calculation overflow in gap event loop

Could possibly cause an endless loop.

Fixes #1400.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3084>

2 years agodashdemux2: fix mpd unit test expectations
Jan Schmidt [Mon, 26 Sep 2022 14:08:41 +0000 (00:08 +1000)]
dashdemux2: fix mpd unit test expectations

Update unit test for some mpd cases that were reporting
timestamps including the period start time, while
dashdemux2 expects that it needs to add the period
start time itself.

Fix the tests to not expect the period start time
to be included.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3025>

2 years agodashdemux2: Set timestamp relative to period start
Junsoo Park [Tue, 23 Aug 2022 14:12:44 +0000 (23:12 +0900)]
dashdemux2: Set timestamp relative to period start

These values will be referred to as timestamp relative to period start
so need to subtract period start time from the values.

Fixes a problem with determining the start position when playing Live content
with SegmentTimeline, presentationTimeOffset and a non-0 period start time.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3025>

2 years agogsturi: When setting the same string again do nothing
Sebastian Dröge [Mon, 26 Sep 2022 11:17:18 +0000 (14:17 +0300)]
gsturi: When setting the same string again do nothing

Otherwise code like gst_uri_set_host(uri, gst_uri_get_host(uri)) would
first free the string, then create a copy of the freed string and then
assigned that.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3076>

2 years agoUpdate all wraps from WrapDB
Xavier Claessens [Sun, 18 Sep 2022 19:27:19 +0000 (15:27 -0400)]
Update all wraps from WrapDB

Update all wraps available in WrapDB. This is simply running the
command "meson wrap update --force" with that Meson PR:
https://github.com/mesonbuild/meson/pull/10357.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3042>

2 years agovp9: check if libvpx supports high bit depth
Jakub Adam [Thu, 22 Sep 2022 17:02:10 +0000 (19:02 +0200)]
vp9: check if libvpx supports high bit depth

Detect at runtime if libvpx is compiled with --enable-vp9-highbitdepth
and enable 10bit video formats in element caps accordingly.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3067>

2 years agomeson: Add a new option to control the source for orc
Nirbheek Chauhan [Wed, 21 Sep 2022 16:18:31 +0000 (21:48 +0530)]
meson: Add a new option to control the source for orc

Previously we were unconditionally cloning the orc subproject because
we want the developer environment to use the latest orc. However, some
people want to use the system orc instead, or want to auto-detect
which to use, with the system orc getting preference. This requires
adding a new option to select that. See discussion at:
https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2556

Fixes https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/1282

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3062>

2 years agoci: Move non-image-tag variables back to .gitlab-ci.yml
Nirbheek Chauhan [Thu, 22 Sep 2022 19:38:49 +0000 (01:08 +0530)]
ci: Move non-image-tag variables back to .gitlab-ci.yml

https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3065#note_1563706

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3068>

2 years agoci: Bump cerbero trigger timeout to 4h
Nirbheek Chauhan [Thu, 22 Sep 2022 12:33:53 +0000 (18:03 +0530)]
ci: Bump cerbero trigger timeout to 4h

macOS runners can get overworked and cause cerbero pipelines to take
longer than 3h to complete, because each cross-macos-universal job
takes about 2h and so it can take that long to get a runner allocated.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3065>

2 years agoci: Use include syntax for files on the same gitlab instance
Nirbheek Chauhan [Thu, 22 Sep 2022 12:31:10 +0000 (18:01 +0530)]
ci: Use include syntax for files on the same gitlab instance

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3065>

2 years agoci: Move image tags into a separate template yml
Nirbheek Chauhan [Thu, 22 Sep 2022 12:22:42 +0000 (17:52 +0530)]
ci: Move image tags into a separate template yml

This will allow gst-plugins-rs documentation CI to include this
template via a remote include and doesn't need to manually update the
image. This causes cascading failures across all repos, for example
https://gitlab.freedesktop.org/gstreamer/gst-plugins-rs/-/merge_requests/874
is required for making gst-plugins-rs CI pass in Cerbero, which is
also run on the gstreamer monorepo.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3065>

2 years agobasetransform: Avoid useless codepath
Edward Hervey [Thu, 15 Sep 2022 14:22:23 +0000 (16:22 +0200)]
basetransform: Avoid useless codepath

If QoS is disabled, skip the whole computation (avoids calculating values which
won't be needed)

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3034>

2 years agobad/soundtouch: Fix interactive test build on MSVC
Nirbheek Chauhan [Thu, 22 Sep 2022 03:44:57 +0000 (09:14 +0530)]
bad/soundtouch: Fix interactive test build on MSVC

Needed for https://gitlab.freedesktop.org/gstreamer/cerbero/-/merge_requests/930

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3064>

2 years agoci: Remove unused FEDORA_DOCS_IMAGE variable
Nirbheek Chauhan [Thu, 22 Sep 2022 04:27:22 +0000 (09:57 +0530)]
ci: Remove unused FEDORA_DOCS_IMAGE variable

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3061>

2 years agoci: Bump meson to 0.62.2 in Fedora docker images
Nirbheek Chauhan [Thu, 22 Sep 2022 04:25:45 +0000 (09:55 +0530)]
ci: Bump meson to 0.62.2 in Fedora docker images

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3061>

2 years agomeson: Use implicit builtin dirs in pkgconfig generation
Nirbheek Chauhan [Wed, 21 Sep 2022 13:49:45 +0000 (19:19 +0530)]
meson: Use implicit builtin dirs in pkgconfig generation

Starting with Meson 0.62, meson automatically populates the variables
list in the pkgconfig file if you reference builtin directories in the
pkgconfig file (whether via a custom pkgconfig variable or elsewhere).
We need this, because ${prefix}/libexec is a hard-coded value which is
incorrect on, for example, Debian.

Bump requirement to 0.62, and remove version compares that retained
support for older Meson versions.

Fixes https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/1245

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3061>

2 years agoadaptivedemux2: Always wake up loop thread on unpause
Seungha Yang [Tue, 20 Sep 2022 15:26:38 +0000 (00:26 +0900)]
adaptivedemux2: Always wake up loop thread on unpause

Otherwise loop thread will sleep forever because
GstAdaptiveDemuxLoop.paused flag update is not signalled
when loop was marked as stopped already from other thread.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3057>

2 years agoadaptivedemux2: Rework input download wakeups
Jan Schmidt [Thu, 15 Sep 2022 16:01:58 +0000 (02:01 +1000)]
adaptivedemux2: Rework input download wakeups

Change the way streams are woken up to download more data.

Instead of checking the level on tracks that are being
output as data is dequeued, calculate a 'wakeup time'
at which it should download more data, and wake up
the stream when the global output position crosses
that threshold.

For efficiency, compute the earliest wakeup time
for all streams and store it on the period, so the
output loop can quickly check only a single value
to decide if something needs waking up.

Does the same buffering as the previous method,
but ensures that as we approach the end of
one period, the next period continues incrementally
downloading data so that it is fully buffered when
the period starts.

Fixes issues with multi-period VOD content where
download of the second period resumes only after
the first period is completely drained.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3055>

2 years agovabaseenc: Move out encoder validation from assertion.
Víctor Manuel Jáquez Leal [Tue, 20 Sep 2022 11:38:26 +0000 (13:38 +0200)]
vabaseenc: Move out encoder validation from assertion.

Assertion can be disabled at compilation time. Still it's important to
validate it the encoder object was opened by the subclass. This patch
removes the assertion and returns if the encoder is open.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3051>

2 years agovalidate:scenario: Make appsrc-push push a sample
Thibault Saunier [Wed, 14 Sep 2022 18:45:28 +0000 (15:45 -0300)]
validate:scenario: Make appsrc-push push a sample

Allowing setting a segment to the sample

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3052>

2 years agovalidate: Handle buffer pts/dts/duration in the appsrc-push action
Thibault Saunier [Wed, 14 Sep 2022 18:36:41 +0000 (15:36 -0300)]
validate: Handle buffer pts/dts/duration in the appsrc-push action

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3052>

2 years agovalidate:scenario: Simplify the way we override appsrc src pad chain
Thibault Saunier [Wed, 14 Sep 2022 18:31:20 +0000 (15:31 -0300)]
validate:scenario: Simplify the way we override appsrc src pad chain

When pushing several buffers while the pipeline is in NULL state, meaning
that the action are executed "interlaced", previous code was deadlocking.

This new implementation makes it so the override is always on and we
expect all buffers to go through to be associated to a function, which
is a safe assumption.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3052>

2 years agohlsdemux2: Handle negative time mappings
Edward Hervey [Thu, 15 Sep 2022 07:04:10 +0000 (09:04 +0200)]
hlsdemux2: Handle negative time mappings

Some servers can return playlists with "old" media playlists and different
Discont Sequence.

In those cases, the segment stream times would be negative when creating a new
time mapping. In order to properly handle such scenarios, shift the values to
stored accordingly to end up with non-negative reference stream time.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3054>

2 years agoges-video-source: Remove limit on frame duplication on EOS in videorate
Thibault Saunier [Thu, 8 Sep 2022 15:38:34 +0000 (11:38 -0400)]
ges-video-source: Remove limit on frame duplication on EOS in videorate

Fixes https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/1352

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3000>

2 years agovideorate: Add a `max-closing-segment-duplication-duration` property
Thibault Saunier [Tue, 6 Sep 2022 21:25:50 +0000 (17:25 -0400)]
videorate: Add a `max-closing-segment-duplication-duration` property

This allows users to let videorate fully fill the segments when received
EOS or on new segment, removing an arbitrary limit of 25 duplicates which
might not be what the user wants (for example on low FPS stream in GES,
that sometimes leaded to broken behavior)

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3000>

2 years agovalidate:scenario: Fix a leak when done waiting for an ASYNC action
Thibault Saunier [Thu, 8 Sep 2022 22:25:07 +0000 (18:25 -0400)]
validate:scenario: Fix a leak when done waiting for an ASYNC action

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3000>

2 years agonlecomposition: Minor debug enhancement
Thibault Saunier [Tue, 6 Sep 2022 21:24:51 +0000 (17:24 -0400)]
nlecomposition: Minor debug enhancement

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3000>

2 years agoges: Enhance element names in video uri sources
Thibault Saunier [Tue, 6 Sep 2022 21:24:22 +0000 (17:24 -0400)]
ges: Enhance element names in video uri sources

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3000>

2 years agomeson: Set install_tag on some targets
Xavier Claessens [Mon, 12 Sep 2022 13:46:43 +0000 (09:46 -0400)]
meson: Set install_tag on some targets

Trying to follow recommendation from Meson documentation:
https://mesonbuild.com/Installing.html#installation-tags

Move tools into 'bin' or 'bin-devel' categories to keep only libs and
plugins in the default 'runtime' category. This simplifies distribution
of GStreamer application skipping parts that are not needed, similarly
to what Cerbero does by hardcoding huge list of files.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3017>

2 years agoqueue2: Fix deadlock when deactivate is called in pull mode
Paweł Stawicki [Wed, 31 Aug 2022 16:08:08 +0000 (18:08 +0200)]
queue2: Fix deadlock when deactivate is called in pull mode

check is flush was called before waiting on condition

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2967>

2 years agomsdkav1enc: We should not set other alignment here
Mengkejiergeli Ba [Tue, 20 Sep 2022 08:21:24 +0000 (16:21 +0800)]
msdkav1enc: We should not set other alignment here

According to spec, we have 32bit alignment for height (progressive) and
16bit alignment for width, so here we don't need to add other alignment
settings.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3050>

2 years agoshout: fix minimum requirement to libshout >= 2.4.3
Mart Raudsepp [Mon, 19 Sep 2022 17:48:01 +0000 (20:48 +0300)]
shout: fix minimum requirement to libshout >= 2.4.3

commit e64c6f0b93ced added usage of the SHOUT_USAGE_UNKNOWN symbol, but this
became available in the Icecast-libshout 2.4.3 release, not 2.4.2

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3048>

2 years agova: h264enc: Fix rate control enum register failure for the second GPU.
He Junyan [Sat, 17 Sep 2022 09:49:47 +0000 (17:49 +0800)]
va: h264enc: Fix rate control enum register failure for the second GPU.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3041>

2 years agovideo-converter: Fix doc
Edward Hervey [Mon, 19 Sep 2022 06:54:15 +0000 (08:54 +0200)]
video-converter: Fix doc

Argument names weren't correct

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3043>

2 years agoci: Bootstrap with visualstudio variant enabled
Nirbheek Chauhan [Sat, 17 Sep 2022 21:01:01 +0000 (02:31 +0530)]
ci: Bootstrap with visualstudio variant enabled

visualstudio variant now pulls in more build-tools deps on top of
mingw since it enables the rust variant by default. It's also the
variant that's tested on Cerbero CI.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2507>

2 years agoci: bump image tags so subprojects get updated
Tim-Philipp Müller [Fri, 27 May 2022 10:04:43 +0000 (11:04 +0100)]
ci: bump image tags so subprojects get updated

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2507>

2 years agosubprojects: harfbuzz: bump to 5.2.0
Tim-Philipp Müller [Sat, 17 Sep 2022 19:25:39 +0000 (20:25 +0100)]
subprojects: harfbuzz: bump to 5.2.0

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2507>

2 years agosubprojects: libmicrodns: bump to 0.2.0 + meson fixes
Tim-Philipp Müller [Fri, 27 May 2022 12:14:01 +0000 (13:14 +0100)]
subprojects: libmicrodns: bump to 0.2.0 + meson fixes

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2507>

2 years agosubprojects: pango: bump to 1.50.10
Tim-Philipp Müller [Sat, 17 Sep 2022 19:28:54 +0000 (20:28 +0100)]
subprojects: pango: bump to 1.50.10

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2507>

2 years agosubprojects: fribidi: bump to 1.0.12
Tim-Philipp Müller [Fri, 27 May 2022 08:55:59 +0000 (09:55 +0100)]
subprojects: fribidi: bump to 1.0.12

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2507>

2 years agosubprojects: cairo: bump to 1.17.6
Tim-Philipp Müller [Fri, 27 May 2022 08:51:28 +0000 (09:51 +0100)]
subprojects: cairo: bump to 1.17.6

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2507>

2 years agosubprojects: glib, glib-networking: bump to 2.72
Tim-Philipp Müller [Fri, 27 May 2022 08:48:51 +0000 (09:48 +0100)]
subprojects: glib, glib-networking: bump to 2.72

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2507>

2 years agosubprojects: freetyp2: bump to 2.12.1
Tim-Philipp Müller [Fri, 27 May 2022 08:48:25 +0000 (09:48 +0100)]
subprojects: freetyp2: bump to 2.12.1

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2507>

2 years agosubprojects: fontconfig: bump to 2.14.0
Tim-Philipp Müller [Fri, 27 May 2022 08:48:03 +0000 (09:48 +0100)]
subprojects: fontconfig: bump to 2.14.0

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2507>

2 years agov4l2videodec: add cap negotiation fail check in the capture configuration change
Elliot Chen [Wed, 31 Aug 2022 03:13:06 +0000 (11:13 +0800)]
v4l2videodec: add cap negotiation fail check in the capture configuration change

The capture configuration change may cause negotiation fail.
Need to check it to avoid enter the endless loop.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2960>

2 years agovah264enc: Avoid precision lost by just rounding up.
Víctor Manuel Jáquez Leal [Sat, 17 Sep 2022 08:14:40 +0000 (10:14 +0200)]
vah264enc: Avoid precision lost by just rounding up.

The code where dividing by 16 and later multiplying by 16, which is
spurious and a potential loose of precision.

2 years agovah264enc: Update AUD property if driver can't handle raw data.
Víctor Manuel Jáquez Leal [Fri, 16 Sep 2022 12:17:38 +0000 (14:17 +0200)]
vah264enc: Update AUD property if driver can't handle raw data.

2 years agovah264enc: Remove unused dispose method.
Víctor Manuel Jáquez Leal [Fri, 16 Sep 2022 11:55:11 +0000 (13:55 +0200)]
vah264enc: Remove unused dispose method.

2 years agovadeinterlace: Fix passthrough latency query
Eric Knapp [Fri, 16 Sep 2022 19:45:11 +0000 (15:45 -0400)]
vadeinterlace: Fix passthrough latency query

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3038>

2 years agovalidate: Update action types documentation
Thibault Saunier [Sat, 17 Sep 2022 01:05:18 +0000 (22:05 -0300)]
validate: Update action types documentation

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3040>

2 years agonvh265sldec: Sync up with d3d11h265dec implementation
Seungha Yang [Fri, 16 Sep 2022 20:01:55 +0000 (05:01 +0900)]
nvh265sldec: Sync up with d3d11h265dec implementation

Each RefPicSetStCurrBefore/RefPicSetStCurrAfter/RefPicSetLtCurr array
might have empty element (i.e., reference pictures might not be stored
sequentially). Don't error out for the empty element case,
but instead, iterates each array and fill NVDEC's reference list
as much as possible

Fixes: https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/1441
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3039>

2 years agova: baseenc: Fix gobject style for chained method.
Víctor Manuel Jáquez Leal [Fri, 16 Sep 2022 10:50:33 +0000 (12:50 +0200)]
va: baseenc: Fix gobject style for chained method.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3035>

2 years agocodectimestamper: Update document
Seungha Yang [Tue, 13 Sep 2022 14:20:54 +0000 (23:20 +0900)]
codectimestamper: Update document

Add more description about those elements

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3021>

2 years agobaseparse: If available, return average bitrate upstream
Edward Hervey [Thu, 15 Sep 2022 14:23:16 +0000 (16:23 +0200)]
baseparse: If available, return average bitrate upstream

Helps improve queue2 buffering for single stream playback (ex: FLAC or mp3)

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3033>

2 years agova: av1dec: user internal buffer pool for non output layers.
He Junyan [Thu, 9 Jun 2022 14:32:16 +0000 (22:32 +0800)]
va: av1dec: user internal buffer pool for non output layers.

The AV1 support multi spatial layers within one TU with different
resolutions, and only the highest spatial layer need to be output.
For example, there are two spatial layer, base level is 800x600
and higher level is 1920x1080. We need to decode both because the
higher level needs base layer as reference, but we only need to output
1920x1080 frames here.

The current manner always renegotiates the caps once we detect the
current picture resolution changes, so we renegotiate again and
again between different layers. That's a big waste and has very
low performance. We now only do the renegotiation for the highest
output layer. For other non output layers, we just keep a internal
buffer pool which is big enough to handle the surface allocation.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2382>

2 years agocodecs: av1decoder: Add the highest_spatial_layer field.
He Junyan [Thu, 12 May 2022 02:12:37 +0000 (10:12 +0800)]
codecs: av1decoder: Add the highest_spatial_layer field.

As SPEC says, when multi spatial layer exists, we should only output
one frame with the highest spatial id from each TU. We now store the
highest spatial layer information in the base class in order to let
the sub class handle different layers easily.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2382>

2 years agovoamrwbenc: Fix truncation of audio data at end-of-stream when audio data
Devin Anderson [Thu, 15 Sep 2022 00:52:14 +0000 (00:52 +0000)]
voamrwbenc: Fix truncation of audio data at end-of-stream when audio data
doesn't align on 20 millisecond frame size.

The AMR-WB codec imposes a fixed 20 millisecond frame size.  In its current
form, the `voamrwbenc` plugin deals with this limitation by discarding any
audio at the end of the stream that falls short of 20 milliseconds.  This patch
keeps the audio data, and appends silence to the end to preserve frame size
alignment.

The patch also adds tests to check for the updated behavior.  I noticed that
tests weren't being built, so I changed the build to allow for building the
tests when the `tests` and `voamrwbenc` options are set.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3027>

2 years agocodectimestamper: Fix for unknown framerate
Seungha Yang [Thu, 15 Sep 2022 12:56:27 +0000 (21:56 +0900)]
codectimestamper: Fix for unknown framerate

Use default framerate if numerator or denominator is unknown

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3031>

2 years agodecodebin2: Do not fail if one of the decoders isn't able to output the requested...
Thibault Saunier [Fri, 9 Sep 2022 20:57:18 +0000 (16:57 -0400)]
decodebin2: Do not fail if one of the decoders isn't able to output the requested format

when expose-all=False

When trying to find an decoder in that case, we loop over the different
decoder factories, and check that it outputs a format that matches the
requested one (through the :caps property), but if we find a decoder
that do match but later on some other don't we end up failing
autopluging. This patch ensures that we still plug the decoder that can
work.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3011>

2 years agoBuild documentation for rust plugins
Thibault Saunier [Fri, 26 Aug 2022 12:43:34 +0000 (08:43 -0400)]
Build documentation for rust plugins

- Update the docker image we use, starting using the standard one adding
  `gtk4-doc` as required by rust plugins
- Update the plugins_doc_caches as required, some more plugins are built
  with the new image
- Install ninja from pip as the version from F31 is too old
- Avoid buildings all GSreamer plugins when building the doc as it takes
  time and resources for no good reason
- Stop linking to `GInstanceInitFunc` as it is not present in latest GLib
  documentation, leading to warnings in hotdoc.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2954>

2 years agogl:meson: Minor typo fix
Thibault Saunier [Mon, 29 Aug 2022 14:20:55 +0000 (10:20 -0400)]
gl:meson: Minor typo fix

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2954>

2 years agodoc: Do not build plugins to build the doc
Thibault Saunier [Mon, 29 Aug 2022 14:17:45 +0000 (10:17 -0400)]
doc: Do not build plugins to build the doc

It is not actually necessary

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2954>

2 years agodocs: Generate gst-plugins-rs documentation
Thibault Saunier [Fri, 26 Aug 2022 12:39:00 +0000 (08:39 -0400)]
docs: Generate gst-plugins-rs documentation

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2954>

2 years agodoc: Add an option to enable fatal warnings
Thibault Saunier [Mon, 29 Aug 2022 14:43:52 +0000 (10:43 -0400)]
doc: Add an option to enable fatal warnings

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2954>

2 years agodocs: plugin-scanner: Minor debug enhancement
Thibault Saunier [Fri, 26 Aug 2022 12:39:16 +0000 (08:39 -0400)]
docs: plugin-scanner: Minor debug enhancement

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2954>

2 years agodiscoverer: Fix discovering source that expose raw audio/video
Thibault Saunier [Wed, 7 Sep 2022 14:36:09 +0000 (10:36 -0400)]
discoverer: Fix discovering source that expose raw audio/video

Exposes a "uridecodebin:post-stream-topology" property as the discoverer
needs to have topology information about all streams so we need
`uridecodebin` to always plug decodebins for that case.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3009>

2 years agouridecodebin: Fix some property documentation syntax
Thibault Saunier [Wed, 7 Sep 2022 21:11:08 +0000 (17:11 -0400)]
uridecodebin: Fix some property documentation syntax

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3009>

2 years agouridecodebin: Ensure that pads caps are set before exposing them
Thibault Saunier [Wed, 7 Sep 2022 14:39:21 +0000 (10:39 -0400)]
uridecodebin: Ensure that pads caps are set before exposing them

We are supposed to guarantee that pads that are exposed have the caps
set, but for sources that have pad with "all raw caps" templates, we end
up exposing pads that don't have caps set yet, which can break code (in
GES for example).

To avoid that we let uridecodebin plug a `decodebin` after such pads and
let decodebin to handle that for us. In the end the only thing that
decodebin does in those cases is to wait for pads to be ready and expose
them, after that `uridecodebin` will expose those pads.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3009>

2 years agosubprojects: orc: bump to latest commit
Tim-Philipp Müller [Thu, 15 Sep 2022 13:13:49 +0000 (14:13 +0100)]
subprojects: orc: bump to latest commit

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3032>

2 years agotests: add a few more orc tests
Tim-Philipp Müller [Thu, 15 Sep 2022 11:14:56 +0000 (12:14 +0100)]
tests: add a few more orc tests

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3029>

2 years agogst-full: Register GIO modules when glib-networking is a subproject
Xavier Claessens [Thu, 23 Jun 2022 15:50:00 +0000 (11:50 -0400)]
gst-full: Register GIO modules when glib-networking is a subproject

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2653>

2 years agohlsdemux2: Fix crash on live playlist with single entry
Edward Hervey [Wed, 14 Sep 2022 08:15:41 +0000 (10:15 +0200)]
hlsdemux2: Fix crash on live playlist with single entry

If there is a single entry, we would end up computing a minimum distance of 0,
and would therefore read entries from after the segment array

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3024>