Message ID | 20190704232835.32235-3-michael@niedermayer.cc |
---|---|
State | Accepted |
Commit | 923d5c489fd4ffd0b9dbfdc6c14f594bd134ab47 |
Headers | show |
On Fri, Jul 05, 2019 at 01:28:35AM +0200, Michael Niedermayer wrote: > Fixes: memleak > Fixes: 15528/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_STL_fuzzer-5735993371525120 > > Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg > Signed-off-by: Michael Niedermayer <michael@niedermayer.cc> > --- > libavcodec/utils.c | 1 + > 1 file changed, 1 insertion(+) will apply with several additional cases mentioned in the commit message as i found more cases that hit this same leak [...]
diff --git a/libavcodec/utils.c b/libavcodec/utils.c index 1b1feca38c..56ff70b744 100644 --- a/libavcodec/utils.c +++ b/libavcodec/utils.c @@ -1038,6 +1038,7 @@ FF_ENABLE_DEPRECATION_WARNINGS av_dict_free(&tmp); av_freep(&avctx->priv_data); + av_freep(&avctx->subtitle_header); if (avctx->internal) { av_frame_free(&avctx->internal->to_free); av_frame_free(&avctx->internal->compat_decode_frame);
Fixes: memleak Fixes: 15528/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_STL_fuzzer-5735993371525120 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer <michael@niedermayer.cc> --- libavcodec/utils.c | 1 + 1 file changed, 1 insertion(+)