From: Kevin Song Date: Fri, 9 Jun 2023 13:08:36 +0000 (+0800) Subject: appsink: unref gstbuffer in prev sample early X-Git-Tag: 1.22.7~221 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=e8f9bd16490d34c9c8f729c45780f9b7bcdcce56;p=platform%2Fupstream%2Fgstreamer.git appsink: unref gstbuffer in prev sample early Appsink will unref prev sample in dispose function. Which is later when V4L2 video decoder link with appsink as V4L2 video decoder will close V4L2 device fd during GST_STATE_CHANGE_READY_TO_NULL. If the video buffer return to V4L2 video decoder after the decoder closed V4L2 device fd, V4L2 can't release the video frame buffer which allocated with MMAP mode as application can't call VIDIOC_REQBUFS 0 to release the video frame buffer by V4L2 driver. The memory of the video frame will leak. Unref the gstbuffer in stop() function, so V4L2 video decoder can received all video frame buffers and release it before close V4L2 device fd. Part-of: --- diff --git a/subprojects/gst-plugins-base/gst-libs/gst/app/gstappsink.c b/subprojects/gst-plugins-base/gst-libs/gst/app/gstappsink.c index fb6ce0d..86da7f4 100644 --- a/subprojects/gst-plugins-base/gst-libs/gst/app/gstappsink.c +++ b/subprojects/gst-plugins-base/gst-libs/gst/app/gstappsink.c @@ -775,6 +775,11 @@ gst_app_sink_stop (GstBaseSink * psink) gst_caps_replace (&priv->last_caps, NULL); gst_segment_init (&priv->preroll_segment, GST_FORMAT_UNDEFINED); gst_segment_init (&priv->last_segment, GST_FORMAT_UNDEFINED); + priv->sample = gst_sample_make_writable (priv->sample); + gst_sample_set_buffer (priv->sample, NULL); + gst_sample_set_buffer_list (priv->sample, NULL); + gst_sample_set_caps (priv->sample, NULL); + gst_sample_set_segment (priv->sample, NULL); g_mutex_unlock (&priv->mutex); return TRUE;