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 605904 - mpegtsdemux doesn't handle e-ac-3 in PMT
mpegtsdemux doesn't handle e-ac-3 in PMT
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
unspecified
Other Linux
: Normal normal
: 0.10.18
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2010-01-02 16:43 UTC by Arnaud Patard
Modified: 2010-01-04 09:16 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
mpegtsdemux: Handle eac3 in PMT (1.87 KB, patch)
2010-01-02 16:43 UTC, Arnaud Patard 		committed Details | Review

Description Arnaud Patard 2010-01-02 16:43:52 UTC 
Created attachment 150688 [details] [review]
mpegtsdemux: Handle eac3 in PMT

When the stream type is set to private data, gst-mpegtsdemux is trying to find
audio descriptors in PMT and look for AC3 (tag 0x6a) but doesn't look for EAC3
(tag 0x7a).
This means that I can't play dvb-t streams containing h264 and eac3 (here, it's
some french dvb-t "HD" channels).

The patch I'm joining to this bug is fixing the issue for me (this patch has been produced by git format-patch as asked)

Side note : it may be possible to find AAC (tag 0x7c) on french dvb-t according
to http://www.csa.fr/upload/publication/CSATNT.pdf [ in french, sorry ] but as
I can't test it I've not added it to the patch.
Comment 1 Sebastian Dröge (slomo) 2010-01-04 09:15:54 UTC 
commit 24965dc41522578d61c29c61a7788d5d80c8bc12
Author: Arnaud Patard <arnaud.patard@rtp-net.org>
Date:   Sat Jan 2 17:33:57 2010 +0100

    mpegtsdemux: Handle eac3 in PMT
    
    When the stream type is set to private data, gst-mpegtsdemux is trying to fi
    audio descriptors in PMT and look for AC3 (tag 0x6a) but doesn't look for EA
    (tag 0x7a). Handle this case too.
    
    Fixes bug #605904.