[FFmpeg-cvslog] avfilter/vf_vpp_qsv: Don't overrun stack array

Andreas Rheinhardt git at videolan.org
Wed Aug 4 13:58:48 EEST 2021


ffmpeg | branch: master | Andreas Rheinhardt <andreas.rheinhardt at outlook.com> | Wed Aug  4 02:24:47 2021 +0200| [cc6f6a1eefcfbfcae30e558a7d77f75284ff2c2f] | committer: Andreas Rheinhardt

avfilter/vf_vpp_qsv: Don't overrun stack array

8b83dad82512a6948b63408f964463b063ad24c9 added another potentially used
video enhancement filter without increasing a define for the number of
such options which is used as the size of stack array. This can lead to
a buffer overrun if all filters are used simultaneously. So increase
said number.

Fixes Coverity ticket #1489775.

Reviewed-by: Linjie Fu <linjie.justin.fu at gmail.com>
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at outlook.com>

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

 libavfilter/vf_vpp_qsv.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavfilter/vf_vpp_qsv.c b/libavfilter/vf_vpp_qsv.c
index f2309ed9d9..70bd5310c3 100644
--- a/libavfilter/vf_vpp_qsv.c
+++ b/libavfilter/vf_vpp_qsv.c
@@ -42,7 +42,7 @@
 #define FLAGS (AV_OPT_FLAG_VIDEO_PARAM | AV_OPT_FLAG_FILTERING_PARAM)
 
 /* number of video enhancement filters */
-#define ENH_FILTERS_COUNT (7)
+#define ENH_FILTERS_COUNT (8)
 #define QSV_HAVE_ROTATION       QSV_VERSION_ATLEAST(1, 17)
 #define QSV_HAVE_MIRRORING      QSV_VERSION_ATLEAST(1, 19)
 #define QSV_HAVE_SCALING_CONFIG QSV_VERSION_ATLEAST(1, 19)



More information about the ffmpeg-cvslog mailing list