[Bug 684701] flacparse: fails to parse a valid flac file: "Invalid metablock size for STREAMINFO: 8196"

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=684701
  GStreamer | gst-plugins-good | unspecified

--- Comment #4 from Tim-Philipp Müller <t.i.m at zen.co.uk> 2012-10-23 11:37:42 UTC ---
This file is far from "valid". Even flac's own tools choke on it when trying to
analyze/test it.

However, we should still be able to play it.

Basically there's a 8192 bytes block of zero padding that is marked as
STREAMINFO header instead of a padding header, followed by a block of padding
that's actually marked as padding (and has the 'last header' flag set). Then
there's some garbage between that and the actual audio data.

I don't think your patch is right, we need some of those values and if they're
not available we should error out. What's happening here is that we already
have a valid STREAMINFO header, which is all we need to parse the audio date,
so I think if we run into any header parsing issues after that we should just
go straight into 'sync to audio frame' mode.

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