[FFmpeg-cvslog] lavc: make avcodec_open2() fail when the timebase is not set for encoding
Anton Khirnov
git at videolan.org
Wed Jun 22 19:31:35 CEST 2016
ffmpeg | branch: master | Anton Khirnov <anton at khirnov.net> | Mon May 2 18:38:41 2016 +0200| [e62ff72fc1052273deb708ba715f73e5187281d4] | committer: Anton Khirnov
lavc: make avcodec_open2() fail when the timebase is not set for encoding
Many encoders use it. There is also a divide by the timebase lower in
this function, which would crash when it is not set.
> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=e62ff72fc1052273deb708ba715f73e5187281d4
---
libavcodec/utils.c | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/libavcodec/utils.c b/libavcodec/utils.c
index d42885d..a0352b8 100644
--- a/libavcodec/utils.c
+++ b/libavcodec/utils.c
@@ -994,6 +994,13 @@ FF_DISABLE_DEPRECATION_WARNINGS
}
FF_ENABLE_DEPRECATION_WARNINGS
#endif
+
+ if (avctx->time_base.num <= 0 || avctx->time_base.den <= 0) {
+ av_log(avctx, AV_LOG_ERROR, "The encoder timebase is not set.\n");
+ ret = AVERROR(EINVAL);
+ goto free_and_end;
+ }
+
if (avctx->codec->sample_fmts) {
for (i = 0; avctx->codec->sample_fmts[i] != AV_SAMPLE_FMT_NONE; i++) {
if (avctx->sample_fmt == avctx->codec->sample_fmts[i])
More information about the ffmpeg-cvslog
mailing list