On 20/10/2016 01:00, Mark Thompson wrote:
> The buffer map/unmap code was in an early version of this before it
> was committed, but the unmap was never removed.  While wrong, this
> was harmless (and therefore unnoticed) because the buffers can't be
> mapped at this point - all drivers just did nothing with the call.
> ---
>  libavcodec/vaapi_decode.c | 5 +----
>  1 file changed, 1 insertion(+), 4 deletions(-)
> 
> diff --git a/libavcodec/vaapi_decode.c b/libavcodec/vaapi_decode.c
> index 9e1087f..b02f2b8 100644
> --- a/libavcodec/vaapi_decode.c
> +++ b/libavcodec/vaapi_decode.c
> @@ -150,14 +150,11 @@ int ff_vaapi_decode_issue(AVCodecContext *avctx,
>  {
>      VAAPIDecodeContext *ctx = avctx->internal->hwaccel_priv_data;
>      VAStatus vas;
> -    int err, i;
> +    int err;
> 
>      av_log(avctx, AV_LOG_DEBUG, "Decode to surface %#x.\n",
>             pic->output_surface);
> 
> -    for (i = 0; i < pic->nb_param_buffers; i++)
> -        vaUnmapBuffer(ctx->hwctx->display, pic->param_buffers[i]);
> -
>      vas = vaBeginPicture(ctx->hwctx->display, ctx->va_context,
>                           pic->output_surface);
>      if (vas != VA_STATUS_SUCCESS) {
> 

Sure :)
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to