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 607926 - [oggdemux] regression with certain chained ogg stream
[oggdemux] regression with certain chained ogg stream
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
git master
Other Linux
: Normal blocker
: 0.10.26
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2010-01-24 13:04 UTC by Tim-Philipp Müller
Modified: 2010-01-25 14:16 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
ogg/vorbis stream dump with apparently dodgy headers (999.00 KB, application/octet-stream)
2010-01-24 13:07 UTC, Tim-Philipp Müller
Details

Description Tim-Philipp Müller 2010-01-24 13:04:50 UTC
The attached ogg stream is a dump from mpd's in-built http server.

This plays fine for me with the last core/base release, but not with git:

 gst-launch-0.10 playbin2 uri=file:///path/to/iamfirecracker.ogg
 gst-launch-0.10 playbin2 uri=pushfile:///path/to/iamfirecracker.ogg

With git I get this:

(a)

 gst-launch-0.10 playbin2 uri=file:///path/to/iamfirecracker.ogg

endless loop at 100% cpu at
oggdemux gstoggdemux.c:1343:gst_ogg_demux_seek:<oggdemux0> seeking to 0

(b)

 gst-launch-0.10 playbin2 uri=pushfile:///path/to/iamfirecracker.ogg

Lots of:
gstoggdemux.c:2826:gst_ogg_demux_handle_page:<oggdemux0> warning: unknown ogg pad for serial eaa42c7b detected
and then an error.

oggz-validate shows:
iamfirecracker.ogg: Error:
serialno 18446744073351212155: Terminal header page contains non-header segment


So looks like there's something dodgy going on with the headers handling.
Comment 1 Tim-Philipp Müller 2010-01-24 13:07:12 UTC
Created attachment 152140 [details]
ogg/vorbis stream dump with apparently dodgy headers
Comment 2 Wim Taymans 2010-01-25 14:16:40 UTC
commit e4b68a3658b9830261194a65f09fe431051b23d6
Author: Wim Taymans <wim.taymans@collabora.co.uk>
Date:   Mon Jan 25 15:14:56 2010 +0100

    oggdemux: use right type for the serialno
    
    Use a consistent type for the serialno to avoid problems when comparing between
    signed and unsigned variants.
    
    Fixes #607926