After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 590470 - [typefinding] certain flac-in-ogg files not detected any longer
[typefinding] certain flac-in-ogg files not detected any longer
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
git master
Other Linux
: Normal blocker
: 0.10.24
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2009-08-01 16:32 UTC by Tim-Philipp Müller
Modified: 2009-08-01 18:09 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
fix flac detection in case of 4-byte ogg ID packet (2.98 KB, patch)
2009-08-01 16:32 UTC, Tim-Philipp Müller
committed Details | Review

Description Tim-Philipp Müller 2009-08-01 16:32:10 UTC
Playback of certain flac-in-ogg files is broken because oggdemux doesn't recognise them properly any longer. These files aren't framed according to the ogg mapping spec, but basically just contain the flac bytestream in ogg packets. These files then have a 4-byte packet as first ogg packet, and we need to detect the right type from these 4 bytes. The typefinder, however, requires at least 6 bytes and hence fails to detect the stream properly.

Sample file: samples-mplayer/flac.samples/Yesterday.ogg
Comment 1 Tim-Philipp Müller 2009-08-01 16:32:55 UTC
Created attachment 139695 [details] [review]
fix flac detection in case of 4-byte ogg ID packet
Comment 2 Tim-Philipp Müller 2009-08-01 18:09:51 UTC
Thanks, committed:

 commit e199d7e1cd25eee6d7d3314f83bdda461eb1b509
 Author: Tim-Philipp Müller <tim.muller@collabora.co.uk>
 Date:   Sat Aug 1 17:26:23 2009 +0100

    typefinding: fix detection of fLaC id packet in broken flac-in-ogg
    
    There are flac-in-ogg files without the usual flac packet framing
    and these files just have a 4-byte fLaC ID packet as first packet.
    We need to recognise the type just from these four bytes if we
    want oggdemux to recognise these streams correctly.