[Libva] [PATCH] Fix incorrect if status checking in i965_proc_picture.

Lim, Siew Hoon siew.hoon.lim at intel.com
Tue Nov 17 16:47:32 PST 2015


Please ignore this fixed.  The real fixed for If statement should be return if  not equal to VA STATUS  success then only return status.

...siewhoon

> -----Original Message-----
> From: Libva [mailto:libva-bounces at lists.freedesktop.org] On Behalf Of Lim
> Siew Hoon
> Sent: Tuesday, November 17, 2015 6:06 PM
> To: libva at lists.freedesktop.org
> Subject: [Libva] [PATCH] Fix incorrect if status checking in i965_proc_picture.
> 
> Once the va status return VA_STATUS_ERROR_UNIMPLEMENTED, it should be
> return back to upper layer instead continue to execuse the following code.
> 
> https://bugs.freedesktop.org/show_bug.cgi?id=92811
> Signed-off-by: Lim Siew Hoon <siew.hoon.lim at intel.com>
> ---
>  src/i965_post_processing.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/i965_post_processing.c b/src/i965_post_processing.c index
> bf88c94..c2d8d7b 100755
> --- a/src/i965_post_processing.c
> +++ b/src/i965_post_processing.c
> @@ -5798,7 +5798,7 @@ i965_proc_picture(VADriverContextP ctx,
>      int in_width, in_height;
> 
>      status = i965_proc_picture_fast(ctx, proc_context, proc_state);
> -    if (status != VA_STATUS_ERROR_UNIMPLEMENTED)
> +    if (status == VA_STATUS_ERROR_UNIMPLEMENTED)
>          return status;
> 
>      if (pipeline_param->surface == VA_INVALID_ID ||
> --
> 2.1.0
> 
> _______________________________________________
> Libva mailing list
> Libva at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/libva


More information about the Libva mailing list