[Bug 735520] New: aacparse: skip valid ADTS/LOAS frames

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=735520
  GStreamer | gst-plugins-good | 1.4.0

           Summary: aacparse: skip valid ADTS/LOAS frames
    Classification: Platform
           Product: GStreamer
           Version: 1.4.0
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gst-plugins-good
        AssignedTo: gstreamer-bugs at lists.freedesktop.org
        ReportedBy: nicolas.huet at parrot.com
         QAContact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---


When parsing an incomplete ADTS/LOAS frame, the first byte of this frame is
skipped which make the frame invalid.

In the function gst_aac_parse_handle_frame, if gst_aac_parse_check_adts_frame
returns FALSE with needed data > 0, skipsize remains -1. If skipsize == -1,
skipsize becomes 1 to skip the first byte (still needed).

See the attached patch that fix the issue.

-- 
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.