diff mbox

[FFmpeg-devel] avcodec/movtextdec: Call mov_text_cleanup() on close

Message ID 20170304195848.23683-1-michael@niedermayer.cc
State Accepted
Commit a9f9b7f5c70e80245983e67b8ed23212d9637645
Headers show

Commit Message

Michael Niedermayer March 4, 2017, 7:58 p.m. UTC 
Fixes memleak
Fixes: 548/clusterfuzz-testcase-5511470875934720

Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/targets/ffmpeg
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
---
 libavcodec/movtextdec.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Michael Niedermayer March 6, 2017, 3:04 p.m. UTC | #1 
On Sat, Mar 04, 2017 at 08:58:48PM +0100, Michael Niedermayer wrote:
> Fixes memleak
> Fixes: 548/clusterfuzz-testcase-5511470875934720
> 
> Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/targets/ffmpeg
> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
> ---
>  libavcodec/movtextdec.c | 1 +
>  1 file changed, 1 insertion(+)

applied

[...]
diff mbox

Patch

diff --git a/libavcodec/movtextdec.c b/libavcodec/movtextdec.c
index 81fd1d6deb..6de15004fc 100644
--- a/libavcodec/movtextdec.c
+++ b/libavcodec/movtextdec.c
@@ -521,6 +521,7 @@  static int mov_text_decode_close(AVCodecContext *avctx)
 {
     MovTextContext *m = avctx->priv_data;
     mov_text_cleanup_ftab(m);
+    mov_text_cleanup(m);
     return 0;
 }