Commit cb7c4f73 authored by Andreas Cadhalpun's avatar Andreas Cadhalpun

cafdec: free extradata before allocating it

This fixes a memleak if read_kuki_chunk is executed more than once.
Reviewed-by: 's avatarCarl Eugen Hoyos <cehoyos@ag.or.at>
Signed-off-by: 's avatarAndreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>
parent a3ede6b7
...@@ -134,6 +134,7 @@ static int read_kuki_chunk(AVFormatContext *s, int64_t size) ...@@ -134,6 +134,7 @@ static int read_kuki_chunk(AVFormatContext *s, int64_t size)
return AVERROR_INVALIDDATA; return AVERROR_INVALIDDATA;
} }
av_freep(&st->codec->extradata);
if (ff_alloc_extradata(st->codec, ALAC_HEADER)) if (ff_alloc_extradata(st->codec, ALAC_HEADER))
return AVERROR(ENOMEM); return AVERROR(ENOMEM);
...@@ -166,6 +167,7 @@ static int read_kuki_chunk(AVFormatContext *s, int64_t size) ...@@ -166,6 +167,7 @@ static int read_kuki_chunk(AVFormatContext *s, int64_t size)
avio_skip(pb, size - ALAC_NEW_KUKI); avio_skip(pb, size - ALAC_NEW_KUKI);
} }
} else { } else {
av_freep(&st->codec->extradata);
if (ff_get_extradata(st->codec, pb, size) < 0) if (ff_get_extradata(st->codec, pb, size) < 0)
return AVERROR(ENOMEM); return AVERROR(ENOMEM);
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment