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 742938 - v4l2videodec: Could fail early on unsupported video that has codec_data
v4l2videodec: Could fail early on unsupported video that has codec_data
Status: RESOLVED WONTFIX
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
git master
Other Linux
: Normal normal
: git master
Assigned To: Nicolas Dufresne (ndufresne)
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2015-01-14 22:24 UTC by Nicolas Dufresne (ndufresne)
Modified: 2015-08-10 21:47 UTC
See Also:
GNOME target: ---
GNOME version: ---


Description Nicolas Dufresne (ndufresne) 2015-01-14 22:24:58 UTC 
Unfortunately MFC fails when reading certain video headers. It is not clear why, but failing early would allow decodebin to skip the decoder and use a SW decoder as replacement.

One of the failing media is found in gst-validate:
gst-validate/gst-integration-testsuites/medias/defaults/mxf/op2b-mpeg2-wave_hd.mxf

The solution would be to setup the output format and queue in set_format() method if a codec_data is provided instead of doing it in handle_frame(). Failing at that stage allow decodebin to skip the decoder instead of failing the pipeline.
Comment 1 Nicolas Dufresne (ndufresne) 2015-03-09 14:33:46 UTC 
In fact, a lot of the stuff done in handle_frame() should be moved to negotiate(). This way if we find codec_data in the caps, we simply call negotiate.
Comment 2 Nicolas Dufresne (ndufresne) 2015-07-29 00:36:35 UTC 
Turns out that people are getting keener to wait for the first SOURCE_CHANGE event before reading the CAPTURE format instead of making G_FMT synhronize on header parsing. That basically means (unless we wait for the event synchronously) that this idea of "detecting" unsupported stream through header parsing might not be possible.
Comment 3 Nicolas Dufresne (ndufresne) 2015-08-10 21:47:52 UTC 
I'm abandoning this one.