Commit 689e95e5 authored by Rémi Denis-Courmont's avatar Rémi Denis-Courmont

avcodec: use av_get_pix_fmt_name(), fix potential out-of-band read

Our custom codec names table lacked boundary checks and was incomplete.
(cherry picked from commit 88846171954a218ca9c401b87b0a4faffb5fc385)

Conflicts:
	modules/codec/avcodec/video.c
parent e5d66497
...@@ -63,6 +63,7 @@ ...@@ -63,6 +63,7 @@
#include "va.h" #include "va.h"
#if defined(HAVE_AVCODEC_VAAPI) || defined(HAVE_AVCODEC_DXVA2) #if defined(HAVE_AVCODEC_VAAPI) || defined(HAVE_AVCODEC_DXVA2)
# define HAVE_AVCODEC_VA # define HAVE_AVCODEC_VA
# include <libavutil/pixdesc.h>
#endif #endif
/***************************************************************************** /*****************************************************************************
...@@ -1160,18 +1161,9 @@ static enum PixelFormat ffmpeg_GetFormat( AVCodecContext *p_context, ...@@ -1160,18 +1161,9 @@ static enum PixelFormat ffmpeg_GetFormat( AVCodecContext *p_context,
/* Try too look for a supported hw acceleration */ /* Try too look for a supported hw acceleration */
for( int i = 0; pi_fmt[i] != PIX_FMT_NONE; i++ ) for( int i = 0; pi_fmt[i] != PIX_FMT_NONE; i++ )
{ {
static const char *ppsz_name[PIX_FMT_NB] = { const char *name = av_get_pix_fmt_name(pi_fmt[i]);
[PIX_FMT_VDPAU_H264] = "PIX_FMT_VDPAU_H264", msg_Dbg( p_dec, "Available decoder output format %d (%s)", pi_fmt[i],
[PIX_FMT_VAAPI_IDCT] = "PIX_FMT_VAAPI_IDCT", name ? name : "unknown" );
[PIX_FMT_VAAPI_VLD] = "PIX_FMT_VAAPI_VLD",
[PIX_FMT_VAAPI_MOCO] = "PIX_FMT_VAAPI_MOCO",
#ifdef HAVE_AVCODEC_DXVA2
[PIX_FMT_DXVA2_VLD] = "PIX_FMT_DXVA2_VLD",
#endif
[PIX_FMT_YUYV422] = "PIX_FMT_YUYV422",
[PIX_FMT_YUV420P] = "PIX_FMT_YUV420P",
};
msg_Dbg( p_dec, "Available decoder output format %d (%s)", pi_fmt[i], ppsz_name[pi_fmt[i]] ?: "Unknown" );
/* Only VLD supported */ /* Only VLD supported */
if( pi_fmt[i] == PIX_FMT_VAAPI_VLD ) if( pi_fmt[i] == PIX_FMT_VAAPI_VLD )
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment