[Bug 676799] [gstrtpmp2tdepay] Don't push out stray data

[GStreamer (bugzilla.gnome.org)]

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

Edward Hervey <bilboed> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|HEAD                        |0.10.31

--- Comment #2 from Edward Hervey <bilboed at gmail.com> 2012-05-26 10:00:45 UTC ---
commit 9f485d884cf2ecc1f336ecafacc52b332e072ffe
Author: Edward Hervey <edward.hervey at collabora.co.uk>
Date:   Sat May 26 11:57:16 2012 +0200

    rtpmp2tdepay: Only output integral mpeg-ts packets

    From RFC 2250

    2. Encapsulation of MPEG System and Transport Streams
    ...
       For MPEG2 Transport Streams the RTP payload will contain an integral
       number of MPEG transport packets.  To avoid end system
       inefficiencies, data from multiple small MTS packets (normally fixed
       in size at 188 bytes) are aggregated into a single RTP packet.  The
       number of transport packets contained is computed by dividing RTP
       payload length by the length of an MTS packet (188).
    ....

    Since it needs to contain "an integral number of MPEG transport packets", a
    simple fix is to check that's the case, and strip off any leftover data.

    Fixes #676799

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