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:[email protected]] On Behalf Of Lim > Siew Hoon > Sent: Tuesday, November 17, 2015 6:06 PM > To: [email protected] > 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 <[email protected]> > --- > 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 > [email protected] > http://lists.freedesktop.org/mailman/listinfo/libva _______________________________________________ Libva mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/libva
