[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