[FFmpeg-devel] [PATCH 01/12] avcodec/vorbisenc: Remove always-false check

[Andreas Rheinhardt]

On Thu, Mar 25, 2021 at 4:50 PM Andreas Rheinhardt <
andreas.rheinhardt at gmail.com> wrote:

> The PutBitContext is big enough: It has just been initialized to 8192B.
>
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>
> ---
>  libavcodec/vorbisenc.c | 5 -----
>  1 file changed, 5 deletions(-)
>
> diff --git a/libavcodec/vorbisenc.c b/libavcodec/vorbisenc.c
> index 18a679f2dc..f8a08f816f 100644
> --- a/libavcodec/vorbisenc.c
> +++ b/libavcodec/vorbisenc.c
> @@ -1135,11 +1135,6 @@ static int vorbis_encode_frame(AVCodecContext
> *avctx, AVPacket *avpkt,
>
>      init_put_bits(&pb, avpkt->data, avpkt->size);
>
> -    if (pb.size_in_bits - put_bits_count(&pb) < 1 + ilog(venc->nmodes -
> 1)) {
> -        av_log(avctx, AV_LOG_ERROR, "output buffer is too small\n");
> -        return AVERROR(EINVAL);
> -    }
> -
>      put_bits(&pb, 1, 0); // magic bit
>
>      put_bits(&pb, ilog(venc->nmodes - 1), 1); // Mode for current frame
> --
> 2.27.0
>
>
Ping. WIll apply this patchset tomorrow unless there are objections.

- Andreas