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

[GStreamer (GNOME Bugzilla)]

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

--- Comment #11 from Reynaldo H. Verdejo Pinochet <reynaldo at opendot.cl> ---
(In reply to Carlos Rafael Giani from comment #10)
> [..]
> earlier). I do not think the "Skipping %u bytes of padding metadata" log
> should use the warning level, since warnings should indicate that something
> unexpected, undefined, incorrect happened that might lead to trouble.
> Perhaps use INFO instead of WARNING.

I understand your concern but IMHO I'm abusing the level for a good
reason: The parser is discarding valid data. The warning is signaling a
switchable (default) action that might lead to (sic) *problems* afterwards ->
The stream being stripped of a - valid - metadata block the user might
actually be relying on.

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