From: Edward Hervey <edward.hervey@collabora.co.uk>
Date: Tue, 1 May 2012 14:11:58 +0000 (+0200)
Subject: videoencoder: Add a reference to frame passed to subclass
X-Git-Tag: 1.19.3~511^2~6512
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=67a4c0c7b21379173073416ef5bf495409503e6d;p=platform%2Fupstream%2Fgstreamer.git

videoencoder: Add a reference to frame passed to subclass

We have one reference owned by the internal frame list and one reference
passed to the subclass.
---

diff --git a/gst-libs/gst/video/gstvideoencoder.c b/gst-libs/gst/video/gstvideoencoder.c
index 933bb9b..5e5527c 100644
--- a/gst-libs/gst/video/gstvideoencoder.c
+++ b/gst-libs/gst/video/gstvideoencoder.c
@@ -1185,6 +1185,7 @@ gst_video_encoder_chain (GstPad * pad, GstObject * parent, GstBuffer * buf)
   GST_LOG_OBJECT (encoder, "passing frame pfn %d to subclass",
       frame->presentation_frame_number);
 
+  gst_video_codec_frame_ref (frame);
   ret = klass->handle_frame (encoder, frame);
 
 done:
@@ -1496,7 +1497,9 @@ gst_video_encoder_finish_frame (GstVideoEncoder * encoder,
 done:
   /* handed out */
   priv->frames = g_list_remove (priv->frames, frame);
-
+  /* Remove the reference from the list and the reference that
+   * was provided to us */
+  gst_video_codec_frame_unref (frame);
   gst_video_codec_frame_unref (frame);
 
   GST_VIDEO_ENCODER_STREAM_UNLOCK (encoder);