[FFmpeg-cvslog] avcodec/setts_bsf: fix setting output timebase

[James Almer]

ffmpeg | branch: master | James Almer <jamrial at gmail.com> | Wed Mar 16 23:41:55 2022 -0300| [c834083861a5e81595a2834cb11cd329d46f6a65] | committer: James Almer

avcodec/setts_bsf: fix setting output timebase

Set a valid max value for the option, and also check numerator before
assuming the value is useful.

Signed-off-by: James Almer <jamrial at gmail.com>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=c834083861a5e81595a2834cb11cd329d46f6a65
---

 libavcodec/setts_bsf.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/libavcodec/setts_bsf.c b/libavcodec/setts_bsf.c
index 69977bc166..b7732214ee 100644
--- a/libavcodec/setts_bsf.c
+++ b/libavcodec/setts_bsf.c
@@ -145,7 +145,7 @@ static int setts_init(AVBSFContext *ctx)
         }
     }
 
-    if (s->time_base.den)
+    if (s->time_base.num > 0 && s->time_base.den > 0)
         ctx->time_base_out = s->time_base;
 
     s->frame_number= 0;
@@ -259,7 +259,7 @@ static const AVOption options[] = {
     { "pts", "set expression for packet PTS", OFFSET(pts_str), AV_OPT_TYPE_STRING, {.str=NULL}, 0, 0, FLAGS },
     { "dts", "set expression for packet DTS", OFFSET(dts_str), AV_OPT_TYPE_STRING, {.str=NULL}, 0, 0, FLAGS },
     { "duration", "set expression for packet duration", OFFSET(duration_str), AV_OPT_TYPE_STRING, {.str="DURATION"}, 0, 0, FLAGS },
-    { "time_base", "set output timebase", OFFSET(time_base), AV_OPT_TYPE_RATIONAL, {.dbl=0}, 0, 0, FLAGS },
+    { "time_base", "set output timebase", OFFSET(time_base), AV_OPT_TYPE_RATIONAL, {.dbl=0}, 0, INT_MAX, FLAGS },
     { NULL },
 };