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 429476 - [mpeg2dec] can't play auge.mpg
[mpeg2dec] can't play auge.mpg
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-ugly
0.10.5
Other Linux
: Normal normal
: 0.10.19
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2007-04-13 19:14 UTC by Sven Arvidsson
Modified: 2011-08-23 08:05 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
mpeg2dec: fix spurious failure to decode some files (1.23 KB, patch)
2011-08-22 16:51 UTC, Vincent Penquerc'h
committed Details | Review

Description Sven Arvidsson 2007-04-13 19:14:43 UTC
This MPEG file fails to play in Totem,
http://www.tu-chemnitz.de/misc/demo/auge.mpg

** Message: Error: Could not decode stream.
gstmpeg2dec.c(1082): gst_mpeg2dec_chain (): /play/decodebin0/mpeg2dec0

It seems to play fine with ffplay.

GStreamer versions,
gstreamer0.10-ffmpeg     0.10.2-1
gstreamer0.10-gnomevfs   0.10.12-2
gstreamer0.10-plugins-base       0.10.12-2
gstreamer0.10-plugins-good       0.10.5-5
gstreamer0.10-plugins-really-bad         0.10.3-0.5
gstreamer0.10-plugins-ugly       0.10.5-2
gstreamer0.10-x          0.10.12-2
libgstreamer0.10-0       0.10.12-3
libgstreamer0.10-dev     0.10.12-3
Comment 1 Wim Taymans 2007-04-16 08:03:50 UTC
It seems mpeg2dec does not like this file. flump2vdec plays fine.
Comment 2 David Schleef 2007-04-16 18:31:37 UTC
mpeg2dec (the program) plays it fine, so it's likely a bug in the gstreamer element.
Comment 3 Jan Schmidt 2007-04-17 19:07:50 UTC
Ignoring STATE_INVALID and continuing to attempt decoding instead of bailing out makes it work fine. 

I suspect it's a corner case inside libmpeg2 because the video appears to have only 1 slice.

I don't know what we can do to work around this, but it seems the current idea of using STATE_INVALID to increase the error count and skip is flawed - there's no decoding errors in this file.
Comment 4 Wim Taymans 2008-11-04 18:40:23 UTC
I don't think we should use such an error count. We should only produce a fatal error when we have no chance of recovering from a situation.
Comment 5 Edward Hervey 2009-09-09 15:36:07 UTC
ffdec_mpeg2video also plays it fine :(
Comment 6 Christian Fredrik Kalager Schaller 2011-05-26 15:24:43 UTC
Just tested this, bug is still around with current git code.
Comment 7 Vincent Penquerc'h 2011-08-22 16:51:44 UTC
Created attachment 194389 [details] [review]
mpeg2dec: fix spurious failure to decode some files

Parsing can return with an 'invalid' state, but this is not
actually fatal. For one, the mpeg2dec command line tool that
comes with the libmpeg2 library blithely ignores this condition
and merrily goes on. So we do this same, logging the error,
and going on with parsing. This makes something work that did
not use to work, and brings happiness to the world.
Comment 8 Sebastian Dröge (slomo) 2011-08-23 08:05:16 UTC
commit a09bad4d98f513149c4e29c97f82c1b49d583fda
Author: Vincent Penquerc'h <vincent.penquerch@collabora.co.uk>
Date:   Mon Aug 22 17:49:14 2011 +0100

    mpeg2dec: fix spurious failure to decode some files
    
    Parsing can return with an 'invalid' state, but this is not
    actually fatal. For one, the mpeg2dec command line tool that
    comes with the libmpeg2 library blithely ignores this condition
    and merrily goes on. So we do this same, logging the error,
    and going on with parsing. This makes something work that did
    not use to work, and brings happiness to the world.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=429476