From 96e340760824e537e2d034abf9a3b8be3e2b312c Mon Sep 17 00:00:00 2001 From: Dale Curtis Date: Tue, 21 Nov 2017 15:10:08 -0800 Subject: [PATCH] Free extradata before reallocating. Otherwise ff_alloc_extradata() just leaks any existing allocated memory. Signed-off-by: Dale Curtis Reviewed-by: Derek Buitenhuis Signed-off-by: Michael Niedermayer --- libavformat/oggparseogm.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavformat/oggparseogm.c b/libavformat/oggparseogm.c index e7a501b5a7..fad093b629 100644 --- a/libavformat/oggparseogm.c +++ b/libavformat/oggparseogm.c @@ -110,6 +110,7 @@ ogm_header(AVFormatContext *s, int idx) size -= 52; if (bytestream2_get_bytes_left(&p) < size) return AVERROR_INVALIDDATA; + av_freep(&st->codecpar->extradata); if (ff_alloc_extradata(st->codecpar, size) < 0) return AVERROR(ENOMEM); bytestream2_get_buffer(&p, st->codecpar->extradata, st->codecpar->extradata_size);