diff options
| author | Marios Titas <redneb@gmx.com> | 2016-04-02 21:11:44 +0300 |
|---|---|---|
| committer | Michael Niedermayer <michael@niedermayer.cc> | 2016-04-03 01:55:24 +0200 |
| commit | 47f0d80ee01b1ae5f3ab68ef697681d9a1327b00 (patch) | |
| tree | ec48d5cb3c68c22c02563fd0ac40b37e0ddadeb7 /libavfilter/vf_showinfo.c | |
| parent | 5d79566ab3ddfc6fc46cc2fe7f3b894b4c384f3e (diff) | |
| download | ffmpeg-47f0d80ee01b1ae5f3ab68ef697681d9a1327b00.tar.gz | |
avfilter/src_movie: fix how we check for overflows with seek_point
Currently, if the movie source filter is used and a seek_point is
specified on a file that has a negative start time, ffmpeg will fail.
An easy way to reproduce this is as follows:
$ ffmpeg -vsync passthrough -filter_complex 'color=d=10,setpts=PTS-1/TB' test.mp4
$ ffmpeg -filter_complex 'movie=filename=test.mp4:seek_point=2' -f null -
The problem is caused by checking for int64_t overflow the wrong way.
In general, to check whether a + b overflows, it is not enough to do:
a > INT64_MAX - b
because b might be negative; the correct way is:
b > 0 && > a > INT64_MAX - b
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
(cherry picked from commit c1f9734f977f59bc0034096afbe8e43e40d93a5d)
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
Diffstat (limited to 'libavfilter/vf_showinfo.c')
0 files changed, 0 insertions, 0 deletions