The branch, release/8.0 has been updated
via a8fdfddedb1ec1167378e9306e73864f5de23c72 (commit)
from 89849ca92a86dfbc47ea4e65788588f6382c7f5b (commit)
- Log -----------------------------------------------------------------
commit a8fdfddedb1ec1167378e9306e73864f5de23c72
Author: Leo Izen <[email protected]>
AuthorDate: Sat Aug 9 22:41:22 2025 -0400
Commit: Leo Izen <[email protected]>
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 <[email protected]>
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
--
_______________________________________________
ffmpeg-cvslog mailing list
[email protected]
https://ffmpeg.org/mailman/listinfo/ffmpeg-cvslog
To unsubscribe, visit link above, or email
[email protected] with subject "unsubscribe".