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 582056 - [asfdemux] forcing an EOS on the pipeline doesn't work with wmv files
[asfdemux] forcing an EOS on the pipeline doesn't work with wmv files
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-ugly
git master
Other All
: Normal minor
: 0.10.12
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2009-05-10 11:17 UTC by John
Modified: 2009-05-30 12:13 UTC
See Also:
GNOME target: ---
GNOME version: 2.25/2.26



Description John 2009-05-10 11:17:09 UTC
Steps to reproduce:
1. use gst_element_send_event to send gst_event_new_eos to force to end the stream
2. Sometimes get an error and sometimes doesn't do nothing

Stack trace:
I try to use gst_event_new_eos to force to end the stream, it works in all video formats i try, but in wmv i get errors or do nothing.

Other information:
Comment 1 John 2009-05-10 18:29:40 UTC
video codec : Windows Media VC-1
Comment 2 Sebastian Dröge (slomo) 2009-05-13 13:27:44 UTC
You must not send EOS events from the application to some element. This is only meant to be done from inside elements to downstream elements.
Comment 3 John 2009-05-13 19:53:48 UTC
I send the EOS event to playbin element.

With other video formats it works.
Comment 4 Tim-Philipp Müller 2009-05-14 00:18:45 UTC
Could you attach a (gzipped)  GST_DEBUG=*src:5,*demux:5  debug log?
Comment 5 John 2009-05-14 05:05:55 UTC
How can I do GST_DEBUG=*src:5,*demux:5 ?

I'm using pygst bindings (GStreamer for Python).
Comment 6 John 2009-05-14 21:50:47 UTC
I cannot attach the log file (4Mb), so i upload to :

http://www.1filesharing.com/download/16O2XJL8/log.gz
Comment 7 Tim-Philipp Müller 2009-05-30 12:13:29 UTC
I haven't tested it, but this should fix it:

 commit 7c40c992388d8eaca9a8999dd281c24ccc69c99e
 Author: Tim-Philipp Müller <tim.muller@collabora.co.uk>
 Date:   Fri May 29 20:07:14 2009 +0100

    asfdemux: fix bogus flow return handling in eos handler
    
    Don't overwrite the origin flow return by whatever flow we get
    when trying to push the remaining internally queued payloads.
    We want to do our eos logic, ie. send an EOS event or segment-done
    message in any case. Makes things EOS properly when an EOS event
    is forced upon the pipeline so that the source returns
    FLOW_UNEXPECTED to a pulling asfdemux. Should fix #582056.

I'm not entirely sure why you do what you do though - forcing an eos event on a playing pipeline is a slightly unusual thing to do. You might want to have a look at "gnonlin".