Am 22.08.2016 um 22:59 schrieb Eric Engestrom:
On Mon, Aug 22, 2016 at 09:44:59AM +0200, Christian König wrote:
Am 21.08.2016 um 23:11 schrieb Eric Engestrom:
Fixes: c59628d11b134fc01638 ("st/va: enable dual instances encode by sync 
surface")
Signed-off-by: Eric Engestrom <e...@engestrom.ch>
Reviewed-by: Christian König <christian.koe...@amd.com>

Do you have commit access?
I do not. Feel free to push this and any of my other patches :)

Note to self: I need to go through my outstanding patches and
ping people to review/push them...

Thought so. Just pushed this one.

Feel free to ping me or others if you still have open patches not comitted yet.

Thanks for the help,
Christian.


Cheers,
   Eric

Regards,
Christian.

---
   src/gallium/state_trackers/va/surface.c | 4 +++-
   1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/gallium/state_trackers/va/surface.c 
b/src/gallium/state_trackers/va/surface.c
index 012e48e..3ee1cdd 100644
--- a/src/gallium/state_trackers/va/surface.c
+++ b/src/gallium/state_trackers/va/surface.c
@@ -106,8 +106,10 @@ vlVaSyncSurface(VADriverContextP ctx, VASurfaceID 
render_target)
      pipe_mutex_lock(drv->mutex);
      surf = handle_table_get(drv->htab, render_target);
-   if (!surf || !surf->buffer)
+   if (!surf || !surf->buffer) {
+      pipe_mutex_unlock(drv->mutex);
         return VA_STATUS_ERROR_INVALID_SURFACE;
+   }
      context = handle_table_get(drv->htab, surf->ctx);
      if (!context) {


_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to