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 585845 - 0.10.7.2 breaks swfdec
0.10.7.2 breaks swfdec
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-libav
git master
Other Linux
: Normal blocker
: 0.10.8
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2009-06-15 12:10 UTC by Sven Arvidsson
Modified: 2009-06-25 15:27 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
proposed patch to fix gst-ffmpeg timestamp tracking (3.72 KB, patch)
2009-06-18 00:15 UTC, Jan Schmidt
committed Details | Review

Description Sven Arvidsson 2009-06-15 12:10:47 UTC
gst-ffmpeg 0.10.7.2 breaks swfdec when it comes to playing back videos on YouTube. As far as I can tell, it happens with all videos, and possibly on other sites than YouTube as well.

swfdec prints this error message, before hanging the browser:

SWFDEC: ERROR: swfdec_video_decoder_gst.c(156):
swfdec_video_decoder_gst_decode: failed to pull decoded buffer. Broken stream?

Downgrading to 0.10.7 solves this problem.
Comment 1 Tim-Philipp Müller 2009-06-15 12:58:38 UTC
Marking as blocker for now.
Comment 2 Jan Schmidt 2009-06-17 06:54:39 UTC
It broke with this commit:

commit a26b94d92c2e9e9f6dac0cdb5fb44ba06ee089c8
Author: Vanista <vanista@gmail.com>
Date:   Wed May 13 12:37:42 2009 +0200

    gstffmpegdec: Handle out-of-order frames. Fixes #580796

Not sure why yet.
Comment 3 Jan Schmidt 2009-06-18 00:15:51 UTC
Created attachment 136880 [details] [review]
proposed patch to fix gst-ffmpeg timestamp tracking

This patch fixes an infinite loop in the gst_ts_handler_consume() function when no buffer sizes have been stored into the timestamp arrays - which happened when all incoming timestamps (so far) are -1
Comment 4 Jan Schmidt 2009-06-18 14:15:59 UTC
Vanista, can you confirm things still work for you with this patch?
Comment 5 Jan Schmidt 2009-06-24 14:45:24 UTC
In the absence of commentary, I pushed the patch to git. Edward said he'll do some testing today, then I'll make a new gst-ffmpeg pre-release so the release is not held up much longer.
Comment 6 Sebastian Dröge (slomo) 2009-06-24 14:57:50 UTC
I'm using this patch since some time now and the Debian/experimental package also includes it and it fixes the swfdec breakage and doesn't seem to break anything new... but I didn't explicitly test it on out-of-order frames.
Comment 7 vanista 2009-06-25 15:22:26 UTC
Hi, sorry for the delay, I've been pretty busy lately. I just tested the latest fix and everything look good with my samples!

Also, simply looking at the fix now, I know it's valid, my original condition was definitely faulty.

Thanks for supporting my solution.
Comment 8 Jan Schmidt 2009-06-25 15:27:56 UTC
Thanks for confirming