[FFmpeg-devel] [PATCH] movie video source
Stefano Sabatini
stefano.sabatini-lala
Tue Feb 1 00:27:35 CET 2011
On date Sunday 2011-01-30 18:19:42 +0100, Stefano Sabatini encoded:
> On date Sunday 2011-01-30 16:15:10 +0100, Michael Niedermayer encoded:
> > On Sun, Jan 30, 2011 at 12:32:25AM +0100, Stefano Sabatini wrote:
> > [...]
> > > + if (movie->seek_point_d > (INT64_MAX-0.5) / 1000000) {
> > > + av_log(ctx, AV_LOG_ERROR, "Value for seek point is too big\n");
> > > + return AVERROR(EINVAL);
> > > + }
> >
> > i think you can set the max in the AVOptions table correctly
>
> Added, I'm using (INT64_MAX-1) / 1000000 for preventing rounding
> errors.
>
> Also added one more check on the seeking value for preventing overflow
> when doing movie->seek_point + start_time.
>
> > except that lgtm if tested
Ping (note to the committer: update the patch after the one applied in
the videolan repo, with regards to the minor number).
--
FFmpeg = Foolish and Free Mean Peaceless Elastic Gangster
More information about the ffmpeg-devel
mailing list