mirrors/gstreamer
mirrors/gstreamer
1
1
Fork 0
mirror of https://gitlab.freedesktop.org/gstreamer/gstreamer.git synced 2025-02-17 03:35:21 +00:00
Code Issues 1 Projects Releases Wiki Activity

rtpmp2tdepay: Only output integral mpeg-ts packets

Browse source 
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

Conflicts:

	gst/rtp/gstrtpmp2tdepay.c
This commit is contained in:
Edward Hervey 2012-05-26 11:57:16 +02:00
parent 5b3d3b0885
commit 923be8a85b
1 changed files with 21 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

25
gst/rtp/gstrtpmp2tdepay.c
View file

@ -150,7 +150,7 @@ gst_rtp_mp2t_depay_process (GstRTPBaseDepayload * depayload, GstBuffer * buf)
{
GstRtpMP2TDepay *rtpmp2tdepay;
GstBuffer *outbuf;
gint payload_len;
gint payload_len, leftover;
GstRTPBuffer rtp = { NULL };
rtpmp2tdepay = GST_RTP_MP2T_DEPAY (depayload);
@ -161,11 +161,28 @@ gst_rtp_mp2t_depay_process (GstRTPBaseDepayload * depayload, GstBuffer * buf)
if (G_UNLIKELY (payload_len <= rtpmp2tdepay->skip_first_bytes))
goto empty_packet;
outbuf = gst_rtp_buffer_get_payload_subbuffer (&rtp,
rtpmp2tdepay->skip_first_bytes, -1);
payload_len -= rtpmp2tdepay->skip_first_bytes;
/* 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.
*/
leftover = payload_len % 188;
if (G_UNLIKELY (leftover)) {
GST_WARNING ("We don't have an integral number of buffers (leftover: %d)",
leftover);
payload_len -= leftover;
}
outbuf =
gst_rtp_buffer_get_payload_subbuffer (&rtp,
rtpmp2tdepay->skip_first_bytes, payload_len);
gst_rtp_buffer_unmap (&rtp);
if (outbuf)
GST_DEBUG ("gst_rtp_mp2t_depay_chain: pushing buffer of size %"
G_GSIZE_FORMAT, gst_buffer_get_size (outbuf));