Commit 2344ab6a authored by Rémi Denis-Courmont's avatar Rémi Denis-Courmont

AVI: fix potential crash on seek (Closes: LP#803006)

If all activated streams are EOF ones, 'i_stream' was incorrectly set
to p_sys->i_track. Then AVI_StreamChunkSet() crashes.
parent f50f57ff
...@@ -1303,15 +1303,16 @@ static int Seek( demux_t *p_demux, mtime_t i_date, int i_percent ) ...@@ -1303,15 +1303,16 @@ static int Seek( demux_t *p_demux, mtime_t i_date, int i_percent )
{ {
demux_sys_t *p_sys = p_demux->p_sys; demux_sys_t *p_sys = p_demux->p_sys;
unsigned int i_stream;
msg_Dbg( p_demux, "seek requested: %"PRId64" seconds %d%%", msg_Dbg( p_demux, "seek requested: %"PRId64" seconds %d%%",
i_date / 1000000, i_percent ); i_date / 1000000, i_percent );
if( p_sys->b_seekable ) if( p_sys->b_seekable )
{ {
unsigned i_stream;
if( !p_sys->i_length ) if( !p_sys->i_length )
{ {
avi_track_t *p_stream; avi_track_t *p_stream = NULL;
int64_t i_pos; int64_t i_pos;
/* use i_percent to create a true i_date */ /* use i_percent to create a true i_date */
...@@ -1327,17 +1328,19 @@ static int Seek( demux_t *p_demux, mtime_t i_date, int i_percent ) ...@@ -1327,17 +1328,19 @@ static int Seek( demux_t *p_demux, mtime_t i_date, int i_percent )
/* try to find chunk that is at i_percent or the file */ /* try to find chunk that is at i_percent or the file */
i_pos = __MAX( i_percent * stream_Size( p_demux->s ) / 100, i_pos = __MAX( i_percent * stream_Size( p_demux->s ) / 100,
p_sys->i_movi_begin ); p_sys->i_movi_begin );
/* search first selected stream (and prefer non eof ones) */ /* search first selected stream (and prefer non-EOF ones) */
for( i_stream = 0, p_stream = NULL; for( unsigned i = 0; i < p_sys->i_track; i++ )
i_stream < p_sys->i_track; i_stream++ )
{ {
if( !p_stream || p_stream->b_eof ) avi_track_t *p_track = p_sys->track[i];
p_stream = p_sys->track[i_stream]; if( !p_track->b_activated )
continue;
if( p_stream->b_activated && !p_stream->b_eof ) p_stream = p_track;
i_stream = i;
if( !p_track->b_eof )
break; break;
} }
if( !p_stream || !p_stream->b_activated ) if( p_stream == NULL )
{ {
msg_Warn( p_demux, "cannot find any selected stream" ); msg_Warn( p_demux, "cannot find any selected stream" );
return VLC_EGENERIC; return VLC_EGENERIC;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment