]> git.sesse.net Git - ffmpeg/commitdiff
avcodec/librav1e: Require a bitrate to be set when using 2-pass mode
authorDerek Buitenhuis <derek.buitenhuis@gmail.com>
Sat, 2 May 2020 13:10:48 +0000 (14:10 +0100)
committerDerek Buitenhuis <derek.buitenhuis@gmail.com>
Sun, 3 May 2020 16:04:07 +0000 (17:04 +0100)
Not requiring this leads to unexpected result, since Rav1e's current
two pass API has no way to fail in such a case.

Signed-off-by: Derek Buitenhuis <derek.buitenhuis@gmail.com>
libavcodec/librav1e.c

index b0ff60d8c78a5b6ecb80cfde72837f78e03b670c..6f9b4cce4c75a0b18ebd674ddb5878205e8148b8 100644 (file)
@@ -202,6 +202,12 @@ static av_cold int librav1e_encode_init(AVCodecContext *avctx)
                                    });
     }
 
+    if ((avctx->flags & AV_CODEC_FLAG_PASS1 || avctx->flags & AV_CODEC_FLAG_PASS2) && !avctx->bit_rate) {
+        av_log(avctx, AV_LOG_ERROR, "A bitrate must be set to use two pass mode.\n");
+        ret = AVERROR_INVALIDDATA;
+        goto end;
+    }
+
     if (avctx->flags & AV_CODEC_FLAG_PASS2) {
         if (!avctx->stats_in) {
             av_log(avctx, AV_LOG_ERROR, "No stats file provided for second pass.\n");