[FFmpeg-cvslog] r25380 - trunk/libavformat/utils.c

[Aurelien Jacobs]

On Thu, Oct 07, 2010 at 12:40:43PM +0200, Michael Niedermayer wrote:
> On Thu, Oct 07, 2010 at 11:24:41AM +0200, Aurelien Jacobs wrote:
> > On Wed, Oct 06, 2010 at 11:50:16PM +0200, Michael Niedermayer wrote:
> > > On Wed, Oct 06, 2010 at 11:46:28PM +0200, Michael Niedermayer wrote:
> > > > On Wed, Oct 06, 2010 at 10:52:26PM +0200, aurel wrote:
> > > > > Author: aurel
> > > > > Date: Wed Oct  6 22:52:26 2010
> > > > > New Revision: 25380
> > > > > 
> > > > > Log:
> > > > > dynamically use nb_streams instead of static use of MAX_STREAMS
> > > > 
> > > > revert this, its exloitable
> > > 
> > >             ret = av_read_packet(ic, pkt);
> > >         }while(ret == AVERROR(EAGAIN));
> > >         if (ret != 0)
> > >             break;
> > >         read_size += pkt->size;
> > >         st = ic->streams[pkt->stream_index];
> > >         if (pkt->pts != AV_NOPTS_VALUE &&
> > >             start_time[pkt->stream_index] != AV_NOPTS_VALUE) {
> > >                 ^^^^^^^^^^^^^^^
> > > only crash it seems but still
> > > 
> > > start_time should not be an array but a value in AVStream
> > 
> > I guess you mean something like this ?
> 
> no, this sets start_time wrongly if its not set.
> either the code has to use first_dts if start_time is unavailable or
> a new field has to be added to AVstream

OK. So the attached patch should be better.

Aurel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: start_time2.diff
Type: text/x-diff
Size: 2075 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20101007/adef1ffc/attachment.diff>