[Bug 695412] New: mpegtsmux AAC ADTS header seems incorrect

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=695412
  GStreamer | gst-plugins-bad | git

           Summary: mpegtsmux AAC ADTS header seems incorrect
    Classification: Platform
           Product: GStreamer
           Version: git
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gst-plugins-bad
        AssignedTo: gstreamer-bugs at lists.freedesktop.org
        ReportedBy: gimmeamilkbttf at gmail.com
         QAContact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---


Created an attachment (id=238346)
 View: https://bugzilla.gnome.org/attachment.cgi?id=238346
 Review: https://bugzilla.gnome.org/review?bug=695412&attachment=238346

Fix ADTS header

I don't think mpegtsmux_prepare_aac() extracts the info from the codec_data
correctly. I get a bad ADTS header when muxing a 16kHz AAC-LC stream.

Please see attached patch - I've changed it so the codec_data parsing matches
that in gst_qtdemux_handle_esds(). This makes my 16kHz LC stream play ok, and
doesn't appear to break other streams. The generated header bytes also match
those created by ffmpeg doing a similar muxing.

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.