platform/upstream/gstreamer.git
2 years agoav1parse: Correct the meaning of color_range flag.
He Junyan [Sun, 24 Apr 2022 14:54:58 +0000 (22:54 +0800)]
av1parse: Correct the meaning of color_range flag.

According to spec:
color range equal to 0 shall be referred to as the studio swing
representation and color range equal to 1 shall be referred to as
the full swing representation.

The current status is just the opposite.

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

2 years agoqsvencoder: Workaround for invalid DTS with VP9
Seungha Yang [Mon, 25 Apr 2022 13:59:14 +0000 (22:59 +0900)]
qsvencoder: Workaround for invalid DTS with VP9

SDK runtime seems to report zero DTS for all frames in case of VP9 but
we don't need to use DTS for the codec.

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

2 years agonvcodec: Stop building for non-{Windows,Linux} target
Seungha Yang [Sat, 23 Apr 2022 15:37:40 +0000 (00:37 +0900)]
nvcodec: Stop building for non-{Windows,Linux} target

The other platforms, specifically macOS have not been supported already
because this plugin loads so or dll. Moreover, NVIDIA dropped
support for macOS as of CUDA 11.0. See also
https://developer.nvidia.com/nvidia-cuda-toolkit-11_0_0-developer-tools-mac-hosts

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

2 years agosrtp: add unit tests
Stéphane Cerveau [Mon, 28 Mar 2022 15:06:46 +0000 (17:06 +0200)]
srtp: add unit tests

Enable unit tests in meson.build
Add test_play_key_error to check the stats

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

2 years agosrtpdec: add counts in stats
Stéphane Cerveau [Fri, 25 Mar 2022 16:47:23 +0000 (17:47 +0100)]
srtpdec: add counts in stats

In order to count the buffers which have been received and dropped for
decryption reason, add a stats to track it.

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

2 years agod3d11videosink: Fix for unhandled mouse double click events
Seungha Yang [Fri, 22 Apr 2022 19:00:21 +0000 (04:00 +0900)]
d3d11videosink: Fix for unhandled mouse double click events

Only window created with CS_DBLCLKS style can receive those mouse
double click events, so we need to use the style for internal/external
windows can get double click events.

Also, passthrough mouse events to parent window in the same message pumping
threads instead of manually forwarding each mouse event.

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

2 years agortpredenc: quieten warning about ignoring header extensions
Mathieu Duponchelle [Fri, 22 Apr 2022 23:57:53 +0000 (01:57 +0200)]
rtpredenc: quieten warning about ignoring header extensions

Turn it into a FIXME, and only log once

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

2 years agoMeson: Fix deprecation warnings
Xavier Claessens [Mon, 7 Mar 2022 13:46:57 +0000 (08:46 -0500)]
Meson: Fix deprecation warnings

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

2 years agorvsg: fix cairo include
Stéphane Cerveau [Fri, 22 Apr 2022 15:44:06 +0000 (17:44 +0200)]
rvsg: fix cairo include

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

2 years agoRevert "videorate: Update the base time on segment updates"
Sebastian Dröge [Thu, 21 Apr 2022 08:41:43 +0000 (11:41 +0300)]
Revert "videorate: Update the base time on segment updates"

This reverts commit 75b4809ebc23814009bebc70c775ab44d85decf2.

See https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2186

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

2 years agoRevert "videorate: Add test for segment update"
Sebastian Dröge [Thu, 21 Apr 2022 08:41:36 +0000 (11:41 +0300)]
Revert "videorate: Add test for segment update"

This reverts commit a76f38b2c7ddbed546bb058c32ebcf8a553c003f.

See https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2186

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

2 years agoRevert "videorate: Only "close" the segment if it is discontinous"
Sebastian Dröge [Thu, 21 Apr 2022 08:41:25 +0000 (11:41 +0300)]
Revert "videorate: Only "close" the segment if it is discontinous"

This reverts commit 6f7922b4dbba5ed780e7b0988669a81848a9e333.

See https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2186

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

2 years agoRevert "videorate: Drop incoming buffers that are outside of the segment"
Sebastian Dröge [Thu, 21 Apr 2022 08:41:15 +0000 (11:41 +0300)]
Revert "videorate: Drop incoming buffers that are outside of the segment"

This reverts commit 24fd80344dbc059b72e13d813ca82f414a9d6cce.

See https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2186

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

2 years agoRevert "videorate: Add unit test for closing a segment and opening a separate one"
Sebastian Dröge [Thu, 21 Apr 2022 08:40:47 +0000 (11:40 +0300)]
Revert "videorate: Add unit test for closing a segment and opening a separate one"

This reverts commit 98f2a84a289ed4d4cfac9bc5c73182a56eefa99c.

See https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2186

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

2 years agortprtxsend: mark RTX buffers with GST_RTP_BUFFER_FLAG_RETRANSMISSION
Havard Graff [Mon, 1 Feb 2021 09:36:42 +0000 (10:36 +0100)]
rtprtxsend: mark RTX buffers with GST_RTP_BUFFER_FLAG_RETRANSMISSION

It is useful for elements downstream from rtxsend to know if the RTP
buffer they are dealing with is an RTX buffer or not.

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

2 years agocuda-converter: fix nvrtc compilation on non-English locale systems
Corentin Damman [Fri, 22 Apr 2022 10:29:29 +0000 (12:29 +0200)]
cuda-converter: fix nvrtc compilation on non-English locale systems

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

2 years agortspclientsink: fix possible shutdown deadlock collect_streams()
Pierre Bourré [Tue, 15 Feb 2022 13:39:43 +0000 (13:39 +0000)]
rtspclientsink: fix possible shutdown deadlock collect_streams()

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

2 years agovideoaggregator: keep old buffer when processing a MISSING_DATA gap
Mathieu Duponchelle [Thu, 26 Nov 2020 17:18:52 +0000 (18:18 +0100)]
videoaggregator: keep old buffer when processing a MISSING_DATA gap

GAP events flagged with MISSING_DATA are transformed into GAP buffers
flagged with CORRUPTED.

In these cases, it is preferable to simply keep rendering the previous
buffer (if there was one) instead of flashing the pad in and out of
view.

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

2 years agocompositor: fix prepare_frame obscuring check
Mathieu Duponchelle [Thu, 26 Nov 2020 17:16:10 +0000 (18:16 +0100)]
compositor: fix prepare_frame obscuring check

A pad without a buffer or with a GAP buffer cannot obscure a
pad below it. Ignore those when considering whether a pad should
be drawn.

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

2 years agocompositor: a pad without a frame can't obscure the background
Mathieu Duponchelle [Thu, 26 Nov 2020 14:57:10 +0000 (15:57 +0100)]
compositor: a pad without a frame can't obscure the background

Skip those when considering whether the background should be
drawn

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

2 years agoaggregator: forward gap event information to gap buffer
Mathieu Duponchelle [Thu, 26 Nov 2020 17:11:12 +0000 (18:11 +0100)]
aggregator: forward gap event information to gap buffer

When the GAP event was flagged with MISSING_DATA, subclasses
may want to adopt a different behaviour, for example by repeating
the last buffer.

As we turn these gap events into gap buffers, we need to flag
those, we do so with a new custom meta.

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

2 years agoUpdate proxy-libintl wrap to 0.4
Xavier Claessens [Thu, 21 Apr 2022 13:52:36 +0000 (09:52 -0400)]
Update proxy-libintl wrap to 0.4

It has fixes for Windows static build.

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

2 years agowebrtcbin: Avoid access of freed memory
Sangchul Lee [Wed, 20 Apr 2022 15:38:37 +0000 (00:38 +0900)]
webrtcbin: Avoid access of freed memory

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

2 years agomp4mux: fix spelling
Tristan Matthews [Tue, 19 Apr 2022 22:40:31 +0000 (18:40 -0400)]
mp4mux: fix spelling

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

2 years agova: h264enc: Do not let L0 number surplus the HW limitation.
He Junyan [Fri, 15 Apr 2022 14:19:09 +0000 (22:19 +0800)]
va: h264enc: Do not let L0 number surplus the HW limitation.

The algorithm to calculate the L0/L1 number may let the L0 number
surplus the HW limitation. We should ensure that limitation after
that calculation.

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

2 years agovideoscale: Add test to ensure that non-scaled metas are kept
Olivier Crête [Wed, 20 Apr 2022 21:25:49 +0000 (16:25 -0500)]
videoscale: Add test to ensure that non-scaled metas are kept

Just make sure that we're not breaking non-scaled metas

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

2 years agovideoscale: Add test for transform_meta function
James Cowgill [Wed, 2 Feb 2022 15:28:15 +0000 (15:28 +0000)]
videoscale: Add test for transform_meta function

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

2 years agovideoscale: Don't copy scaled metas
James Cowgill [Wed, 2 Feb 2022 15:28:21 +0000 (15:28 +0000)]
videoscale: Don't copy scaled metas

Returning TRUE from the `transform_meta` function tells
GstBaseTransform to copy the meta into the new buffer. If videoscale
has already transformed a meta by scaling it, it should always return
FALSE to avoid duplicating the meta.

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

2 years agox11: Fix unused variable warnings
Nirbheek Chauhan [Fri, 1 Apr 2022 09:46:20 +0000 (15:16 +0530)]
x11: Fix unused variable warnings

These are emitted when XInput 2 is not available.

```
../subprojects/gst-plugins-base/sys/ximage/ximagesink.c: In function ‘gst_x_image_sink_handle_xevents’:
../subprojects/gst-plugins-base/sys/ximage/ximagesink.c:696:29: warning: unused variable ‘state’ [-Wunused-variable]
  696 |   GstNavigationModifierType state = GST_NAVIGATION_MODIFIER_NONE;
      |                             ^~~~~
../subprojects/gst-plugins-base/sys/ximage/ximagesink.c:694:35: warning: unused variable ‘touch_frame_open’ [-Wunused-variable]
  694 |   gboolean pointer_moved = FALSE, touch_frame_open = FALSE;
      |                                   ^~~~~~~~~~~~~~~~
../subprojects/gst-plugins-base/sys/xvimage/xvimagesink.c: In function ‘gst_xv_image_sink_handle_xevents’:
../subprojects/gst-plugins-base/sys/xvimage/xvimagesink.c:427:35: warning: unused variable ‘touch_frame_open’ [-Wunused-variable]
  427 |   gboolean pointer_moved = FALSE, touch_frame_open = FALSE;
      |                                   ^~~~~~~~~~~~~~~~
../subprojects/gst-plugins-base/sys/xvimage/xvimagesink.c:426:29: warning: unused variable ‘state’ [-Wunused-variable]
  426 |   GstNavigationModifierType state = GST_NAVIGATION_MODIFIER_NONE;
      |                             ^~~~~
```

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

2 years agomingw: Remove deprecated section in example cross file
Xavier Claessens [Thu, 31 Mar 2022 12:50:47 +0000 (08:50 -0400)]
mingw: Remove deprecated section in example cross file

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

2 years agomeson: Add PYTHONPATH to load GDB helper module
Xavier Claessens [Fri, 25 Feb 2022 21:54:37 +0000 (16:54 -0500)]
meson: Add PYTHONPATH to load GDB helper module

Meson generates a gdbinit file that will automatically load gstreamer
script. However that script uses a helper python module that needs
PYTHONPATH to be pointing into the right location in the source
tree to be able to find gst_gdb.py.

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

2 years agomultiudpsink: allow binding to IPv6 address
Jonas Bonn [Fri, 21 Jan 2022 13:21:18 +0000 (14:21 +0100)]
multiudpsink: allow binding to IPv6 address

When the sink is configured to create sockets with an explicit bind
address, then the created socket gets set to the udp_socket field
irregardless of whether the bind address indicated that the socket
family should be IPv4 or IPv6.  When binding to an IPv6 address, this
results in the following error:

gstmultiudpsink.c:1285:gst_multiudpsink_configure_client:<rtcpsink>
error: Invalid address family (got 10)

This patch adds a check of the address family being bound to and sets
the created socket to used_socket or used_socket_v6, accordingly.

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

2 years agortphdrextsdes: fixup test trying to g_free a local variable
Camilo Celis Guzman [Mon, 18 Apr 2022 09:20:00 +0000 (18:20 +0900)]
rtphdrextsdes: fixup test trying to g_free a local variable

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

2 years agohls/m3u8: Fix starting segment for live playlist
Edward Hervey [Thu, 21 Apr 2022 09:47:55 +0000 (11:47 +0200)]
hls/m3u8: Fix starting segment for live playlist

RFC 8216 6.3.3 "Playing the Media Playlist File" : states that for live media
playlists "the client SHOULD NOT choose a segment that starts less than three
target durations from the end of the Playlist file"

This is an off-by-one error. Since we are looking for the "index" of the
segment, we need to subtract 1 from the searched position.

Ex: For a playlist with 12 entries, we want to start playback on the 9th segment
... which is at index 8.

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

2 years agotest: Add test cases for the H265 bitwriter.
He Junyan [Wed, 2 Mar 2022 06:42:42 +0000 (14:42 +0800)]
test: Add test cases for the H265 bitwriter.

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

2 years agocodecparsers: bitwriter: Add the common bit writer functions for H265.
He Junyan [Wed, 2 Mar 2022 06:31:05 +0000 (14:31 +0800)]
codecparsers: bitwriter: Add the common bit writer functions for H265.

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

2 years agova: vpp: Update the colorimetry and HDR fields when fixup src caps.
He Junyan [Wed, 20 Apr 2022 05:18:34 +0000 (13:18 +0800)]
va: vpp: Update the colorimetry and HDR fields when fixup src caps.

When we fixup src caps, the current way of handling the HDR fields is not
correct.
1. We trim the HDR fields only when the input caps is not a subset of the
fixup src caps. But in fact, the input caps with HDR fields such as the
"mastering-display-info" can possibly be the subset of the fixup src caps,
if they have all same other fields.
2. We always copy the colorimetry from input caps to src caps if it is
absent. But when hdr-tone-mapping is enabled, the HDR->SDR conversion makes
the colorimetry change. We should use downstream's setting, or just use the
default colorimetry of SDR.

We changes to:
1. If hdr-tone-mapping is enabled, we trim all HDR fields and add a correct
colorimetry.
2. Copy colorimetry from input if it is still absent.
3. Consider the subset replacement.

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

2 years agovalidate: Do not print Known error tests
Thibault Saunier [Thu, 21 Apr 2022 14:38:24 +0000 (10:38 -0400)]
validate: Do not print Known error tests

Those are passing tests in practice and it doesn't give any useful information

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

2 years agotests: convertscale: Do not log EOS events in validateflow
Thibault Saunier [Thu, 21 Apr 2022 14:21:37 +0000 (10:21 -0400)]
tests: convertscale: Do not log EOS events in validateflow

We already checked that we are getting an EOS before going back to ready
and adding the EOS at the end of the .validateflow is racy as the ERROR
message might reach the bus before it is logged.

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

2 years agoci: Trigger cerbero on user project if a branch with the same name exists there
Thibault Saunier [Thu, 21 Apr 2022 14:14:00 +0000 (10:14 -0400)]
ci: Trigger cerbero on user project if a branch with the same name exists there

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

2 years agodocs:examples: Fix building iOS examples
Thibault Saunier [Thu, 21 Apr 2022 13:18:56 +0000 (09:18 -0400)]
docs:examples: Fix building iOS examples

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

2 years agovalidate:pipeline-monitor: Minor cleanup setting unused variable
Thibault Saunier [Tue, 19 Apr 2022 22:38:12 +0000 (18:38 -0400)]
validate:pipeline-monitor: Minor cleanup setting unused variable

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

2 years agovideoconvertscale: Add properties to disable scaling/converting in videoconvert/video...
Thibault Saunier [Thu, 14 Apr 2022 13:48:14 +0000 (09:48 -0400)]
videoconvertscale: Add properties to disable scaling/converting in videoconvert/videoscale

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

2 years agovalidate: scenario: Add a 'allow-error' configuration
Thibault Saunier [Thu, 14 Apr 2022 13:09:35 +0000 (09:09 -0400)]
validate: scenario: Add a 'allow-error' configuration

This new configuration allows the scenario to keep running even
after receiving an "error" message on the bus.

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

2 years agovideoconvertscale: Don't claim we can support any kind of memory
Thibault Saunier [Fri, 21 May 2021 22:55:25 +0000 (18:55 -0400)]
videoconvertscale: Don't claim we can support any kind of memory

Since d0133a2d11566ff4c0cded7af8dfdff0046e0e8b "videoconvert: Allow
passthrough for ANY caps features" videoconvert will always claim that
it supports any kind of memory which is true in very specific case (when
it is running in passthrough mode). To get elements that autoplug
converters depending on the caps running in the pipeline (like
autovideoconvert), we need to have converters no lie about what they can
do when queried `accept_caps` or `query_caps`.

This still accepts any caps feature as before but it introduces
a restriction in the way we handle memory capsfeatures.

We keep previous behaviour in videoconvert and videoscale.

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

2 years agoIntroduce the videocolorscale element
Thibault Saunier [Fri, 17 Apr 2020 19:23:03 +0000 (15:23 -0400)]
Introduce the videocolorscale element

Now that videoconvert and videoscale's are both based on
GstVideoConverter and are using the exact same code, it makes much more
sense to have one element doing the two operation, and it can be
more efficient in some cases (one single path for both operations).

This removes the `videoscale` and `videoconvert` plugins but keeps the element
but makes them also do both operations (adding some APIs to each element).

There is a small change in API for the `videoscale:dither` property which
was previously a totally unused boolean, it is now an enum and is used.

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

2 years agoqsv: Add GPU description to element long-name on Windows
Seungha Yang [Wed, 20 Apr 2022 14:06:56 +0000 (23:06 +0900)]
qsv: Add GPU description to element long-name on Windows

Would be useful for a multi-Intel-GPU system

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

2 years agohls: Relax webvtt checks
Edward Hervey [Wed, 20 Apr 2022 12:50:35 +0000 (14:50 +0200)]
hls: Relax webvtt checks

If no hour field is present (which is allowed), the remaining data can be less
than 15 character.

Fix time translation failures if the hour field wasn't present

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

2 years agovapostproc: Notify if hdr-tone-mapping property changes.
Víctor Manuel Jáquez Leal [Tue, 19 Apr 2022 19:38:31 +0000 (21:38 +0200)]
vapostproc: Notify if hdr-tone-mapping property changes.

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

2 years agogstleaks: log the number of alive objects
Guillaume Desmottes [Wed, 20 Apr 2022 07:38:16 +0000 (09:38 +0200)]
gstleaks: log the number of alive objects

Can help checking if the number of alive objects is growing over time or stays stable.

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

2 years agova: Expose a couple va surface/buffer methods.
Víctor Manuel Jáquez Leal [Wed, 30 Mar 2022 16:09:36 +0000 (18:09 +0200)]
va: Expose a couple va surface/buffer methods.

And use them in va plugin.

Also, set as default logging category the vadisplay one.

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

2 years agortpmanager: Move some duplicated constant and helper function to a single place
Sebastian Dröge [Wed, 20 Apr 2022 07:53:16 +0000 (10:53 +0300)]
rtpmanager: Move some duplicated constant and helper function to a single place

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

2 years agogstreamer: Document various caps for the reference timestamp meta
Sebastian Dröge [Tue, 19 Apr 2022 11:05:16 +0000 (14:05 +0300)]
gstreamer: Document various caps for the reference timestamp meta

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

2 years agortpbin/rtpjitterbuffer: Don't parse RTCP SRs twice unless needed
Sebastian Dröge [Mon, 18 Apr 2022 13:22:50 +0000 (16:22 +0300)]
rtpbin/rtpjitterbuffer: Don't parse RTCP SRs twice unless needed

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

2 years agortpjitterbuffer: Add property to throttle handling of RTCP SR / NTP-64 syncing
Sebastian Dröge [Mon, 18 Apr 2022 08:50:48 +0000 (11:50 +0300)]
rtpjitterbuffer: Add property to throttle handling of RTCP SR / NTP-64 syncing

This proxies the "rtcp-sync-interval" property of rtpbin.

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

2 years agortpsession: Handle RTCP-SR-REQ (RFC6051) RTCP feedback message
Sebastian Dröge [Mon, 11 Apr 2022 16:14:43 +0000 (19:14 +0300)]
rtpsession: Handle RTCP-SR-REQ (RFC6051) RTCP feedback message

This causes an RTCP SR to be sent at the earliest possible time.

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

2 years agortpbin/rtpjitterbuffer: Allow syncing to an SR without CNAME if the CNAME is already...
Sebastian Dröge [Mon, 11 Apr 2022 16:25:43 +0000 (19:25 +0300)]
rtpbin/rtpjitterbuffer: Allow syncing to an SR without CNAME if the CNAME is already known

The RTCP SR packet might be without SDES in case of a reduced-size RTCP
packet. For syncing purposes the CNAME is needed but it might be known
already from an earlier RTCP packet or out of band, via the SDP for
example.

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

2 years agortpbin/jitterbuffer: Use inband 64-bit NTP timestamps according to RFC6051 for faster...
Sebastian Dröge [Thu, 7 Apr 2022 15:59:07 +0000 (18:59 +0300)]
rtpbin/jitterbuffer: Use inband 64-bit NTP timestamps according to RFC6051 for faster synchronization

When signalled via the caps that the header extension is used, it will
be read and used in the same way as the RTP/NTP time mapping from RTCP
SRs.

If the CNAME of the stream's SSRC is provided out of band via e.g. the
SDP then this allows streams to be synchronized immediately on the first
packet instead of having to wait for the first RTCP SR to arrive.

Fixes https://gitlab.freedesktop.org/gstreamer/gst-plugins-good/-/issues/383

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

2 years agortpsession: Only add send latency to the running time if it is actually known
Sebastian Dröge [Sat, 9 Apr 2022 08:00:52 +0000 (11:00 +0300)]
rtpsession: Only add send latency to the running time if it is actually known

Otherwise we can't know the running time yet if rtcp-sync-send-time is
set, and have to wait until the latency is known later.

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

2 years agortpsession: Update 64-bit NTP header extensions with the actual NTP time in senders
Sebastian Dröge [Wed, 6 Apr 2022 12:39:14 +0000 (15:39 +0300)]
rtpsession: Update 64-bit NTP header extensions with the actual NTP time in senders

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

2 years agortpmanager: Add header extension implementation for the 64-bit RFC6051 NTP header...
Sebastian Dröge [Tue, 5 Apr 2022 17:05:57 +0000 (20:05 +0300)]
rtpmanager: Add header extension implementation for the 64-bit RFC6051 NTP header extension

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

2 years agosdp: Parse the RFC5576 Source-specific media SDP attributes into caps
Sebastian Dröge [Thu, 7 Apr 2022 16:36:25 +0000 (19:36 +0300)]
sdp: Parse the RFC5576 Source-specific media SDP attributes into caps

The format of the caps fields is
    ssrc-(SSRC_VALUE)-(ATTRIBUTE_NAME)=(ATTRIBUTE_VALUE)
.

Parsing of the attributes from the caps into the SDP is not implemented
as this depends not only a single stream's caps but on the whole rtpbin
configuration.

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

2 years agortsp-server: Add RFC5576 Source-specific media attribute to the SDP media for signall...
Sebastian Dröge [Thu, 7 Apr 2022 16:14:27 +0000 (19:14 +0300)]
rtsp-server: Add RFC5576 Source-specific media attribute to the SDP media for signalling the CNAME

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

2 years agodashsink: Unlock when failed to get content
Wonchul Lee [Tue, 19 Apr 2022 23:09:58 +0000 (08:09 +0900)]
dashsink: Unlock when failed to get content

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

2 years agogstglmixerbin: minor refactor of _find_element_pad_template
Camilo Celis Guzman [Mon, 18 Apr 2022 09:53:45 +0000 (18:53 +0900)]
gstglmixerbin: minor refactor of _find_element_pad_template

As suggested by @slomo, make the loop more readable and prevent returning
a garbage value to the caller from the previous implementation.

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

2 years agogstav1parse: fixup various possible logical errors
Camilo Celis Guzman [Mon, 18 Apr 2022 08:50:22 +0000 (17:50 +0900)]
gstav1parse: fixup various possible logical errors

Found via an analyzed build for Clang. Specifically we had:

gstav1parse.c[1850,11] in gst_av1_parse_detect_stream_format: Logic error: The left operand of '==' is a garbage value
gstav1parse.c[1606,11] in gst_av1_parse_handle_to_small_and_equal_align: Logic error: The left operand of '==' is a garbage value

Also a couple of false-positives:

gstav1parse.c[1398,24] in gst_av1_parse_handle_one_obu: Logic error: Branch condition evaluates to a garbage value
gstav1parse.c[1440,37] in gst_av1_parse_handle_one_obu: Logic error: The left operand of '-' is a garbage value

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

2 years agoAlways define ENABLE_NLS
Xavier Claessens [Fri, 25 Mar 2022 14:18:34 +0000 (10:18 -0400)]
Always define ENABLE_NLS

GLib guarantees libintl API is always available, provided by
proxy-libintl as last resort. GLib itself unconditionally define
ENABLE_NLS.

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

2 years agoDelete unused i18n headers
Xavier Claessens [Fri, 25 Mar 2022 14:20:24 +0000 (10:20 -0400)]
Delete unused i18n headers

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

2 years agoReplace gst-i18n-*.h with gi18n-lib.h
Xavier Claessens [Fri, 25 Mar 2022 13:59:23 +0000 (09:59 -0400)]
Replace gst-i18n-*.h with gi18n-lib.h

GLib guarantees libintl is always present, using proxy-libintl as
last resort. There is no need to mock gettex API any more.

This fix static build on Windows because G_INTL_STATIC_COMPILATION must
be defined before including libintl.h, and glib does it for us as part
as including glib.h.

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

2 years agogst-play: Do not split translatable string
Xavier Claessens [Fri, 25 Mar 2022 15:21:03 +0000 (11:21 -0400)]
gst-play: Do not split translatable string

Concatenating N_() strings does not work with gi18n.h macro, was working
only with GStreamer's redefinition.

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

2 years agocuda: Fix typo in gst_cuda_buffery_copy_type_to_string
Thibault Saunier [Wed, 13 Apr 2022 19:23:56 +0000 (15:23 -0400)]
cuda: Fix typo in gst_cuda_buffery_copy_type_to_string

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

2 years agocuda: Add a `gst_cuda_context_new_wrapped` constructor
Thibault Saunier [Sun, 12 Dec 2021 13:19:52 +0000 (13:19 +0000)]
cuda: Add a `gst_cuda_context_new_wrapped` constructor

So that application or people implementing cuda element from
outside GStreamer itself can ensure that their contexts are used
by GStreamer.

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

2 years agocuda: Factor out a public GstCUDA library
Thibault Saunier [Wed, 8 Dec 2021 11:48:08 +0000 (11:48 +0000)]
cuda: Factor out a public GstCUDA library

So applications and elements implemented outside GStreamer can reuse
our infrastructure

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

2 years agonvenc: Give more details about errors
Thibault Saunier [Fri, 3 Dec 2021 13:12:42 +0000 (13:12 +0000)]
nvenc: Give more details about errors

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

2 years agovideo: Fix possible overrun when iterating comp[] array
Nicolas Dufresne [Tue, 19 Apr 2022 15:05:05 +0000 (11:05 -0400)]
video: Fix possible overrun when iterating comp[] array

Fix 2 iterations that can overrun the array if the number of component is
equal to the size of the array.

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

2 years agoRevert "video-frame: avoid possible out of bound memory access"
Nicolas Dufresne [Tue, 19 Apr 2022 14:53:15 +0000 (10:53 -0400)]
Revert "video-frame: avoid possible out of bound memory access"

This reverts commit c4255f08f787ff6a41504b538fa4c2e911d40ec7.

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

2 years agomeson_options: declare glib and gobject checks at top level
Camilo Celis Guzman [Wed, 27 Oct 2021 17:38:07 +0000 (02:38 +0900)]
meson_options: declare glib and gobject checks at top level

This makes it easier to specify this features for all subprojects.

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

2 years agomeson: Add -Wl,-rpath,${libdir} on macOS
Nirbheek Chauhan [Mon, 18 Apr 2022 10:14:47 +0000 (15:44 +0530)]
meson: Add -Wl,-rpath,${libdir} on macOS

We made the gstreamer installation prefix relocatable by picking up
plugins relative to the location of libgstreamer-1.0.dylib, similar to
how it's done for Windows:

https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1627

This had a lot of side-effects:

https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/1051
https://gitlab.freedesktop.org/gstreamer/cerbero/-/issues/363
https://gitlab.freedesktop.org/gstreamer/cerbero/-/issues/371
https://gitlab.freedesktop.org/gstreamer/cerbero/-/issues/362

A partial fix for the cerbero side of these was:

https://gitlab.freedesktop.org/gstreamer/cerbero/-/merge_requests/807

However, this relied on the consumers knowing that they need to add
`LC_RPATH` entries to the libdir of the prefix. This is done
automatically by build systems like Meson, but not by others, such as
Autotools, CMake, Cargo, XCode, etc. For those, we need to add the
RPATH entries to the gstreamer-1.0.pc file.

This also has the side-effect of fixing the loading of gstreamer rust
plugins on macOS:

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

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

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

2 years agovideo-frame: avoid possible out of bound memory access
Camilo Celis Guzman [Mon, 18 Apr 2022 09:14:44 +0000 (18:14 +0900)]
video-frame: avoid possible out of bound memory access

Although the components' initialization code would fill in -1 to all
unset components, make the code a bit more defensive and check for an
index bound first.

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

2 years agotests: dash_mpd: fix linker issues with non-optimizing compilers
Tim-Philipp Müller [Tue, 19 Apr 2022 08:52:51 +0000 (09:52 +0100)]
tests: dash_mpd: fix linker issues with non-optimizing compilers

undefined reference to `download_request_take_buffer'

Fixes https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2117#note_1344646

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

2 years agogst_plugin_load_file: force plugin reload if diff filename
Ruben Gonzalez [Fri, 12 Nov 2021 19:13:10 +0000 (20:13 +0100)]
gst_plugin_load_file: force plugin reload if diff filename

If a file includes a new version of a plugin that exits in the
registry, the output of gst-inspect is incorrect. The output has the
correct version but incorrect filename, and element description.

This seems to have also fixed some documentation issues.

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

2 years agodocs: design: fix up some document titles
Tim-Philipp Müller [Mon, 18 Apr 2022 19:05:54 +0000 (20:05 +0100)]
docs: design: fix up some document titles

These show up in the index.

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

2 years agodocs: design: add adaptive streaming design docs to sitemap
Tim-Philipp Müller [Mon, 18 Apr 2022 19:04:26 +0000 (20:04 +0100)]
docs: design: add adaptive streaming design docs to sitemap

.. and some small additions to make it clearer what exist
and what's new.

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

2 years agod3dvideosink: Remove outdated comment
Seungha Yang [Mon, 18 Apr 2022 19:37:05 +0000 (04:37 +0900)]
d3dvideosink: Remove outdated comment

d3dvideosink is not a primary video sink element

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

2 years agoh264decoder,h265decoder: Do not hold codec_data buffer
Seungha Yang [Sun, 17 Apr 2022 15:25:18 +0000 (00:25 +0900)]
h264decoder,h265decoder: Do not hold codec_data buffer

... also do not need to extract it from caps.

Decoder uses codec_data only once per set_format() and
baseclass will extract codec_data buffer already.

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

2 years agod3d11decoder: Do not preallocate texture using downstream d3d11 buffer pool
Seungha Yang [Sun, 17 Apr 2022 14:55:55 +0000 (23:55 +0900)]
d3d11decoder: Do not preallocate texture using downstream d3d11 buffer pool

Our decoder implementation does not use downstream d3d11 pool for
decoding because of special requirement of D3D11/DXVA. So preallocation
using the downstream buffer pool will waste GPU memory in most cases.

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

2 years agogstdevicemonitor: added cleanup of signal handlers and hidden providers list
Tulio Beloqui [Fri, 29 May 2020 14:29:56 +0000 (11:29 -0300)]
gstdevicemonitor: added cleanup of signal handlers and hidden providers list

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

2 years agoNew HLS, DASH and MSS adaptive demuxer elements
Edward Hervey [Fri, 11 Mar 2022 16:11:50 +0000 (17:11 +0100)]
New HLS, DASH and MSS adaptive demuxer elements

This provides new HLS, DASH and MSS adaptive demuxer elements as a single plugin.

These elements offer many improvements over the legacy elements. They will only
work within a streams-aware context (`urisourcebin`, `uridecodebin3`,
`decodebin3`, `playbin3`, ...).

Stream selection and buffering is handled internally, this allows them to
directly manage the elementary streams and stream selection.

Authors:
* Edward Hervey <edward@centricular.com>
* Jan Schmidt <jan@centricular.com>
* Piotrek Brzeziński <piotr@centricular.com>
* Tim-Philipp Müller <tim@centricular.com>

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

2 years agov4l2videodec: copy colorimetry values to output_state caps
Hou Qi [Fri, 15 Apr 2022 01:53:19 +0000 (09:53 +0800)]
v4l2videodec: copy colorimetry values to output_state caps

This is to avoid transcoding negotiation fail between v4l2h265dec
and v4l2h264enc caused by colorimetry mismatch.

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

2 years agoh264parse: Include coded-picture-structure info in caps
Vivia Nikolaidou [Thu, 14 Apr 2022 10:22:03 +0000 (13:22 +0300)]
h264parse: Include coded-picture-structure info in caps

This reverts commit 652773de366a98e155332bf8ffb83ee5e6bea9e5 and
modifies it to rename the caps field name to coded-picture-structure.

It was previously removed because it confuses the decoder and we didn't
have a valid use case for including it in the encoded caps at this
stage. We now do have such a use case but still don't want to confuse
the decoder, so the field is renamed.

However, it is still not accurate without looking at the SEI picture
structure of each frame, so it was named coded-picture-structure. If its
value is "frame" it is most likely progressive, if it's "field" it is
most likely interlaced or mixed.

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

2 years agolibav: fix frame leak on negotiation error
Guillaume Desmottes [Fri, 15 Apr 2022 14:38:46 +0000 (16:38 +0200)]
libav: fix frame leak on negotiation error

The function owns a reference on the frame. Drop it if negotiation
failed as we are already doing for the other error cases.

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

2 years agodecodebin3: fix collection leak
Guillaume Desmottes [Tue, 25 May 2021 12:41:51 +0000 (14:41 +0200)]
decodebin3: fix collection leak

get_merged_collection() returns an owned stream collection and was
leaked in the else block.

Fix leak when running:
GST_TRACERS=leaks GST_DEBUG="GST_TRACER:7,leaks:6" gst-play-1.0 --use-playbin3 test.mkv

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

2 years agova: h264enc: Fix the forgotten unlock() when getting device path property.
He Junyan [Mon, 18 Apr 2022 04:44:19 +0000 (12:44 +0800)]
va: h264enc: Fix the forgotten unlock() when getting device path property.

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

2 years agoqsvdecoder: Use system memory for non-d3d11 downstream
Seungha Yang [Sun, 17 Apr 2022 13:25:14 +0000 (22:25 +0900)]
qsvdecoder: Use system memory for non-d3d11 downstream

QSV's memory download implementation seems to be more faster
than our implementation

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

2 years agoqsv: Add H.265 decoder
Seungha Yang [Sat, 16 Apr 2022 17:14:50 +0000 (02:14 +0900)]
qsv: Add H.265 decoder

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

2 years agoqsv: Include single gstva.h
Seungha Yang [Sat, 16 Apr 2022 17:18:37 +0000 (02:18 +0900)]
qsv: Include single gstva.h

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

2 years agoqsvh264dec: Don't register element if it's not supported by device
Seungha Yang [Sat, 16 Apr 2022 17:04:08 +0000 (02:04 +0900)]
qsvh264dec: Don't register element if it's not supported by device

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

2 years agoqsvh264dec: Add support for packetized format
Seungha Yang [Sat, 16 Apr 2022 16:27:10 +0000 (01:27 +0900)]
qsvh264dec: Add support for packetized format

Handle avc/avc3 packetized formats

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

2 years agoqsv: Move debug init into each source file
Seungha Yang [Sat, 16 Apr 2022 15:00:55 +0000 (00:00 +0900)]
qsv: Move debug init into each source file

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

2 years agoqsvdecoder: Reset next_task_index per init_session()
Seungha Yang [Sat, 16 Apr 2022 14:51:05 +0000 (23:51 +0900)]
qsvdecoder: Reset next_task_index per init_session()

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