[gstreamer-bugs] [Bug 301783] [PATCH] enable our get/release_buffer functions in ffmpegdec

[bugzilla-daemon at bugzilla.gnome.org]

Please DO NOT reply to this by email. All additional comments should be made in
the comments box of this bug report.

 http://bugzilla.gnome.org/show_bug.cgi?id=301783
 GStreamer | gst-ffmpeg | Ver: HEAD CVS





------- Additional Comments From Luca Ognibene  2005-05-04 09:36 -------
Created an attachment (id=46008)
 --> (http://bugzilla.gnome.org/attachment.cgi?id=46008&action=view)
fix negotiation and a memory leak

Q) What's last_buffer and why is it needed? 
A) I've tried using this patch with a program (that uses ffdec_mjpeg) and i've
seen a big memory leak, debugging this problem i've found that:
 - ffdec calls avcodec_decode_video
 - ffmpeg calls release_buffer 
 - ffmpeg calls get_buffer
 - ffdec push the buffer

So if i change ffdec status from PLAYING to READY the last buffer is not
unreffed. This patch just unrefs it.

------- You are receiving this mail because: -------
You are the assignee for the bug.
You are the QA contact for the bug.