[FFmpeg-devel] [PATCH 1/4] avcodec/aac/aacdec: Free channel layout

[Michael Niedermayer]

On Thu, Dec 26, 2024 at 01:19:52PM +0900, Lynne via ffmpeg-devel wrote:
> On 25/12/2024 13:26, Michael Niedermayer wrote:
> > Fixes: 371445194/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_AAC_LATM_fuzzer-5981081124274176
> > Fixes: memleak
> > 
> > Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
> > Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> > ---
> >   libavcodec/aac/aacdec.c | 2 ++
> >   1 file changed, 2 insertions(+)
> > 
> > diff --git a/libavcodec/aac/aacdec.c b/libavcodec/aac/aacdec.c
> > index 717ad8aa909..e76392c88d0 100644
> > --- a/libavcodec/aac/aacdec.c
> > +++ b/libavcodec/aac/aacdec.c
> > @@ -1104,6 +1104,8 @@ static av_cold int decode_close(AVCodecContext *avctx)
> >       for (int i = 0; i < 2; i++) {
> >           OutputConfiguration *oc = &ac->oc[i];
> > +        av_channel_layout_uninit(&ac->oc[i].ch_layout);
> > +
> >           AACUSACConfig *usac = &oc->usac;
> >           for (int j = 0; j < usac->nb_elems; j++) {
> >               AACUsacElemConfig *ec = &usac->elems[j];
> 
> LGTM

will apply

thx

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I have never wished to cater to the crowd; for what I know they do not
approve, and what they approve I do not know. -- Epicurus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20241231/d191aea6/attachment.sig>