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 705035 - mpegvideoparse: do not update caps when bitrate changes in seq. header
mpegvideoparse: do not update caps when bitrate changes in seq. header
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
git master
Other All
: Normal normal
: 1.1.3
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2013-07-28 16:27 UTC by Matej Knopp
Modified: 2013-07-29 06:42 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch to only compare part of header before the bitrate (1.23 KB, patch)
2013-07-28 16:29 UTC, Matej Knopp
committed Details | Review

Description Matej Knopp 2013-07-28 16:27:25 UTC
In sample video the bitrate field in sequence header changes, causing mpegtsdemux resend caps every time, which breaks decoding. The bitrate change is not important, and should not cause decoder reopen.

Sample video:
https://s3.amazonaws.com/MatejK/Samples/StarTrekX_Nemesis_snipp.ts
Comment 1 Matej Knopp 2013-07-28 16:29:25 UTC
Created attachment 250315 [details] [review]
Patch to only compare part of header before the bitrate
Comment 2 Sebastian Dröge (slomo) 2013-07-29 06:42:17 UTC
Author: Matej Knopp <matej.knopp@gmail.com>
Date:   Sun Jul 28 18:28:30 2013 +0200

    mpegvideoparse: do not update caps when bitrate changes
    
    https://bugzilla.gnome.org/show_bug.cgi?id=705035