[FFmpeg-cvslog] avcodec/libdavs2: workaround memory leak
Zhao Zhili
git at videolan.org
Mon Jul 4 06:56:23 EEST 2022
ffmpeg | branch: master | Zhao Zhili <quinkblack at foxmail.com> | Mon Jun 27 17:57:27 2022 +0800| [066368e8373d18b496203fa91d98bf11a9b7f59f] | committer: Zhao Zhili
avcodec/libdavs2: workaround memory leak
davs2_decoder_close doesn't free those on the fly frames which
don't get output yet. It's a design bug, but easy to workaround.
Before the patch:
Direct leak of 1198606 byte(s) in 2 object(s) allocated from:
#0 0x563af5e1e5f0 in malloc (ffmpeg+0x6675f0)
#1 0x563af9765ef3 in davs2_malloc davs2/source/common/common.h:1240
#2 0x563af9765ef3 in davs2_alloc_picture davs2/source/common/header.cc:815
Indirect leak of 3595818 byte(s) in 6 object(s) allocated from:
#0 0x563af5e1e5f0 in malloc (ffmpeg+0x6675f0)
#1 0x563af9765ef3 in davs2_malloc davs2/source/common/common.h:1240
#2 0x563af9765ef3 in davs2_alloc_picture davs2/source/common/header.cc:815
Signed-off-by: Zhao Zhili <zhilizhao at tencent.com>
> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=066368e8373d18b496203fa91d98bf11a9b7f59f
---
libavcodec/libdavs2.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/libavcodec/libdavs2.c b/libavcodec/libdavs2.c
index d7625718a2..c2040775ae 100644
--- a/libavcodec/libdavs2.c
+++ b/libavcodec/libdavs2.c
@@ -177,6 +177,7 @@ static av_cold int davs2_end(AVCodecContext *avctx)
/* close the decoder */
if (cad->decoder) {
+ davs2_flush(avctx);
davs2_decoder_close(cad->decoder);
cad->decoder = NULL;
}
More information about the ffmpeg-cvslog
mailing list