mirrors/gstreamer
mirrors/gstreamer
1
1
Fork 0
mirror of https://gitlab.freedesktop.org/gstreamer/gstreamer.git synced 2024-11-27 12:11:13 +00:00
Code Issues 1 Projects Releases Wiki Activity

tsdemux: fix overflow of packet_length field of PESHeader

Browse source 
packet_length is defined as a guint16 in the PESHeader structure. This
definition match the specification. But since we add 6 bytes to the
packet_length value (length of start_code + stream_id + packet_length),
we can overflow the guint16 when the value in the PES header is greater
than 65529.
So use a guint32 instead of a guint16 to avoid overflow.

https://bugzilla.gnome.org/show_bug.cgi?id=736490
This commit is contained in:
Aurélien Zanelli 2014-09-11 18:33:20 +02:00 committed by Edward Hervey
parent bce21e7398
commit 6d767a09d8
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
gst/mpegtsdemux/pesparse.h
View file

@ -143,7 +143,7 @@ typedef enum {
typedef struct {
guint8 stream_id; /* See ID_* above */
guint16 packet_length; /* The size of the PES header and PES data
guint32 packet_length; /* The size of the PES header and PES data
* (if 0 => unbounded packet) */
guint16 header_size; /* The complete size of the PES header */