Module: Mesa Branch: master Commit: 35f302d97e5b6f5fc4964e70f7adae706c5b7162 URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=35f302d97e5b6f5fc4964e70f7adae706c5b7162
Author: Vinson Lee <[email protected]> Date: Wed May 23 22:36:47 2012 -0700 i915g: Check for geometry shader earlier in i915_set_constant_buffer. Fix resource leak defect reported by Coverity. Signed-off-by: Vinson Lee <[email protected]> Reviewed-by: Brian Paul <[email protected]> --- src/gallium/drivers/i915/i915_state.c | 8 ++++---- 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/gallium/drivers/i915/i915_state.c b/src/gallium/drivers/i915/i915_state.c index bd9e8ba..1ff5388 100644 --- a/src/gallium/drivers/i915/i915_state.c +++ b/src/gallium/drivers/i915/i915_state.c @@ -672,16 +672,16 @@ static void i915_set_constant_buffer(struct pipe_context *pipe, unsigned new_num = 0; boolean diff = TRUE; + /* XXX don't support geom shaders now */ + if (shader == PIPE_SHADER_GEOMETRY) + return; + if (cb && cb->user_buffer) { buf = i915_user_buffer_create(pipe->screen, cb->user_buffer, cb->buffer_size, PIPE_BIND_CONSTANT_BUFFER); } - /* XXX don't support geom shaders now */ - if (shader == PIPE_SHADER_GEOMETRY) - return; - /* if we have a new buffer compare it with the old one */ if (buf) { struct i915_buffer *ibuf = i915_buffer(buf); _______________________________________________ mesa-commit mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/mesa-commit
