@@ -1,4 +1,9 @@

 -------------------------------------------------------------------

+Wed Oct 26 13:32:09 UTC 2022 - Bjørn Lie <zaitor@opensuse.org>

+

+- Update to version 1.20.4

+

+-------------------------------------------------------------------

 Sat Jun 25 13:49:39 UTC 2022 - Bjørn Lie <zaitor@opensuse.org>

 - Update to version 1.20.3

​

@@ -7,7 +7,7 @@

 %define _version 1.20.0

 Name:           gstreamer-plugins-bad-codecs

-Version:        1.20.3

+Version:        1.20.4

 Release:        0

 Summary:        Codecs/plugins for gstreamer-plugins-bad

 License:        LGPL-2.1-or-later

​

@@ -1,7 +1,387 @@

+=== release 1.20.4 ===

+

+2022-10-12 16:39:47 +0100  Tim-Philipp Müller <tim@centricular.com>

+

+   * NEWS:

+   * RELEASE:

+   * gst-plugins-bad.doap:

+   * meson.build:

+     Release 1.20.4

+

+2022-10-12 16:39:40 +0100  Tim-Philipp Müller <tim@centricular.com>

+

+   * ChangeLog:

+     Update ChangeLogs for 1.20.4

+

+2022-08-08 23:37:11 +0900  Seungha Yang <seungha@centricular.com>

+

+   * gst/mxf/mxfaes-bwf.c:

+     mxfdemux: Always calculate BlockAlign of raw audio

+     Workaround for nBlockAlign and nBitsPerSample mismatch. Always

+     use the formula described in the specification for BlockAlign value

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

+

+2022-09-14 00:58:37 +0900  Seungha Yang <seungha@centricular.com>

+

+   * sys/nvcodec/gstnvdec.c:

+     nvdec: Fix for HEVC decoding when coded resolution is larger than display resolution

+     As documented in the SDK header, we should set coded width/height

+     values to the corresponding decoder configuration option,

+     instead of display resolution

+     Fixes: https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/1438

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

+

+2022-09-22 22:39:31 +0900  Sangchul Lee <sc11.lee@samsung.com>

+

+   * ext/webrtc/gstwebrtcbin.c:

+     webrtcbin: Fix pointer dereference before null check

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

+

+2022-10-05 15:59:03 +0900  Sangchul Lee <sc11.lee@samsung.com>

+

+   * ext/webrtc/gstwebrtcice.c:

+     webrtc/nice: Make sure to return NULL when validating turn server fails

+     It affects 'add-turn-server' signal action and 'turn-server' property

+     of webrtcbin.

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

+

+2022-09-20 23:31:45 +0300  Mart Raudsepp <mart@leio.tech>

+

+   * gst/mpegtsdemux/mpegtsbase.c:

+     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/3089>

+

+2022-03-25 14:25:02 +1100  Andrew Pritchard <andrew@vivi.io>

+

+   * sys/androidmedia/jni/gstamcsurfacetexture-jni.c:

+     Fix GstAmcSurfaceTexture segfault Check that `self` and `self->callback` are defined. `self` can be set to `NULL` in `remove_listener`, and `self->callback` can be set to `NULL` inside `gst_amc_surface_texture_jni_set_on_frame_available_callback`. This can cause a segfault since the Java object can outlive the C object, and call the callback after `remove_listener` is called.

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

+

+2022-08-25 14:24:25 +0200  Piotr Brzeziński <piotr@centricular.com>

+

+   * sys/applemedia/avfvideosrc.m:

+     avfvideosrc: Fix wrong default framerate value

+     Current default G_MAXINT is not a correct value under any circumstances.

+     This creates an issue with screen capture, during which we currently do

+     not get any framerate info causing G_MAXINT to show up, where elements

+     downstream can possibly misbehave - for example, `vtenc` causes

+     a kernel panic.

+     Replace with 30/1 to avoid such scenarios.

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

+

+2022-08-20 16:15:15 +0100  Philippe Normand <philn@igalia.com>

+

+   * ext/openh264/gstopenh264dec.cpp:

+   * ext/openh264/gstopenh264enc.cpp:

+     openh264: Register debug categories earlier

+     Otherwise the GST_ERROR message logged in case of ABI mismatch would be done on

+     an uninitialized category.

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

+

+2022-08-20 16:57:27 +0100  Philippe Normand <philn@igalia.com>

+

+   * ext/openh264/gstopenh264enc.cpp:

+     openh264enc: Fix constrained-high encoding

+     constrained-high is high without B-frames, there is no EProfileIdc for this, so

+     assume high instead of hitting an assert down the line.

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

+

+2022-08-14 22:31:29 -0400  Daniel Morin <daniel.morin@collabora.com>

+

+   * gst-libs/gst/play/gstplay-media-info.h:

+   * gst-libs/gst/play/gstplay-signal-adapter.h:

+   * gst-libs/gst/play/gstplay-video-overlay-video-renderer.h:

+   * gst-libs/gst/play/gstplay-video-renderer.h:

+   * gst-libs/gst/play/gstplay-visualization.h:

+     gst-play: missing cleanup for g_autoptr

+     Without this change cleanup function for g_autoptr is not defined for

+     GstPlayMediaInfo, GstPlaySignalAdapter, GstPlayVideoRenderer,

+     GstPlayVideoOverlayVideoRenderer and GstPlayVisualization. Cleanup

+     function was defined in gstplay.h, but missing in other header files.

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

+

+2022-08-13 12:24:37 +0300  Sebastian Dröge <sebastian@centricular.com>

+

+   * gst-libs/gst/player/gstplayer-media-info.c:

+     player: Don't leak wrapped video info

+     Fixes https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/1373

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

+

+2022-08-13 11:50:20 +0300  Sebastian Dröge <sebastian@centricular.com>

+

+   * gst-libs/gst/play/gstplay.c:

+     play: Make ownership of video-sink clearer in combination with floating references

+     And correctly handle the case of VideoRenderer::create_video_sink() not

+     actually returning a floating reference, which might be tricky for some

+     bindings.

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

+

+2022-08-13 11:49:08 +0300  Sebastian Dröge <sebastian@centricular.com>

+

+   * gst-libs/gst/play/gstplay.c:

+     play: Fix object construction

+     Ideally new() functions should simply call g_object_new() and not much

+     else, so let's do that here and handle all the construction properly in

+     a GObject way.

+     Now a play object created via g_object_new() is actually usable.

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

+

+2022-08-13 11:39:59 +0300  Sebastian Dröge <sebastian@centricular.com>

+

+   * gst-libs/gst/player/gstplayer.c:

+     player: Fix object construction

+     Ideally new() functions should simply call g_object_new() and not much

+     else, so let's do that here and handle all the construction properly in

+     a GObject way.

+     Now a player object created via g_object_new() is actually usable.

+     In addition, also fix the video-renderer property so that reading it

+     returns an object of the correct type.

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

+

+2022-08-13 11:30:35 +0300  Sebastian Dröge <sebastian@centricular.com>

+

+   * gst-libs/gst/player/gstplayer.c:

+     player: Release signal adapter on finalize

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

+

+2022-08-12 18:24:41 +0300  Matthias Clasen <mclasen@redhat.com>

+

+   * gst-libs/gst/player/gstplayer.c:

+     gstplayer: Plug a memory leak

+     This was showing up as a memory leak in GTK's

+     gstreamer media backend:

+     40 bytes in 1 blocks are definitely lost in loss record 18,487 of 40,868

+     at 0x484586F: malloc (vg_replace_malloc.c:381)

+     by 0x50D5278: g_malloc (gmem.c:125)

+     by 0x50EDBA5: g_slice_alloc (gslice.c:1072)

+     by 0x50EFBCC: g_slice_alloc0 (gslice.c:1098)

+     by 0x51F2F45: g_type_create_instance (gtype.c:1911)

+     by 0x51DAE37: g_object_new_internal (gobject.c:2011)

+     by 0x51DC080: g_object_new_with_properties (gobject.c:2181)

+     by 0x51DCB20: g_object_new (gobject.c:1821)

+     by 0x9855F86: UnknownInlinedFun (gstplayer-wrapped-video-renderer.c:109)

+     by 0x9855F86: gst_player_new (gstplayer.c:579)

+     Fixes https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/1374

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

+

+2022-07-30 02:29:49 +0530  Nirbheek Chauhan <nirbheek@centricular.com>

+

+   * ext/sctp/usrsctp/meson.build:

+     meson: Don't pass -Werror to vendored code

+     Do it the correct way with libusrsctp -- override the option so that

+     it's done in a compiler-agnostic and future-proof way.

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

+

+2022-05-25 18:40:30 +0530  Nirbheek Chauhan <nirbheek@centricular.com>

+

+   * gst/rtmp2/gstrtmp2locationhandler.c:

+   * gst/rtmp2/rtmp/rtmpclient.c:

+     rtsp+rtmp: Forward warning added to tls-validation-flags to our users

+     With the 2.72 release, glib-networking developers have decided that

+     TLS certificate validation cannot be implemented correctly by them, so

+     they've deprecated it.

+     In a nutshell: a cert can have several validation errors, but there

+     are no guarantees that the TLS backend will return all those errors,

+     and things are made even more complicated by the fact that the list of

+     errors might refer to certs that are added for backwards-compat and

+     won't actually be used by the TLS library.

+     Our best option is to ignore the deprecation and pass the warning onto

+     users so they can make an appropriate security decision regarding

+     this.

+     We can't deprecate the tls-validation-flags property because it is

+     very useful when connecting to RTSP cameras that will never get

+     updates to fix certificate errors.

+     Relevant upstream merge requests / issues:

+     https://gitlab.gnome.org/GNOME/glib/-/merge_requests/2214

+     https://gitlab.gnome.org/GNOME/glib-networking/-/issues/179

+     https://gitlab.gnome.org/GNOME/glib-networking/-/merge_requests/193

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

+

+2022-05-25 16:03:22 +0530  Nirbheek Chauhan <nirbheek@centricular.com>

+

+   * ext/dtls/gstdtlscertificate.c:

+     dtls: Disable OpenSSL 3.0 deprecation warnings for now

+     Fedora 36 ships with OpenSSL 3.0, which deprecates all low-level APIs,

+     so this code needs to be rewritten. There is no easy fix in the

+     porting guide, and it recommends disabling the warnings if you can't

+     use the high-level API.

+     https://wiki.openssl.org/index.php/OpenSSL_3.0#Upgrading_to_OpenSSL_3.0_from_OpenSSL_1.1.1

+     Here's the replacement API:

+     https://www.openssl.org/docs/man3.0/man7/migration_guide.html#Deprecated-low-level-object-creation

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

+

+2022-07-29 02:36:40 +0900  Seungha Yang <seungha@centricular.com>

+

+   * sys/wasapi2/gstwasapi2ringbuffer.cpp:

+     wasapi2: Fix initial mute/volume setting

+     Fix up volume/mute change flag setting

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

+

+2022-07-21 16:11:03 +0300  Sebastian Dröge <sebastian@centricular.com>

+

+   * gst/audiobuffersplit/gstaudiobuffersplit.c:

+     audiobuffersplit: Actually store number of samples to drop in gapless mode

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

+

+2022-07-21 16:10:18 +0300  Sebastian Dröge <sebastian@centricular.com>

+

+   * gst/audiobuffersplit/gstaudiobuffersplit.c:

+     audiobuffersplit: Use input running time for comparison instead of the currently tracked running time

+     Otherwise gapless mode would do completely wrong calculations on

+     discontinuities and cause input/output to drift slowly.

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

+

+2022-07-21 13:38:22 +0300  Sebastian Dröge <sebastian@centricular.com>

+

+   * gst/audiobuffersplit/gstaudiobuffersplit.c:

+     audiobuffersplit: Combine two if expressions to reduce indentation

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

+

+2022-07-06 16:14:13 +0300  Jordan Petridis <jordan@centricular.com>

+

+   * ext/openmpt/gstopenmptdec.c:

+     openmpt: update from now deprecated api

+     https://lib.openmpt.org/doc/classopenmpt_1_1module.html#ab2695af0baa274054f5687741fa7c05b

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

+

+2022-06-30 11:04:29 +0200  Ignazio Pillai <ignazp@amazon.com>

+

+   * sys/wasapi/gstwasapiutil.c:

+     wasapi: Implement default audio channel mask

+     Some multichannel capture devices does not provide a channel mask value

+     which will result in a pipeline failure due to the empty channel mask.

+     Implemented the same fix used for wasapi2

+     Fixes https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/1204

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

+

+2022-07-07 02:17:56 +0900  Seungha Yang <seungha@centricular.com>

+

+   * gst/proxy/gstproxysink.c:

+     proxysink: Fix GstProxySrc leak

+     Clear weak pointer to peer src when disposing.

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

+

+2022-07-06 03:14:25 +0900  Seungha Yang <seungha@centricular.com>

+

+   * gst/proxy/gstproxysink.c:

+   * gst/proxy/gstproxysink.h:

+   * tests/check/elements/proxysink.c:

+   * tests/check/meson.build:

+     proxysink: Make sure stream-start and caps events are forwarded

+     There might be a sequence of event and buffer flow:

+     - Got stream-start/caps/segment events

+     - Got flush events

+     - And then buffers with a new segment event

+     In the above case, stream-start and caps event might not be reached to

+     peer proxysrc if peer proxysrc is not ready to receive them.

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

+

+2022-06-30 09:09:02 +0300  Sebastian Dröge <sebastian@centricular.com>

+

+   * ext/webrtc/gstwebrtcbin.c:

+     webrtcbin: Reject caps that are not valid for creating an SDP media.

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

+

+2022-06-29 10:55:13 +0100  Tim-Philipp Müller <tim@centricular.com>

+

+   * meson.build:

+     coding style: allow declarations after statement

+     See https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1243/

+     and https://gitlab.freedesktop.org/gstreamer/gstreamer-project/-/issues/78

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

+

+2022-06-28 17:40:56 +0900  Seungha Yang <seungha@centricular.com>

+

+   * sys/d3d11/gstd3d11videosink.cpp:

+     d3d11videosink: Fix for force-aspect-ratio setting when rendering on shared texture

+     Set specified force-aspect-ratio value on window object

+     in case of shared texture rendering as well

+     Fixes: https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/1304

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

+

+2022-06-25 19:50:10 +0100  Tim-Philipp Müller <tim@centricular.com>

+

+   * tests/check/meson.build:

+     tests: skip unit tests for dependency-less elements that have been disabled

+     Fixes https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/-/issues/1136

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

+

+2022-06-28 01:29:06 +0100  Tim-Philipp Müller <tim@centricular.com>

+

+   * ext/opus/gstopusheader.h:

+     dv, opusparse: fix duplicate symbols in static build

+     Fixes https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/-/issues/1262

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

+

+2022-06-23 14:31:10 +0200  Víctor Manuel Jáquez Leal <vjaquez@igalia.com>

+

+   * sys/va/gstvaallocator.c:

+     va: allocator: Use always lseek to get dmabuf size.

+     Gallium drivers historically have reported strange dmabuf sizes, from always

+     zero to the whole frame (multiple fds). The simplest solution is to use lseek

+     SEEK_END to get the prime descriptor size.

+     Also the allocator raises a warning if both values differ in order to report

+     it to driver.

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

+

+2022-06-08 09:02:52 -0400  Nicolas Dufresne <nicolas.dufresne@collabora.com>

+

+   * sys/va/gstvaallocator.c:

+     va: allocator: Fix translation of VADRMPRIMESurfaceDescriptor

+     VADRMPRIMESurfaceDescriptor structure describes the offsets from the

+     point of view of the specific handle (DMABuf). While GstVideoInfo

+     (and the meta) describes offsets from the point of the view of the

+     GstBuffer, an aggregate of all the GstMemory (1 per handle).

+     This changes combined with Mesa Fix(https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/16813)

+     fixes decoding failure with AMD driver.

+     Fixes #1223

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

+

+2022-03-24 21:39:30 +0800  He Junyan <junyan.he@intel.com>

+

+   * sys/va/gstvah265dec.c:

+     va: h265dec: Fix a crash because of missing reference frame.

+     Some problematic H265 stream may miss the reference frame in the DPB,

+     and get some message like: "No short term reference picture for xxx".

+     So there may be empty entries in ref_pic_list0/1 when passing to

+     decode_slice() function of sub class. We need to check the NULL pointer.

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

+

+2022-06-18 04:05:53 +0900  Seungha Yang <seungha@centricular.com>

+

+   * sys/d3d11/gstd3d11decoder.h:

+     d3d11decoder: Check 16K resolution support

+     16K decoding is supported by some GPUs

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

+

+2022-06-15 15:06:20 -0400  Olivier Crête <olivier.crete@collabora.com>

+

+   * ext/webrtc/gstwebrtcbin.c:

+     webrtcbin: Limit sink query to sink pads

+     This allows the reception of streams that don't exactly match

+     the codec preferences. In particular, the ssrc in the codec preferences

+     is local sender SSRC, the other side is expected to send a different SSRC.

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

+

+2022-06-16 00:59:00 +0100  Tim-Philipp Müller <tim@centricular.com>

+

+   * meson.build:

+     Back to development

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

+

 === release 1.20.3 ===

 2022-06-15 23:36:18 +0100  Tim-Philipp Müller <tim@centricular.com>

+   * ChangeLog:

    * NEWS:

    * RELEASE:

    * gst-plugins-bad.doap:

​

@@ -2,13 +2,13 @@

 GStreamer 1.20.0 was originally released on 3 February 2022.

-The latest bug-fix release in the 1.20 series is 1.20.3 and was released

-on 15 June 2022.

+The latest bug-fix release in the 1.20 series is 1.20.4 and was released

+on 12 October 2022.

 See https://gstreamer.freedesktop.org/releases/1.20/ for the latest

 version of this document.

-Last updated: Monday 15 June 2022, 17:00 UTC (log)

+Last updated: Wednesday 12 October 2022, 15:00 UTC (log)

 Introduction

@@ -757,7 +757,8 @@

 -   matroskamux: accept in-band SPS/PPS/VPS for H.264 and H.265

     (i.e. stream-format avc3 and hev1) which allows on-the-fly

-    profile/level/resolution changes.

+    profile/level changes, and from 1.20.4 onwards also resolution

+    changes.

 -   matroskamux: new "cluster-timestamp-offset" property, useful for use

     cases where the container timestamps should map to some absolute

@@ -909,7 +910,7 @@

 -   vp8enc: finish support for temporal scalability: two new properties

     ("temporal-scalability-layer-flags",

     "temporal-scalability-layer-sync-flags") and a unit change on the

-    "temporal-scalability-target-bitrate" property (now expects bps);

+    "temporal-scalability-target-bitrate” property (now expects bps);

     also make temporal scalability details available to RTP payloaders

     as buffer metadata.

@@ -1754,7 +1755,7 @@

     names of internal elements anyway.

 -   The vp8enc element now expects bps (bits per second) for the

-    "temporal-scalability-target-bitrate" property, which is consistent

+    "temporal-scalability-target-bitrate” property, which is consistent

     with the "target-bitrate" property. Since additional configuration

     is required with modern libvpx to make temporal scaling work anyway,

     chances are that very few people will have been using this property

@@ -2503,15 +2504,293 @@

 -   List of Merge Requests applied in 1.20.3

 -   List of Issues fixed in 1.20.3

+1.20.4

+

+The fourth 1.20 bug-fix release (1.20.4) was released on 12 October

+2022.

+

+This release only contains bugfixes and it should be safe to upgrade

+from 1.20.x.

+

+Highlighted bugfixes in 1.20.4

+

+-   avaudiodec: fix playback issue with WMA files, would throw an error

+    at EOS with FFmpeg 5.x

+-   Fix deadlock when loading gst-editing-services plugin

+-   Fix input buffering capacity in live mode for aggregator,

+    video/audio aggregator subclasses, muxers

+-   glimagesink: fix crash on Android

+-   subtitle handling and subtitle overlay fixes

+-   matroska-mux: allow width + height changes for avc3|hev1|vp8|vp9

+-   rtspsrc: fix control url handling for spec compliant servers and add

+    fallback for incompliant servers

+-   WebRTC fixes

+-   RTP retransmission fixes

+-   video: fixes for formats with 4x subsampling and horizontal co-sited

+    chroma (Y41B, YUV9, YVU9 and IYU9)

+-   macOS build and packaging fixes, in particular fix finding of gio

+    modules on macOS for https/TLS support

+-   Performance improvements

+-   Miscellaneous bug fixes, memory leak fixes, and other stability and

+    reliability improvements

+

+gstreamer

+

+-   buffer: drop parent meta in deep copy/foreach_metadata

+-   devicemonitor: Use a sync bus handler for the provider to avoid

+    accumulating all messages until the provider is stopped

+-   element: Fix requesting of pads with string templates

+-   gst: Protect initialization state with a recursive mutex

+-   gst: add missing define guard for build without gstreamer debug

+    logging support

+-   gst_init: Initialize static plugins just before dynamic plugins

+-   info: Parse “NONE” as a valid level name

+-   meta: Set the parent refcount of the GstStructure correctly

+-   pluginloader: Don’t hang on short reads/writes

+-   tracers: leaks: fix potentially invalid memory access when trying to

+    detect object type

+-   tracers: leaks: fix object-refings.class flags

+-   uri: When setting the same string again do nothing

+-   value: Don’t loop forever when serializing invalid flag

+

+Base Libraries

+

+-   aggregator: fix input buffering in live mode (was too low before in

+    many cases)

+-   aggregator: fix reversed active/flushing arguments in debug log

+    output

+-   aggregator: Reset EOS flag after receiving a stream-start event

+

+Core Elements

+

+-   queue2: Hold the lock when modifying sinkresult

+-   queue2: Fix deadlock when deactivate is called in pull mode

+

+gst-plugins-base

+

+-   decodebin3: fix mutex leaks

+-   decodebin3: Fix memory issues with active selection list

+-   decodebin3, uridecodebin3, urisourcebin: Event handling fixes

+-   decodebin3: fix EOS event sequence

+-   parsebin: Avoid crash with unknown streams

+-   parsebin: SIGSEGV during HLS stream using souphttpsrc

+-   glimagesink: only allow setting the GL display/context if it is a

+    valid value

+-   glimagesink: segfault on android devices

+-   gstgl: Fix several memory leaks in macOS

+-   opusenc: improve inband-fec property documentation

+-   playsink: Hold a reference to the soft volume element

+-   pbutils: descriptions: fix gst_pb_utils_get_caps_description_flags()

+-   rtspurl: Use gst_uri_join_strings() in

+    gst_rtsp_url_get_request_uri_with_control() instead of a

+    hand-crafted, wrong version

+-   rtspconnection: protect cancellable by a mutex

+-   sdpmessage: Don’t set SDP medias from caps without

+    media/payload/clock-rate fields

+-   samiparse: fix handling of self-closing tags

+-   ssaparse: include required system headers for isspace() and sscanf()

+    functions

+-   subparse: fix crash when parsing invalid timestamps in mpl2

+-   subparse fixes

+-   textoverlay: Don’t miscalculate text running times

+-   videoaggregator: always convert when user provides converter-config

+-   video: Fix scaling in 4x horizontal co-sited chroma (Y41B, YUV9,

+    YVU9 and IYU9)

+-   xmptag: register musicbrainz tags during init to fix critical in

+    jpegparse

+-   xvimagesink: fix image leaks in error code path

+-   tests: skip unit tests for dependency-less elements that have been

+    disabled

+

+Tools

+

+-   No changes

+

+gst-plugins-good

+

+-   alpha: fix stride issue when out buffer has padding on right

+-   isoff: Fix earliest pts field parse issue

+-   matroska-mux: allow width + height changes for avc3|hev1|vp8|vp9

+-   qt: Fix another instance of Qt/GStreamer both defining GLsync

+    differently

+-   qtdemux: Avoid crash on reconfiguring.

+-   qtdemux: guard against timestamp calculation overflow in gap event

+    loop

+-   qtdemux: Don’t use invalid values from failed trex parsing

+-   qtdemux: possible endless loop

+-   rtpjitterbuffer: Only unschedule timers for late packets if they’re

+    not RTX packets and only once

+-   rtpjitterbuffer: remove lost timer for out of order packets

+-   rtspsrc: SETUP generates 400 Bad Request

+-   rtspsrc: Retry SETUP with non-compliant URL resolution on “Bad

+    Request” and “Not found”

+-   rtpst2022-1-fecenc: Drain column packets on EOS

+-   rtpvp8depay: If configured to wait for keyframes after packet loss,

+    also do that if incomplete frames are detected

+-   splitmuxsink: Don’t crash on EOS without buffer

+-   splitmuxsrc: Stop pad task before cleanup

+-   splitmuxsrc: don’t consider unlinked pads when deactivating part

+-   soup: libsoup3 makes audio streaming stop

+-   v4l2: fix critical when unreferencign buffer with no data

+-   v4l2bufferpool: Fix debug trace

+-   v4l2object: Add support for Apple’s full-range bt709 colorspace

+    variant 1:3:5:1

+-   v4l2videocodec: workaround for failure to fully drain frames

+    preceding MIDSTREAM renegotiation

+-   v4l2allocator: Fix invalid imported dmabuf fd

+-   videoflip: Fix caps negotiation when method is selected

+-   build failure trying to build jack examples

+-   examples: don’t try and build jack examples if jack was disabled

+-   tests: skip unit tests for dependency-less elements that have been

+    disabled

+

+gst-plugins-bad

+

+-   amcvideodec: fix GstAmcSurfaceTexture segfault

+-   audiobuffersplit: Fix drift that was introduced by wrong

+    calculations in gapless mode

+-   avfvideosrc: Fix wrong default framerate value

+-   audiovisualizer: fix buffer mapping to not increase refcount

+-   d3d11decoder: Check 16K resolution support

+-   d3d11videosink: Fix for force-aspect-ratio setting when rendering on

+    shared texture

+-   mxfdemux: Always calculate BlockAlign of raw audio to work around

+    files with broken BlockAlign field in the headers

+-   nvdec: Fix for HEVC decoding when coded resolution is larger than

+    display resolution

+-   openh264: Register debug categories earlier

+-   openh264enc: Fix constrained-high encoding

+-   openmpt: update from now deprecated api

+-   GstPlay: missing cleanup for g_autoptr

+-   player/play: Fix object construction and various leaks

+-   player: Plug a memory leak

+-   proxysink: Make sure stream-start and caps events are forwarded, and

+    fix memory leak

+-   tsdemux: Don’t trigger a program change when falling back to

+    ignore-pcr behaviour

+-   va: allocator: Fix translation of VADRMPRIMESurfaceDescriptor

+-   va: h265dec: Fix a crash because of missing reference frame.

+-   vah265dec: Decoder segfaults on seek

+-   wasapi: Implement default audio channel mask

+-   wasapi2: Fix initial mute/volume setting

+-   webrtcbin: Limit sink query to sink pads

+-   webrtcbin: Fix pointer dereference before null check

+-   webrtc: Make sure to return NULL when validating TURN server fails

+-   tests: skip unit tests for dependency-less elements that have been

+    disabled

+

+gst-plugins-ugly

+

+-   tests: skip unit tests for dependency-less elements that have been

+    disabled

+

+gst-libav

+

+-   avauddec: fix regression with WMA files, would throw an error at EOS

+-   avauddec: fix unnecessary reconfiguration if the audio layout isn’t

+    specified

+-   libav: Fix for APNG encoder property registration

+-   Failure to decode end of WMA file

+

+gst-rtsp-server

+

+-   gst-rtsp-server: Fix pushing backlog to client

+-   rtsp-server: stream: Don’t loop forever if binding to the multicast

+    address fails

+

+gstreamer-vaapi

+

+-   vaapi: Handle when no encoders/decoders available.

+-   vaapi: Crash in gst_vaapidecode_class_init() when no

+    decoders/encoders available

+

+gstreamer-sharp

+

+-   No changes

+

+gst-omx

+

+-   No changes

+

+gst-python

+

+-   python: Do not call gst_init when it is already is_initialized

+

+gst-editing-services

+

+-   Deadlock in ges because of recursive gst_init() call

+-   ges/gstframepositioner: don’t create one compositor per frame meta

+-   nle: clear seek event properly

+

+gst-examples:

+

+-   examples/webrtc/signalling: Fix compatibility with Python 3.10

+

+Development build environment + gst-full build

+

+-   build: Fix some compiler warnings by upgrading wraps

+-   dv, opusparse: fix duplicate symbols in static build

+-   Fix fedora 36 warnings - OpenSSL 3.0 deprecations + GLib 2.72

+    tls-validation deprecations

+-   Various macOS build fixes

+-   meson: Improve certifi documentation on macOS

+

+Cerbero build tool and packaging changes in 1.20.4

+

+-   Add Ubuntu 22.04 Jammy Jellyfish

+-   Add gst-rtsp-server library to the macOS framework

+-   cerbero: Quick fix for gen-cache breakage

+-   macos: Fix the install_name for the GStreamer framework

+-   Download using powershell on Windows and rework download func

+-   macos: Add arm64 to the metadata for the installer

+-   cerbero: Allow building on Linux ARM64

+-   pkg-config.recipe: Add to core platform files list

+-   git: Fix issue with last security patch

+-   distros: Fix CentOS allowance

+-   cerbero: Print working directory for commands that are run

+-   cerbero: Fix license property usage example

+-   Fix issue getting distro_version in Debian Bookworm

+-   glib: Fix gio modules loading on macOS

+-   cmake: Fix macOS ARM64 -> x86_64 cross-compilation

+-   Fix logo display in macOS installer

+-   openssl.recipe: Fix segfault on latest macOS

+-   msvc: Fix for broken CRT linking at application project because of

+    MSVCRT linking

+-   cerbero: Do not add rpaths that already exist on macOS

+-   android: fix build with android gradle plugin 7.2

+-   macOS framework is unusable starting from 1.18.0

+

+Contributors to 1.20.4

+

+Adrian Fiergolski, Aleksandr Slobodeniuk, Andoni Morales Alastruey,

+Andrew Pritchard, Bruce Liang, Corentin Damman, Daniel Morin, Edward

+Hervey, Elliot Chen, Fabian Orccon, fduncanh, Guillaume Desmottes,

+Haihua Hu, He Junyan, Ignazio Pillai, James Cowgill, James Hilliard, Jan

+Alexander Steffens (heftig), Jan Schmidt, Jianhui Dai, Jonas Danielsson,

+Jordan Petridis, Khem Raj, Krystian Wojtas, Martin Dørum, Mart Raudsepp,

+Mathieu Duponchelle, Matthew Waters, Matthias Clasen, Nicolas Dufresne,

+Nirbheek Chauhan, Olivier Crête, Paweł Stawicki, Philippe Normand,

+Philipp Zabel, Piotr Brzeziński, Rafael Caricio, Rafael Sobral, Raul

+Tambre, Ruben Gonzalez, Sangchul Lee, Sebastian Dröge, Seungha Yang,

+Stéphane Cerveau, Thibault Saunier, Tim-Philipp Müller, Tristan

+Matthews, Víctor Manuel Jáquez Leal, Xavier Claessens, Zhiyuan Liu,

+

+… and many others who have contributed bug reports, translations, sent

+suggestions or helped testing. Thank you all!

+

+List of merge requests and issues fixed in 1.20.4

+

+-   List of Merge Requests applied in 1.20.4

+-   List of Issues fixed in 1.20.4

+

 Schedule for 1.22

 Our next major feature release will be 1.22, and 1.21 will be the

 unstable development version leading up to the stable 1.22 release. The

 development of 1.21/1.22 will happen in the git main branch.

-The plan for the 1.22 development cycle is yet to be confirmed. Assuming

-no major project-wide reorganisations in the 1.22 cycle we might try and

-aim for a release around August 2022.

+The plan for the 1.22 development cycle is now confirmed, and we aim for

+a 1.22.0 release in December 2022.

 1.22 will be backwards-compatible to the stable 1.20, 1.18, 1.16, 1.14,

 1.12, 1.10, 1.8, 1.6, 1.4, 1.2 and 1.0 release series.

​

@@ -1,4 +1,4 @@

-This is GStreamer gst-plugins-bad 1.20.3.

+This is GStreamer gst-plugins-bad 1.20.4.

 The GStreamer team is thrilled to announce a new major feature release

 of your favourite cross-platform multimedia framework!

​

@@ -221,14 +221,24 @@

 #if OPENSSL_VERSION_NUMBER < 0x10100001L

   rsa = RSA_generate_key (2048, RSA_F4, NULL, NULL);

 #else

+  /*

+   * OpenSSL 3.0 deprecated all low-level APIs, so we need to rewrite this code

+   * to get rid of the warnings. The porting guide explicitly recommends

+   * disabling the warnings if this is not feasible, so let's do that for now:

+   * https://wiki.openssl.org/index.php/OpenSSL_3.0#Upgrading_to_OpenSSL_3.0_from_OpenSSL_1.1.1

+   */

+  G_GNUC_BEGIN_IGNORE_DEPRECATIONS;

   rsa = RSA_new ();

+  G_GNUC_END_IGNORE_DEPRECATIONS;

   if (rsa != NULL) {

     BIGNUM *e = BN_new ();

+    G_GNUC_BEGIN_IGNORE_DEPRECATIONS;

     if (e == NULL || !BN_set_word (e, RSA_F4)

         || !RSA_generate_key_ex (rsa, 2048, e, NULL)) {

       RSA_free (rsa);

       rsa = NULL;

     }

+    G_GNUC_END_IGNORE_DEPRECATIONS;

     if (e)

       BN_free (e);

   }

@@ -236,16 +246,20 @@

   if (!rsa) {

     GST_WARNING_OBJECT (self, "failed to generate RSA");

+    G_GNUC_BEGIN_IGNORE_DEPRECATIONS;

     EVP_PKEY_free (priv->private_key);

+    G_GNUC_END_IGNORE_DEPRECATIONS;

     priv->private_key = NULL;

     X509_free (priv->x509);

     priv->x509 = NULL;

     return;

   }

+  G_GNUC_BEGIN_IGNORE_DEPRECATIONS;

   if (!EVP_PKEY_assign_RSA (priv->private_key, rsa)) {

     GST_WARNING_OBJECT (self, "failed to assign RSA");

     RSA_free (rsa);

+    G_GNUC_END_IGNORE_DEPRECATIONS;

     rsa = NULL;

     EVP_PKEY_free (priv->private_key);

     priv->private_key = NULL;

@@ -259,7 +273,9 @@

   /* Set a random 64 bit integer as serial number */

   serial_number = BN_new ();

+  G_GNUC_BEGIN_IGNORE_DEPRECATIONS;

   BN_pseudo_rand (serial_number, 64, 0, 0);

+  G_GNUC_END_IGNORE_DEPRECATIONS;

   asn1_serial_number = X509_get_serialNumber (priv->x509);

   BN_to_ASN1_INTEGER (serial_number, asn1_serial_number);

   BN_free (serial_number);

​

@@ -86,10 +86,7 @@

 /* class initialization */

-G_DEFINE_TYPE_WITH_CODE (GstOpenh264Dec, gst_openh264dec,

-    GST_TYPE_VIDEO_DECODER,

-    GST_DEBUG_CATEGORY_INIT (gst_openh264dec_debug_category, "openh264dec", 0,

-        "debug category for openh264dec element"));

+G_DEFINE_TYPE (GstOpenh264Dec, gst_openh264dec, GST_TYPE_VIDEO_DECODER);

 GST_ELEMENT_REGISTER_DEFINE_CUSTOM (openh264dec, openh264dec_element_init);

 static void

@@ -455,10 +452,12 @@

 static gboolean

 openh264dec_element_init (GstPlugin * plugin)

 {

+  GST_DEBUG_CATEGORY_INIT (gst_openh264dec_debug_category, "openh264dec", 0,

+      "debug category for openh264dec element");

   if (openh264_element_init (plugin))

     return gst_element_register (plugin, "openh264dec", GST_RANK_MARGINAL,

         GST_TYPE_OPENH264DEC);

- GST_ERROR ("Incorrect library version loaded, expecting %s", g_strCodecVer);

- return FALSE;

+  GST_ERROR ("Incorrect library version loaded, expecting %s", g_strCodecVer);

+  return FALSE;

 }

​

@@ -234,10 +234,7 @@

 /* class initialization */

 G_DEFINE_TYPE_WITH_CODE (GstOpenh264Enc, gst_openh264enc,

-    GST_TYPE_VIDEO_ENCODER,

-    G_IMPLEMENT_INTERFACE (GST_TYPE_PRESET, NULL);

-    GST_DEBUG_CATEGORY_INIT (gst_openh264enc_debug_category, "openh264enc", 0,

-        "debug category for openh264enc element"));

+    GST_TYPE_VIDEO_ENCODER, G_IMPLEMENT_INTERFACE (GST_TYPE_PRESET, NULL));

 GST_ELEMENT_REGISTER_DEFINE_CUSTOM (openh264enc, openh264enc_element_init);

 static void

@@ -711,11 +708,12 @@

   gst_structure_set (s, "profile", G_TYPE_STRING, profile, NULL);

   if (!g_strcmp0 (profile, "constrained-baseline") ||

-       !g_strcmp0 (profile, "baseline"))

+      !g_strcmp0 (profile, "baseline"))

     return PRO_BASELINE;

-   else if (!g_strcmp0 (profile, "main"))

+  else if (!g_strcmp0 (profile, "main"))

     return PRO_MAIN;

-   else if (!g_strcmp0 (profile, "high"))

+  else if (!g_strcmp0 (profile, "high") ||

+      !g_strcmp0 (profile, "constrained-high"))

     return PRO_HIGH;

   g_assert_not_reached ();

@@ -1056,13 +1054,16 @@

   return GST_FLOW_OK;

 }

+

 static gboolean

 openh264enc_element_init (GstPlugin * plugin)

 {

+  GST_DEBUG_CATEGORY_INIT (gst_openh264enc_debug_category, "openh264enc", 0,

+      "debug category for openh264enc element");

   if (openh264_element_init (plugin))

     return gst_element_register (plugin, "openh264enc", GST_RANK_MARGINAL,

-                                 GST_TYPE_OPENH264ENC);

+        GST_TYPE_OPENH264ENC);

- GST_ERROR ("Incorrect library version loaded, expecting %s", g_strCodecVer);

- return FALSE;

+  GST_ERROR ("Incorrect library version loaded, expecting %s", g_strCodecVer);

+  return FALSE;

 }

​

@@ -562,8 +562,14 @@

    * need to query it here, *before* any openmpt_module_select_subsong()

    * calls are done */

   {

+

+#if OPENMPT_API_VERSION_AT_LEAST(0,5,0)

+    gchar const *subsong_cstr =

+        openmpt_module_ctl_get_text (openmpt_dec->mod, "subsong");

+#else

     gchar const *subsong_cstr =

         openmpt_module_ctl_get (openmpt_dec->mod, "subsong");

+#endif

     gchar *endptr;

     if (subsong_cstr != NULL) {

​

@@ -26,6 +26,10 @@

 G_BEGIN_DECLS

+#define gst_opus_header_is_header gst_opusparse_header_is_header

+#define gst_opus_header_is_id_header gst_opusparse_header_is_id_header

+#define gst_opus_header_is_comment_header gst_opusparse_header_is_comment_header

+

 extern gboolean gst_opus_header_is_header (GstBuffer * buf,

     const char *magic, guint magic_size);

 extern gboolean gst_opus_header_is_id_header (GstBuffer * buf);

​

@@ -31,7 +31,6 @@

         '-Wno-missing-declarations',

         '-Wno-old-style-definition',

         '-Wno-redundant-decls',

-        '-Wno-error',

     )

 endif

@@ -170,6 +169,7 @@

     c_args: compile_args,

     dependencies: dependencies,

     include_directories: include_dirs,

+    override_options: 'werror=false',

     install: false)

 # Declare dependency

​

@@ -488,8 +488,10 @@

       direction, "template", template, NULL);

   gst_object_unref (template);

-  gst_pad_set_event_function (GST_PAD (pad), gst_webrtcbin_sink_event);

-  gst_pad_set_query_function (GST_PAD (pad), gst_webrtcbin_sink_query);

+  if (direction == GST_PAD_SINK) {

+    gst_pad_set_event_function (GST_PAD (pad), gst_webrtcbin_sink_event);

+    gst_pad_set_query_function (GST_PAD (pad), gst_webrtcbin_sink_query);

+  }

   gst_pad_add_probe (GST_PAD (pad), GST_PAD_PROBE_TYPE_BUFFER |

       GST_PAD_PROBE_TYPE_BUFFER_LIST, webrtc_bin_pad_buffer_cb, NULL, NULL);

@@ -3085,7 +3087,15 @@

     /* this only looks at the first structure so we loop over the given caps

      * and add each structure inside it piecemeal */

-    gst_sdp_media_set_media_from_caps (format, media);

+    if (gst_sdp_media_set_media_from_caps (format, media) != GST_SDP_OK) {

+      GST_ERROR_OBJECT (webrtc,

+          "Failed to build media from caps %" GST_PTR_FORMAT

+          " for transceiver %" GST_PTR_FORMAT, format, trans);

+      gst_caps_unref (caps);

+      gst_caps_unref (format);

+      gst_structure_free (extmap);

+      return FALSE;

+    }

     gst_caps_unref (format);

   }

@@ -4125,7 +4135,13 @@

         }

       }

-      gst_sdp_media_set_media_from_caps (answer_caps, media);

+      if (gst_sdp_media_set_media_from_caps (answer_caps, media) != GST_SDP_OK) {

+        GST_WARNING_OBJECT (webrtc,

+            "Could not build media from caps %" GST_PTR_FORMAT, answer_caps);

+        gst_clear_caps (&answer_caps);

+        gst_clear_caps (&offer_caps);

+        goto rejected;

+      }

       _get_rtx_target_pt_and_ssrc_from_caps (answer_caps, &target_pt,

           &target_ssrc);

@@ -5746,6 +5762,12 @@

         continue;

       }

+      if (!pad->trans) {

+        GST_LOG_OBJECT (pad, "doesn't have a transceiver");

+        tmp = tmp->next;

+        continue;

+      }

+

       if (pad->trans->mline >= gst_sdp_message_medias_len (sd->sdp->sdp)) {

         GST_DEBUG_OBJECT (pad, "not mentioned in this description. Skipping");

         tmp = tmp->next;

@@ -5761,12 +5783,6 @@

         tmp = tmp->next;

         continue;

       }

-

-      if (!pad->trans) {

-        GST_LOG_OBJECT (pad, "doesn't have a transceiver");

-        tmp = tmp->next;

-        continue;

-      }

       new_dir = pad->trans->direction;

       if (new_dir != GST_WEBRTC_RTP_TRANSCEIVER_DIRECTION_SENDONLY &&

​

@@ -928,12 +928,19 @@

     }

   }

-out:

   g_list_free (keys);

   g_free (user);

   g_free (pass);

   return uri;

+

+out:

+  g_list_free (keys);

+  g_free (user);

+  g_free (pass);

+  gst_uri_unref (uri);

+

+  return NULL;

 }

 void

​

@@ -215,6 +215,10 @@

 typedef struct _GstPlayMediaInfo GstPlayMediaInfo;

 typedef struct _GstPlayMediaInfoClass GstPlayMediaInfoClass;

+#ifdef G_DEFINE_AUTOPTR_CLEANUP_FUNC

+G_DEFINE_AUTOPTR_CLEANUP_FUNC(GstPlayMediaInfo, g_object_unref)

+#endif

+

 GST_PLAY_API

 GType         gst_play_media_info_get_type (void);

​

@@ -39,6 +39,10 @@

  */

 #define GST_PLAY_SIGNAL_ADAPTER_CAST(obj)        ((GstPlaySignalAdapter*)(obj))

+#ifdef G_DEFINE_AUTOPTR_CLEANUP_FUNC

+G_DEFINE_AUTOPTR_CLEANUP_FUNC(GstPlaySignalAdapter, g_object_unref)

+#endif

+

 GST_PLAY_API

 GType                  gst_play_signal_adapter_get_type               (void);

​

@@ -48,6 +48,10 @@

  */

 #define GST_PLAY_VIDEO_OVERLAY_VIDEO_RENDERER_CAST(obj)        ((GstPlayVideoOverlayVideoRenderer*)(obj))

+#ifdef G_DEFINE_AUTOPTR_CLEANUP_FUNC

+G_DEFINE_AUTOPTR_CLEANUP_FUNC(GstPlayVideoOverlayVideoRenderer, g_object_unref)

+#endif

+

 GST_PLAY_API

 GType gst_play_video_overlay_video_renderer_get_type (void);

​

@@ -49,6 +49,10 @@

   GstElement * (*create_video_sink) (GstPlayVideoRenderer * self, GstPlay * play);

 };

+#ifdef G_DEFINE_AUTOPTR_CLEANUP_FUNC

+G_DEFINE_AUTOPTR_CLEANUP_FUNC(GstPlayVideoRenderer, g_object_unref)

+#endif

+

 GST_PLAY_API

 GType        gst_play_video_renderer_get_type       (void);

​

@@ -56,6 +56,10 @@

 GST_PLAY_API

 void                      gst_play_visualizations_free (GstPlayVisualization **viss);

+#ifdef G_DEFINE_AUTOPTR_CLEANUP_FUNC

+G_DEFINE_AUTOPTR_CLEANUP_FUNC(GstPlayVisualization, gst_play_visualization_free)

+#endif

+

 G_END_DECLS

 #endif /* __GST_PLAY_VISUALIZATION_H__ */

​

@@ -212,6 +212,8 @@

 static gpointer gst_play_main (gpointer data);

+static void gst_play_set_playbin_video_sink (GstPlay * self);

+

 static void gst_play_seek_internal_locked (GstPlay * self);

 static void gst_play_stop_internal (GstPlay * self, gboolean transient);

 static gboolean gst_play_pause_internal (gpointer user_data);

@@ -509,6 +511,8 @@

   self->thread = g_thread_new ("GstPlay", gst_play_main, self);

   while (!self->loop || !g_main_loop_is_running (self->loop))

     g_cond_wait (&self->cond, &self->lock);

+

+  gst_play_set_playbin_video_sink (self);

   g_mutex_unlock (&self->lock);

   G_OBJECT_CLASS (parent_class)->constructed (object);

@@ -594,11 +598,16 @@

 {

   GstElement *video_sink = NULL;

-  if (self->video_renderer != NULL)

+  if (self->video_renderer != NULL) {

     video_sink =

         gst_play_video_renderer_create_video_sink (self->video_renderer, self);

-  if (video_sink)

+  }

+

+  if (video_sink) {

+    gst_object_ref_sink (video_sink);

     g_object_set (self->playbin, "video-sink", video_sink, NULL);

+    gst_object_unref (video_sink);

+  }

 }

 static void

@@ -612,7 +621,14 @@

       g_mutex_lock (&self->lock);

       g_clear_object (&self->video_renderer);

       self->video_renderer = g_value_dup_object (value);

-      gst_play_set_playbin_video_sink (self);

+

+      // When the video_renderer is a GstPlayerWrappedVideoRenderer it cannot be set

+      // at construction time because it requires a valid pipeline which is created

+      // only after GstPlay has been constructed. That is why the video renderer is

+      // set *after* GstPlay has been constructed.

+      if (self->thread) {

+        gst_play_set_playbin_video_sink (self);

+      }

       g_mutex_unlock (&self->lock);

       break;

     case PROP_URI:{

@@ -2648,15 +2664,8 @@

   g_once (&once, gst_play_init_once, NULL);

-  self = g_object_new (GST_TYPE_PLAY, NULL);

+  self = g_object_new (GST_TYPE_PLAY, "video-renderer", video_renderer, NULL);

-  // When the video_renderer is a GstPlayerWrappedVideoRenderer it cannot be set

-  // at construction time because it requires a valid pipeline which is created

-  // only after GstPlay has been constructed. That is why the video renderer is

-  // set *after* GstPlay has been constructed.

-  if (video_renderer != NULL) {

-    g_object_set (self, "video-renderer", video_renderer, NULL);

-  }

   gst_object_ref_sink (self);

   if (video_renderer)

​

@@ -149,9 +149,21 @@

 }

 static void

+gst_player_video_info_finalize (GObject * object)

+{

+  GstPlayerVideoInfo *info = GST_PLAYER_VIDEO_INFO (object);