[FFmpeg-devel] [PATCH] Make sure AVFormatContext->start_time is initialized after the first patch

[Michael Niedermayer]

On Fri, May 28, 2010 at 11:24:54PM +0300, Martin Storsj? wrote:
> On Tue, 25 May 2010, Martin Storsj? wrote:
> 
> > On Tue, 25 May 2010, Michael Niedermayer wrote:
> > 
> > > The problem might be that ffserver calls av_seek_frame()
> > > with that the first packet demuxed might no longer correspond to the
> > > first packet.
> > > thus i think its not unreasonable if lavf doesnt set start_time to the
> > > first dts/pts once seeking has happened
> > 
> > Hmm, good point.
> > 
> > But given this, since we cannot absolutely rely on start_time being 
> > initialized, the if statement
> > 
> >     if (ist->start_time != AV_NOPTS_VALUE)
> >         c->cur_pts -= av_rescale_q(ist->start_time, ist->time_base, AV_TIME_BASE_Q);
> > 
> > in ffserver.c must have an else branch. If not, c->cur_pts will have a 
> > value in the wrong range (absolute instead of relative), which completely 
> > messes things up. And IMO, this would be fixed by the attached patch.
> 
> Ping, Michael and Baptiste?

iam not ffserver maintainer 

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

Breaking DRM is a little like attempting to break through a door even
though the window is wide open and the only thing in the house is a bunch
of things you dont want and which you would get tomorrow for free anyway
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100528/f3ba1c38/attachment.pgp>