Re: [FFmpeg-devel] [PATCH 1/4] avcodec/ac3_parser: use a padded buffer in av_ac3_parse_header()

[James Almer <jamrial@gmail.com>]

[-- Attachment #1.1.1: Type: text/plain, Size: 1474 bytes --]

On 6/5/2025 9:14 PM, James Almer wrote:
> The GetBitContext API requires the buffer to be padded, and the documentation for
> av_ac3_parse_header() does not specify it, so use a temporary local buffer.
> 
> Signed-off-by: James Almer <jamrial@gmail.com>
> ---
>   libavcodec/ac3_parser.c | 16 +++++++++++++---
>   1 file changed, 13 insertions(+), 3 deletions(-)
> 
> diff --git a/libavcodec/ac3_parser.c b/libavcodec/ac3_parser.c
> index 69989690dd..9065d700e2 100644
> --- a/libavcodec/ac3_parser.c
> +++ b/libavcodec/ac3_parser.c
> @@ -202,14 +202,24 @@ int av_ac3_parse_header(const uint8_t *buf, size_t size,
>   {
>       GetBitContext gb;
>       AC3HeaderInfo hdr;
> +    uint8_t *tmp = av_malloc(size + AV_INPUT_BUFFER_PADDING_SIZE);
>       int err;
>   
> -    err = init_get_bits8(&gb, buf, size);
> -    if (err < 0)
> +    if (!tmp)
> +        return AVERROR(ENOMEM);
> +
> +    memcpy(tmp, buf, size);
> +    memset(tmp + size, 0, AV_INPUT_BUFFER_PADDING_SIZE);
> +    err = init_get_bits8(&gb, tmp, size);
> +    if (err < 0) {
> +        av_free(tmp);
>           return AVERROR_INVALIDDATA;
> +    }
>       err = ff_ac3_parse_header(&gb, &hdr);
> -    if (err < 0)
> +    if (err < 0) {
> +        av_free(tmp);
>           return AVERROR_INVALIDDATA;
> +    }
>   
>       *bitstream_id = hdr.bitstream_id;
>       *frame_size   = hdr.frame_size;

Added the missing av_free(tmp) in case of success locally...


[-- Attachment #1.2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 495 bytes --]

[-- Attachment #2: Type: text/plain, Size: 251 bytes --]

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".