[FFmpeg-cvslog] lavc/libx264: Cast cpb bit_rates to int64_t to avoid an integer overflow.

Carl Eugen Hoyos git at videolan.org
Sun Aug 11 00:55:23 EEST 2019


ffmpeg | branch: master | Carl Eugen Hoyos <ceffmpeg at gmail.com> | Sat Aug 10 23:52:59 2019 +0200| [690cab32326eb76cf923a4ca5eea5cc8be3a8600] | committer: Carl Eugen Hoyos

lavc/libx264: Cast cpb bit_rates to int64_t to avoid an integer overflow.

Fixes remaining part of ticket #8071 on next version bump.

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

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

diff --git a/libavcodec/libx264.c b/libavcodec/libx264.c
index eeffb83140..a99c1f3163 100644
--- a/libavcodec/libx264.c
+++ b/libavcodec/libx264.c
@@ -939,8 +939,8 @@ FF_ENABLE_DEPRECATION_WARNINGS
     if (!cpb_props)
         return AVERROR(ENOMEM);
     cpb_props->buffer_size = x4->params.rc.i_vbv_buffer_size * 1000;
-    cpb_props->max_bitrate = x4->params.rc.i_vbv_max_bitrate * 1000;
-    cpb_props->avg_bitrate = x4->params.rc.i_bitrate         * 1000;
+    cpb_props->max_bitrate = x4->params.rc.i_vbv_max_bitrate * 1000LL;
+    cpb_props->avg_bitrate = x4->params.rc.i_bitrate         * 1000LL;
 
     // Overestimate the reordered opaque buffer size, in case a runtime
     // reconfigure would increase the delay (which it shouldn't).



More information about the ffmpeg-cvslog mailing list