diff mbox series

[FFmpeg-devel] avfilter/src_movie: fix always true expression

Message ID 20210524033514.5514-1-val.zapod.vz@gmail.com
State New
Headers show
Series [FFmpeg-devel] avfilter/src_movie: fix always true expression | expand

Checks

Context Check Description
andriy/x86_make success Make finished
andriy/x86_make_fate success Make fate finished
andriy/PPC64_make success Make finished
andriy/PPC64_make_fate success Make fate finished

Commit Message

Valerii Zapodovnikov May 24, 2021, 3:35 a.m. UTC 
Introduced in c1f9734f977f59bc0034096afbe8e43e40d93a5d.
We are in if (movie->seek_point > 0) but seek_point is timestamp.
---
 libavfilter/src_movie.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff mbox series

Patch

diff --git a/libavfilter/src_movie.c b/libavfilter/src_movie.c
index 54f6738f9a..105d1b7b54 100644
--- a/libavfilter/src_movie.c
+++ b/libavfilter/src_movie.c
@@ -252,7 +252,7 @@  static av_cold int movie_common_init(AVFilterContext *ctx)
         timestamp = movie->seek_point;
         // add the stream start time, should it exist
         if (movie->format_ctx->start_time != AV_NOPTS_VALUE) {
-            if (timestamp > 0 && movie->format_ctx->start_time > INT64_MAX - timestamp) {
+            if (movie->format_ctx->start_time > INT64_MAX - timestamp) {
                 av_log(ctx, AV_LOG_ERROR,
                        "%s: seek value overflow with start_time:%"PRId64" seek_point:%"PRId64"\n",
                        movie->file_name, movie->format_ctx->start_time, movie->seek_point);