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 677443 - [tsdemux/tsparse] Handle multiple SI in one packet
[tsdemux/tsparse] Handle multiple SI in one packet
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
git master
Other Linux
: Normal normal
: 1.1.2
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2012-06-05 07:51 UTC by Edward Hervey
Modified: 2013-07-07 06:38 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Edward Hervey 2012-06-05 07:51:07 UTC
Some streams have multiple sections in one packet.

The current code assumes there is only one per packet, leading to loss of information. This seems common on Sky/BBC DVB-T streams for EIT sections.
Comment 1 Edward Hervey 2013-07-07 06:38:11 UTC
commit f58f13e874d2c744942b3e54434495cec54ad937
Author: Edward Hervey <edward@collabora.com>
Date:   Sun Jul 7 08:29:37 2013 +0200

    mpegtsdemux: Fix multiple-SI detection/parsing
    
    We had two issues with the previous code:
    1) We were badly handling PUSI-flagged packets. We were discarding the
       initial data (if pointer != 0) whereas we should have been accumulating
       it with the previous data (if there was a continuity of course).
       => First series of information loss
    2) We were not checking whether there were more sections after the end
       of one (i.e. when the following byte was not a stuff byte).
    
    This fixes those two issues.
    
    Fixes #677443
    
    https://bugzilla.gnome.org/show_bug.cgi?id=677443