diff mbox

[FFmpeg-devel] lavc/libx264: bit_rates > INT_MAX are not supported

Message ID CAB0OVGoNTcB0UJs2FXEeyVWuD+GAKRpNwQ_hSJe-np0PvomL7g@mail.gmail.com
State Superseded
Headers show

Commit Message

Carl Eugen Hoyos Aug. 10, 2019, 12:43 p.m. UTC 
Hi!

Attached patch fixes ticket #8071.

Please comment, Carl Eugen

Comments

Hendrik Leppkes Aug. 10, 2019, 2:53 p.m. UTC | #1 
On Sat, Aug 10, 2019 at 2:43 PM Carl Eugen Hoyos <ceffmpeg@gmail.com> wrote:
>
> Hi!
>
> Attached patch fixes ticket #8071.
>

You could allow the full range of bit_rate since its immediately
divided by 1000, couldn't you?

- Hendrik
Hendrik Leppkes Aug. 10, 2019, 3:06 p.m. UTC | #2 
On Sat, Aug 10, 2019 at 4:53 PM Hendrik Leppkes <h.leppkes@gmail.com> wrote:
>
> On Sat, Aug 10, 2019 at 2:43 PM Carl Eugen Hoyos <ceffmpeg@gmail.com> wrote:
> >
> > Hi!
> >
> > Attached patch fixes ticket #8071.
> >
>
> You could allow the full range of bit_rate since its immediately
> divided by 1000, couldn't you?
>

The full range x264 would accept I mean, that is INT_MAX * 1000.

- Hendrik
diff mbox

Patch

From 8fd6cc5727b824572ba2147dfeec6f00520c9f69 Mon Sep 17 00:00:00 2001
From: Carl Eugen Hoyos <ceffmpeg@gmail.com>
Date: Sat, 10 Aug 2019 14:40:23 +0200
Subject: [PATCH] lavc/libx264: bit_rates > INT_MAX are not supported.

Fixes ticket #8071
---
 libavcodec/libx264.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/libavcodec/libx264.c b/libavcodec/libx264.c
index dc4b4b100d..ea44cc38c6 100644
--- a/libavcodec/libx264.c
+++ b/libavcodec/libx264.c
@@ -594,6 +594,10 @@  static av_cold int X264_init(AVCodecContext *avctx)
     PARSE_X264_OPT("weightp", wpredp);
 
     if (avctx->bit_rate) {
+        if (avctx->bit_rate > INT_MAX || avctx->rc_max_rate > INT_MAX) {
+            av_log(avctx, AV_LOG_ERROR, "bit_rate and rc_max_rate > %d not supported by libx264\n", INT_MAX);
+            return AVERROR(EINVAL);
+        }
         x4->params.rc.i_bitrate   = avctx->bit_rate / 1000;
         x4->params.rc.i_rc_method = X264_RC_ABR;
     }
-- 
2.22.0