diff options
author | Thibaut Mattern <tmattern@users.sourceforge.net> | 2006-03-16 21:47:37 +0000 |
---|---|---|
committer | Thibaut Mattern <tmattern@users.sourceforge.net> | 2006-03-16 21:47:37 +0000 |
commit | 5d8722d69e96158751e64415f781b03edfa1d308 (patch) | |
tree | 279e68a2c89c154bbf75ba8d4f24fa166d9352fd | |
parent | 01714e96d0d26ee49878d3f520aeb75668407234 (diff) | |
download | xine-lib-5d8722d69e96158751e64415f781b03edfa1d308.tar.gz xine-lib-5d8722d69e96158751e64415f781b03edfa1d308.tar.bz2 |
[coverity]
CID: 41
Checker: FORWARD_NULL (help)
File: xine-lib/src/demuxers/demux_mpeg_pes.c
Function: parse_private_stream_1
Description: Variable "(this)->audio_fifo" tracked as NULL was dereferenced.
audio_fifo cannot be NULL
CVS patchset: 7930
CVS date: 2006/03/16 21:47:37
-rw-r--r-- | src/demuxers/demux_mpeg_pes.c | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/src/demuxers/demux_mpeg_pes.c b/src/demuxers/demux_mpeg_pes.c index 50fb15d5f..ea070522b 100644 --- a/src/demuxers/demux_mpeg_pes.c +++ b/src/demuxers/demux_mpeg_pes.c @@ -17,7 +17,7 @@ * along with this program; if not, write to the Free Software * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA * - * $Id: demux_mpeg_pes.c,v 1.31 2006/02/06 12:07:44 miguelfreitas Exp $ + * $Id: demux_mpeg_pes.c,v 1.32 2006/03/16 21:47:37 tmattern Exp $ * * demultiplexer for mpeg 2 PES (Packetized Elementary Streams) * reads streams of variable blocksizes @@ -1008,12 +1008,9 @@ static int32_t parse_private_stream_1(demux_mpeg_pes_t *this, uint8_t *p, buf_el if( !this->preview_mode ) check_newpts( this, this->pts, PTS_AUDIO ); - if(this->audio_fifo) { - this->audio_fifo->put (this->audio_fifo, buf); - lprintf ("A52 PACK put on fifo\n"); - } else { - buf->free_buffer(buf); - } + this->audio_fifo->put (this->audio_fifo, buf); + lprintf ("A52 PACK put on fifo\n"); + if (size == this->packet_len) { return this->packet_len + result; } |