[Bug 654529] New: [basevideodecoder] Does not flush pending input frames when flushing

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=654529
  GStreamer | gst-plugins-bad | 0.10.13

           Summary: [basevideodecoder] Does not flush pending input frames
                    when flushing
    Classification: Platform
           Product: GStreamer
           Version: 0.10.13
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: critical
          Priority: Normal
         Component: gst-plugins-bad
        AssignedTo: gstreamer-bugs at lists.freedesktop.org
        ReportedBy: slomo at circular-chaos.org
         QAContact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---


For example a flushing seek does not clean all GstBaseVideoCodec::frames, which
have a reference to the input buffers. After lots of seeks this can lead to a
huge memory usage just because of obsolete frames.

There's a FIXME comment about this in gst_base_video_decoder_flush():

  /* FIXME perhaps also clear pending frames ?,
   * but again, subclass may still come up with one of those */


IMHO subclasses must reset all their internal state too and not pass any of the
old frames after resetting.

-- 
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.