[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