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 631416 - mpegpsdemux hits EOS too early
mpegpsdemux hits EOS too early
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
git master
Other Linux
: Normal normal
: 0.10.23
Assigned To: GStreamer Maintainers
GStreamer Maintainers
: 643458 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2010-10-05 13:18 UTC by Robert Swain
Modified: 2011-11-07 18:47 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
mpegpsdemux: fix zero length packet handling (1.03 KB, patch)
2011-10-03 17:30 UTC, Vincent Penquerc'h
committed Details | Review

Description Robert Swain 2010-10-05 13:18:42 UTC
With the following sample [1], when using filesrc ! mpegpsdemux ! mpeg2dec ! autovideosink, the video hits EOS too early. Using mpegpsdemux, it hits EOS just before the fight starts after the woman looks around them all. Using dvddemux from ugly, it works perfectly and continues on to show a man in blue shouting something and then the fight beginning. This also plays perfectly in FFmpeg, MPlayer and VLC.

I actually noticed this because using dvdreadsrc on a VIDEO_TS directory containing the vob from which this sample was taken, when it hits the same point, the RAM usage inflates massively and system I/O is hit heavily. I'm guessing it's searching through the data to find a new sync or something and can't.

[1] http://people.freedesktop.org/~robswain/mpegpsdemux_bug.vob
Comment 1 Vincent Penquerc'h 2011-10-03 17:30:15 UTC
Created attachment 198125 [details] [review]
mpegpsdemux: fix zero length packet handling
Comment 2 Vincent Penquerc'h 2011-10-04 12:11:27 UTC
*** Bug 643458 has been marked as a duplicate of this bug. ***
Comment 3 Andreas Frisch 2011-10-04 12:40:47 UTC
i would like to acknowledge without any further investigation on the exact details, that your patch fixes a similar issue which i've had in Bug 643458
Comment 4 Vincent Penquerc'h 2011-11-07 11:18:14 UTC
commit b3d751667d08995332e211bd01825a7a165d7b44
Author: Vincent Penquerc'h <vincent.penquerch@collabora.co.uk>
Date:   Mon Oct 3 18:28:14 2011 +0100

    mpegpsdemux: fix zero length packet handling
    
    https://bugzilla.gnome.org/show_bug.cgi?id=631416