Free extradata before reallocating.

Otherwise ff_alloc_extradata() just leaks any existing allocated memory. Signed-off-by: Dale Curtis <[email protected]> Reviewed-by: Derek Buitenhuis <[email protected]> Signed-off-by: Michael Niedermayer <[email protected]>
SaschaVasarrhelyi · Nov 28, 2017 · 96e3407 · 96e3407
1 parent b3cb9bd
commit 96e3407
Showing 1 changed file with 1 addition and 0 deletions.
diff --git 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);