[FFmpeg-cvslog] [ffmpeg] branch release/8.0 updated. a8fdfddedb avcodec/vp9: fix leaked cbs fragment AVBufferRef
ffmpeg-git at ffmpeg.org
ffmpeg-git at ffmpeg.org
Mon Aug 11 04:25:33 EEST 2025
The branch, release/8.0 has been updated
via a8fdfddedb1ec1167378e9306e73864f5de23c72 (commit)
from 89849ca92a86dfbc47ea4e65788588f6382c7f5b (commit)
- Log -----------------------------------------------------------------
commit a8fdfddedb1ec1167378e9306e73864f5de23c72
Author: Leo Izen <leo.izen at gmail.com>
AuthorDate: Sat Aug 9 22:41:22 2025 -0400
Commit: Leo Izen <leo.izen at gmail.com>
CommitDate: Sun Aug 10 21:05:43 2025 -0400
avcodec/vp9: fix leaked cbs fragment AVBufferRef
When this function returns, the fragment is never reset, so
current_frag->data_ref is never unref-ed, which ends up leaking it.
We call ff_cbs_fragment_reset to release the reference on its buffer.
Signed-off-by: Leo Izen <leo.izen at gmail.com>
diff --git a/libavcodec/vp9.c b/libavcodec/vp9.c
index 47fabf98e3..d0d0238c2c 100644
--- a/libavcodec/vp9.c
+++ b/libavcodec/vp9.c
@@ -1679,8 +1679,6 @@ static int vp9_decode_frame(AVCodecContext *avctx, AVFrame *frame,
&s->s.frames[CUR_FRAME] : &s->s.ref_frames[i]);
}
- ff_cbs_fragment_reset(&s->current_frag);
-
goto finish;
}
@@ -1805,6 +1803,8 @@ static int vp9_decode_frame(AVCodecContext *avctx, AVFrame *frame,
}
finish:
+ ff_cbs_fragment_reset(&s->current_frag);
+
ff_progress_frame_report(&s->s.frames[CUR_FRAME].tf, INT_MAX);
// ref frame setup
for (int i = 0; i < 8; i++)
-----------------------------------------------------------------------
Summary of changes:
libavcodec/vp9.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
hooks/post-receive
--
More information about the ffmpeg-cvslog
mailing list