diff options
author | Andoni Morales Alastruey <ylatuya@gmail.com> | 2010-09-01 22:05:43 +0200 |
---|---|---|
committer | Sebastian Dröge <sebastian.droege@collabora.co.uk> | 2010-09-04 15:01:30 +0200 |
commit | 4668330bdc810256027637cf9aac3178e769d6b9 (patch) | |
tree | 913faf258949e9bf746b914b71818a16d7f4769d /docs/plugins/inspect/plugin-alsaspdif.xml | |
parent | 8af858cf8f166a4ab81d036bdb75b44f354a821b (diff) | |
download | gstreamer-plugins-bad-4668330bdc810256027637cf9aac3178e769d6b9.tar.gz |
mpegtsmux: Initialize PES packet before getting the header size.
The PES header length is calculated before setting the dynamic flags, returning
a wrong value. Small frames that should be sent in a single TS packet are
spawned to a new packet because of that error. For audio streams where a single
frame can cope in one TS packet it introduces a huge overhead.
For a 100B packet, we prepare a TS packet with a payload of(100+9)B. Then, we
write the TS header using this value in tsmux_write_ts_header, and call
tsmux_stream_get_data(). The dynamic flags where not set yet and now
tsmux_stream_pes_header_length() returns 14B instead of 9B. The payload of the
TS packet is 114B, 5B more than what was calculated. 109B are sent in a first
packet and the remaining 5B are sent in another one.
Fixes bug #628548.
Diffstat (limited to 'docs/plugins/inspect/plugin-alsaspdif.xml')
0 files changed, 0 insertions, 0 deletions