From 68b840956ef31bc21cb72f474f5b175d673fdcc9 Mon Sep 17 00:00:00 2001 From: Francois Cartegnie Date: Tue, 10 Mar 2015 14:18:32 +0100 Subject: [PATCH 1/1] demux: ts: remove PES full header check Some full headers (including filling) are more than 34 bytes. Now fail late parsing it, and relegate header>packet size check to caller. --- modules/demux/ts.c | 5 ----- 1 file changed, 5 deletions(-) diff --git a/modules/demux/ts.c b/modules/demux/ts.c index adacd46988..8a49cdefdd 100644 --- a/modules/demux/ts.c +++ b/modules/demux/ts.c @@ -2136,8 +2136,6 @@ static int ParsePESHeader( demux_t *p_demux, const uint8_t *p_header, size_t i_h { /* mpeg2 PES */ i_skip = p_header[8] + 9; - if( i_header < i_skip ) - return VLC_EGENERIC; if( p_header[7]&0x80 ) /* has pts */ { @@ -2203,9 +2201,6 @@ static int ParsePESHeader( demux_t *p_demux, const uint8_t *p_header, size_t i_h break; } - if( i_header < i_skip ) - return VLC_EGENERIC; - *pi_skip = i_skip; return VLC_SUCCESS; } -- 2.39.2