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 482946 - warning in demuxer when receiving GST_EVENT_LATENCY
warning in demuxer when receiving GST_EVENT_LATENCY
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-libav
0.10.2
Other All
: Normal minor
: 0.10.3
Assigned To: Wim Taymans
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2007-10-03 13:46 UTC by Yann Gilquin
Modified: 2007-10-05 15:03 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
gst_ffmpegdemux_src_event function handling GST_EVENT_LATENCY (376 bytes, patch)
2007-10-03 13:49 UTC, Yann Gilquin
committed Details | Review

Description Yann Gilquin 2007-10-03 13:46:43 UTC
Please describe the problem:
When an event GST_EVENT_LATENCY (since 0.10.12) is received on a src pad, it's not managed and the gst_ffmpegdemux_src_event func returns FALSE.

This leads to the following warning on gstreamer side :

Internal GStreamer error: clock problem.  Please file a bug at http://bugzilla.gnome.org/enter_bug.cgi?product=GStreamer.
build/host/gst/gstpipeline.c(487): gst_pipeline_change_state (): /playeraudiovideo:
Failed to configure latency of 0:00:00.000000000 

Steps to reproduce:
1. create a pipeline with ffdemux element and set it to PLAYING state
2. 
3. 


Actual results:
You get the previous warning and latency cannot be set.

Expected results:
The event should be send upstream.

Does this happen every time?
Yes.

Other information:
Comment 1 Yann Gilquin 2007-10-03 13:49:19 UTC
Created attachment 96569 [details] [review]
gst_ffmpegdemux_src_event function handling GST_EVENT_LATENCY

Maybe the way the demux handles src event is bad, and may need a deeper rework. This is only a hack to correctly push the event upstream.
Comment 2 Wim Taymans 2007-10-05 15:03:43 UTC
        Patch by: Yann Gilquin <yann dot gilquin at purplelabs dot com>

        * ext/ffmpeg/gstffmpegdemux.c: (gst_ffmpegdemux_src_event):
        Don't return FALSE for the LATENCY event but instead forward it 
        upstream. Fixes #482946.