[Bug 693772] avdec: decoder frame list getting long

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=693772
  GStreamer | gst-libav | git

--- Comment #19 from Mark Nauwelaerts <mnauw at users.sourceforge.net> 2013-11-28 21:25:05 UTC ---
Thanks for the review and comments.

I think/hope it should also work for the interlaced H264 case.  The intention
at least is to resolve any leaking (other than possibly due to libav), while
also compensating for some other parsing glitches etc.

I am also thinking of introducing a GST_VIDEO_CODEC_FRAME_FLAG_NO_FRAME flag
(akin to the baseparse _NO_FRAME flag) that signals a "frame" to be even more
discardable than _DECODE_ONLY.  That is, it allows making this frame cleaning
re-usable by having it done by the baseclass for frames marked as _NO_FRAME
(which also avoids the frame list copying in addition to being reusable).

Also, coming to think of it, it is probably safer/required to have
_release_frame take _STREAM_LOCK when manipulating the frame list.

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.