[FFmpeg-devel] [PATCH] libavcodec/mpegaudio_parser.c: differentiate MPEG audio dual mono

Anton Khirnov anton at khirnov.net
Thu Dec 12 08:41:49 EET 2024


Quoting Scott Theisen (2024-12-10 21:42:06)
> On 12/9/24 02:31, Anton Khirnov wrote:
> > Quoting Scott Theisen (2024-11-30 08:38:54)
> >> On 11/25/24 00:42, Anton Khirnov wrote:
> >>> Quoting Scott Theisen (2024-11-14 05:37:49)
> >>>> @@ -85,7 +85,13 @@ static int mpegaudio_parse(AVCodecParserContext *s1,
> >>>>                        if (s->header_count > header_threshold) {
> >>>>                            avctx->sample_rate= sr;
> >>>>                            av_channel_layout_uninit(&avctx->ch_layout);
> >>>> -                        av_channel_layout_default(&avctx->ch_layout, channels);
> >>>> +                        if (dual_mono) {
> >>>> +                            av_channel_layout_custom_init(&avctx->ch_layout, 2);
> >>> This can fail - the return code should be checked.
> >>>
> >> It can only fail if av_calloc() fails to allocate 48 bytes.  Should it
> >> return buf_size or just use the default order?
> > Memory allocation failure should always propagate back to the caller.
> >
> 
> The parsers can't return error codes, though.  See line 105 in 
> libavcodec/mpegaudio_parser.c

Huh, disregard my comment then.

And we really should do something about parsers.

-- 
Anton Khirnov


More information about the ffmpeg-devel mailing list