aboutsummaryrefslogtreecommitdiffstats
path: root/libavcodec
diff options
context:
space:
mode:
authorZhao Zhili <quinkblack@foxmail.com>2022-06-27 17:57:27 +0800
committerZhao Zhili <zhilizhao@tencent.com>2022-07-04 11:30:08 +0800
commit066368e8373d18b496203fa91d98bf11a9b7f59f (patch)
tree494494f3ffbdf30ffbe6e95e64e4552d6401baee /libavcodec
parent9fec43a92d7304fc594dcf73c6dd3144b855617d (diff)
downloadffmpeg-066368e8373d18b496203fa91d98bf11a9b7f59f.tar.gz
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@tencent.com>
Diffstat (limited to 'libavcodec')
-rw-r--r--libavcodec/libdavs2.c1
1 files changed, 1 insertions, 0 deletions
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;
}