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 600412 - [asfdemux] Wrong handling of downstream GstFlowReturn
[asfdemux] Wrong handling of downstream GstFlowReturn
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-ugly
git master
Other Linux
: Normal blocker
: 0.10.16
Assigned To: GStreamer Maintainers
GStreamer Maintainers
: 603451 609660 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2009-11-02 12:48 UTC by robin.goodfellow
Modified: 2010-09-09 20:00 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Debug-Logfile of trying to import WMV into PiTiVi (269.19 KB, application/octet-stream)
2009-11-02 12:48 UTC, robin.goodfellow
  Details
asfdemux: Don't error out on non-critical flow returns (1.75 KB, patch)
2010-08-27 15:59 UTC, Edward Hervey
committed Details | Review

Description robin.goodfellow 2009-11-02 12:48:00 UTC
Created attachment 146743 [details]
Debug-Logfile of trying to import WMV into PiTiVi

Trying to import WMV-media gets a timeout after some seconds. An
error-notifaction is then shown by PiTiVi.

On IRC's #pitivi, 'twi_' has already confirmed the bug after analyzing the
generated debug-log (see attachment).

The most likely cause for this bug is WMA2-audio, because it is the common
denominator between several WMVs that have been tested. 

Here's a video that causes the bug for testing-purposes:
http://www.rxnm.net/video/UAS-WMV2.wmv (Sorry for the controversial content...
I really tried to find something with LoLCats in it... but older WMVs seem to
import just fine!)
Comment 1 Stephen Griffiths 2010-02-04 03:42:36 UTC
*** Bug 603451 has been marked as a duplicate of this bug. ***
Comment 2 Rick 2010-05-27 22:19:52 UTC
I have this problem with many many files.  I know it's a crappy MS codec, but there is a ton of existing content.
Comment 3 Edward Hervey 2010-08-27 15:58:29 UTC
The issue is in asfdemux.

The problem is that it errors out if downstream returns GST_FLOW_UNEXPECTED (like if we use a image encoder in snapshot mode).

This is not standard behaviour for a demuxer, all other demuxers that receive GST_FLOW_UNEXPECTED will push an EOS on all pads and silently stop.
Comment 4 Edward Hervey 2010-08-27 15:59:13 UTC
Created attachment 168912 [details] [review]
asfdemux: Don't error out on non-critical flow returns

Only error out when downstream returns:
* NOT_SUPPORTED
* ERROR
* NOT_NEGOTIATED
* NOT_LINKED

If we got _UNEXPECTED, we push an EOS downstream (since maybe only one
of the streams had gone EOS) and then stop the task silently.

In the case of WRONG_STATE we just need to stop silently
Comment 5 Edward Hervey 2010-08-27 16:14:16 UTC
commit 776a09149ed3c08cad02c3746381bd701bf5b1e1
Author: Edward Hervey <bilboed@bilboed.com>
Date:   Fri Aug 27 17:50:59 2010 +0200

    asfdemux: Don't error out on non-critical flow returns
    
    Only error out when downstream returns:
    * NOT_SUPPORTED
    * ERROR
    * NOT_NEGOTIATED
    * NOT_LINKED
    
    If we got _UNEXPECTED, we push an EOS downstream (since maybe only one
    of the streams had gone EOS) and then stop the task silently.
    
    In the case of WRONG_STATE we just need to stop silently
    
    https://bugzilla.gnome.org/show_bug.cgi?id=600412
Comment 6 Jean-François Fortin Tam 2010-09-09 20:00:34 UTC
*** Bug 609660 has been marked as a duplicate of this bug. ***