[Bug 759778] flacparse: large padding headers cause high memory consumption

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=759778

--- Comment #5 from Carlos Rafael Giani <dv at pseudoterminal.org> ---
If anything, it should be opt-out. As the spec says, the padding header is
mostly useful for when mastering FLAC data, so skipping it is preferable in
playback situations. To me, this means that aside from pipelines where both in-
and output are FLAC and the user wants to preserve the headers, there aren't
any use cases where you want to keep the padding header.

The remaining question is if such FLAC-editing pipelines would still use
decodebin. I think it is a possibility. It is however still possible to
intercept the newly created elements during the autoplugging procedure and set
such a "skip-padding" property to FALSE. So, I'd add a note to flacparse about
this issue.

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