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

Michael Niedermayer michaelni
Thu Oct 7 12:40:43 CEST 2010


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

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

No great genius has ever existed without some touch of madness. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20101007/f1e95663/attachment.pgp>



More information about the ffmpeg-cvslog mailing list