[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