diff options
| author | Vinson Lee <[email protected]> | 2020-05-24 15:40:39 -0700 |
|---|---|---|
| committer | Marge Bot <[email protected]> | 2020-06-27 00:21:27 +0000 |
| commit | c0c03f4772fef361137bba5681b71215aca107d1 (patch) | |
| tree | c03f653911fb53864c714835d0cde7666cc0fc00 /src/gallium/auxiliary | |
| parent | 583d7d3d8d0d14d636340b0f12f72db7b0148ceb (diff) | |
rbug: Fix rbug_delete_vs_state lock acquisition.
Fix warning reported by Coverity Scan.
Double unlock (LOCK)
double_unlock: mtx_unlock unlocks rb_pipe->call_mutex while it is
unlocked.
Fixes: 07838ff990a7 ("rbug: Use the call mutex")
Closes: https://gitlab.freedesktop.org/mesa/mesa/-/issues/3023
Signed-off-by: Vinson Lee <[email protected]>
Reviewed-by: Jakob Bornecrantz <[email protected]>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/5196>
Diffstat (limited to 'src/gallium/auxiliary')
| -rw-r--r-- | src/gallium/auxiliary/driver_rbug/rbug_context.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/gallium/auxiliary/driver_rbug/rbug_context.c b/src/gallium/auxiliary/driver_rbug/rbug_context.c index a36f3377e8c..c391adff244 100644 --- a/src/gallium/auxiliary/driver_rbug/rbug_context.c +++ b/src/gallium/auxiliary/driver_rbug/rbug_context.c @@ -490,7 +490,7 @@ rbug_delete_vs_state(struct pipe_context *_pipe, struct rbug_context *rb_pipe = rbug_context(_pipe); struct rbug_shader *rb_shader = rbug_shader(_vs); - mtx_unlock(&rb_pipe->call_mutex); + mtx_lock(&rb_pipe->call_mutex); rbug_shader_destroy(rb_pipe, rb_shader); mtx_unlock(&rb_pipe->call_mutex); } |