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 622407 - [asfdemux] doesn't detect some streams if preroll value is very small
[asfdemux] doesn't detect some streams if preroll value is very small
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-ugly
0.10.15
Other Linux
: Normal normal
: 0.10.16
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2010-06-22 14:48 UTC by Ben
Modified: 2010-06-24 02:26 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
log: GST_DEBUG_NO_COLOR=1 GST_DEBUG=5 (836.86 KB, application/x-bzip)
2010-06-22 15:39 UTC, Nicolò Chieffo
Details

Description Ben 2010-06-22 14:48:50 UTC
I am forwarding this downstream bug which includes example files: https://bugs.launchpad.net/ubuntu/+source/gstreamer0.10/+bug/597190

The said file plays correctly in VLC but not using totem+gstreamer
Comment 2 Nicolò Chieffo 2010-06-22 15:39:28 UTC
Created attachment 164316 [details]
log: GST_DEBUG_NO_COLOR=1 GST_DEBUG=5

This is the log of opening and starting to play the video.
Comment 3 Thiago Sousa Santos 2010-06-22 16:33:54 UTC
Reproduced it here with git version.
Comment 4 Edward Hervey 2010-06-22 16:37:41 UTC
The preroll value for that file is insanely small, meaning we don't see the first packet of the video stream before we go to preroll... and therefore never expose that pad (thinking the stream is empty).

Maybe we should wait for a minimal number of packets/duration before deciding which pad/streams to activate.
Comment 5 Edward Hervey 2010-06-23 09:07:32 UTC
commit 5ac4ea3f1b77655effba2346bfcaef8f13e7a120
Author: Edward Hervey <bilboed@bilboed.com>
Date:   Wed Jun 23 11:05:11 2010 +0200

    asfdemux: Allow at least 500ms of preroll.
    
    Some files have insanely low preroll values which break the
    all_streams_prerolled() logic.
    
    Fixes #622407
Comment 6 Nicolò Chieffo 2010-06-23 16:06:40 UTC
Sorry to write again, but I've just realized that also the opposite thing might happen: some video files are recognized as video only (no audio).
Does the fix include this case too?
Comment 7 Edward Hervey 2010-06-23 16:32:20 UTC
yes
Comment 8 Ben 2010-06-24 02:26:04 UTC
I tested the code change and it is working for me. Thanks for the quick work!