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.
--
Anton Khirnov
_______________________________________________
ffmpeg-devel mailing list
[email protected]
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
[email protected] with subject "unsubscribe".
