va/picture : Added failure check for stability
authorSureshGuttula <sguttula@amd.com>
Sun, 24 Jan 2021 05:32:55 +0000 (11:02 +0530)
committerMarge Bot <eric+marge@anholt.net>
Tue, 2 Feb 2021 13:38:22 +0000 (13:38 +0000)
This patch created to exit from for loop incase handle function
returns error vaStatus. This will help to capture the correct
failure return to application.

Signed-off-by: SureshGuttula <sguttula@amd.com>
Reviewed-by: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer@amd.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/8679>

src/gallium/frontends/va/picture.c

index bcfb79e..39ac664 100644 (file)
@@ -561,7 +561,7 @@ vlVaRenderPicture(VADriverContextP ctx, VAContextID context_id, VABufferID *buff
          handleVAProtectedSliceDataBufferType(context, buf);
    }
 
-   for (i = 0; i < num_buffers; ++i) {
+   for (i = 0; i < num_buffers && vaStatus == VA_STATUS_SUCCESS; ++i) {
       vlVaBuffer *buf = handle_table_get(drv->htab, buffers[i]);
 
       switch (buf->type) {