[FFmpeg-devel] [PATCH] avutil/hwcontext_qsv: Fix mixed declaration and code

Xiang, Haihao haihao.xiang at intel.com
Mon Mar 25 09:14:05 EET 2024


On So, 2024-03-24 at 21:30 +0100, Andreas Rheinhardt wrote:
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at outlook.com>
> ---
>  libavutil/hwcontext_qsv.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/libavutil/hwcontext_qsv.c b/libavutil/hwcontext_qsv.c
> index e5e043d2d1..c7c7878644 100644
> --- a/libavutil/hwcontext_qsv.c
> +++ b/libavutil/hwcontext_qsv.c
> @@ -1121,7 +1121,8 @@ fail:
>  static int qsv_init_internal_session(AVHWFramesContext *ctx,
>                                       mfxSession *session, int upload)
>  {
> -    AVQSVFramesContext *frames_hwctx = ctx->hwctx;
> +    QSVFramesContext              *s = ctx->hwctx;
> +    AVQSVFramesContext *frames_hwctx = &s->p;
>      QSVDeviceContext   *device_priv  = ctx->device_ctx->hwctx;
>      AVQSVDeviceContext *hwctx        = &device_priv->p;
>      int opaque = 0;
> @@ -1153,7 +1154,6 @@ static int qsv_init_internal_session(AVHWFramesContext
> *ctx,
>      }
>  
>  #if QSV_HAVE_OPAQUE
> -    QSVFramesContext              *s = ctx->hwctx;
>      opaque = !!(frames_hwctx->frame_type & MFX_MEMTYPE_OPAQUE_FRAME);
>  #endif
>  

LGTM, 

- Haihao




More information about the ffmpeg-devel mailing list