[FFmpeg-cvslog] avutil/hwcontext_qsv: Fix leak of AVBuffer and AVBufferRef

Andreas Rheinhardt git at videolan.org
Thu Nov 18 21:16:57 EET 2021


ffmpeg | branch: master | Andreas Rheinhardt <andreas.rheinhardt at outlook.com> | Sat Sep 25 12:27:16 2021 +0200| [bd5ec3601f03c8fb5514673e862223defa6f7673] | committer: Andreas Rheinhardt

avutil/hwcontext_qsv: Fix leak of AVBuffer and AVBufferRef

This av_buffer_create() does nothing but leak an AVBuffer and an
AVBufferRef (except on allocation error).

Fixes Coverity issue 1491393.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at outlook.com>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=bd5ec3601f03c8fb5514673e862223defa6f7673
---

 libavutil/hwcontext_qsv.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/libavutil/hwcontext_qsv.c b/libavutil/hwcontext_qsv.c
index c18747f7eb..4ed50a31a0 100644
--- a/libavutil/hwcontext_qsv.c
+++ b/libavutil/hwcontext_qsv.c
@@ -235,8 +235,6 @@ static AVBufferRef *qsv_pool_alloc(void *opaque, size_t size)
 
     if (s->nb_surfaces_used < hwctx->nb_surfaces) {
         s->nb_surfaces_used++;
-        av_buffer_create((uint8_t*)(s->handle_pairs_internal + s->nb_surfaces_used - 1),
-                                sizeof(*s->handle_pairs_internal), qsv_pool_release_dummy, NULL, 0);
         return av_buffer_create((uint8_t*)(s->surfaces_internal + s->nb_surfaces_used - 1),
                                 sizeof(*hwctx->surfaces), qsv_pool_release_dummy, NULL, 0);
     }



More information about the ffmpeg-cvslog mailing list