[FFmpeg-devel] [PATCH] avutil/opt: use AVBPrint to print a channel layout in av_opt_get()

[Andreas Rheinhardt]

James Almer:
> A 128 byte buffer may not be enough for some layouts with lots of channels.
> 
> Signed-off-by: James Almer <jamrial at gmail.com>
> ---
>  libavutil/opt.c | 12 +++++++++---
>  1 file changed, 9 insertions(+), 3 deletions(-)
> 
> diff --git a/libavutil/opt.c b/libavutil/opt.c
> index 0908751752..814b485a15 100644
> --- a/libavutil/opt.c
> +++ b/libavutil/opt.c
> @@ -936,9 +936,15 @@ FF_DISABLE_DEPRECATION_WARNINGS
>          break;
>  FF_ENABLE_DEPRECATION_WARNINGS
>  #endif
> -    case AV_OPT_TYPE_CHLAYOUT:
> -        ret = av_channel_layout_describe(dst, buf, sizeof(buf));
> -        break;
> +    case AV_OPT_TYPE_CHLAYOUT: {
> +        AVBPrint bp;
> +        av_bprint_init(&bp, 0, AV_BPRINT_SIZE_UNLIMITED);
> +        av_channel_layout_describe_bprint(dst, &bp);
> +        if (!av_bprint_is_complete(&bp))
> +            return AVERROR(ENOMEM);

Potential memleak here.

> +        av_bprint_finalize(&bp, (char **)out_val);
> +        return *out_val ? 0 : AVERROR(ENOMEM);
> +    }
>      case AV_OPT_TYPE_DICT:
>          if (!*(AVDictionary **)dst && (search_flags & AV_OPT_ALLOW_NULL)) {
>              *out_val = NULL;