[Ffmpeg-devel] [PATCH] Bug in av_estimate_timings_from_pts

Diego Biurrun diego
Wed Mar 21 11:48:20 CET 2007


On Sun, Mar 18, 2007 at 06:53:06PM +0100, Michael Niedermayer wrote:
> 
> On Sun, Mar 18, 2007 at 02:11:13PM -0000, Wolfram Gloger wrote:
> > 
> > The recent timestamp changes to libavformat/utils.c exposed a
> > longstanding bug:
> > 
> > av_estimate_timings_from_pts() flushes the packet queue but doesn't
> > reset the streams' cur_dts values.  This can lead to a fatal "error,
> > non monotone timestamps ..." message later, because the out-of-date
> > cur_dts values are used to compute some packet's dts.
> > 
> > The appended patch fixes this by calling av_read_frame_flush(), which
> > also eliminates code duplication.
> > 
> > The additional hunk gives more detailed error messages.
> 
> looks ok

Applied.

Diego




More information about the ffmpeg-devel mailing list