On 02/23/2013 06:16 PM, Vinson Lee wrote:
Fixes resource leak defect reported by Coverity.

Signed-off-by: Vinson Lee<v...@freedesktop.org>
---
  src/gallium/state_trackers/vega/shaders_cache.c | 4 +++-
  1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/gallium/state_trackers/vega/shaders_cache.c 
b/src/gallium/state_trackers/vega/shaders_cache.c
index eceae54..c1082ca 100644
--- a/src/gallium/state_trackers/vega/shaders_cache.c
+++ b/src/gallium/state_trackers/vega/shaders_cache.c
@@ -225,8 +225,10 @@ combine_shaders(const struct shader_asm_info 
*shaders[SHADER_STAGES], int num_sh
     ureg_END(ureg);

     shader->tokens = ureg_finalize(ureg);
-   if(!shader->tokens)
+   if (!shader->tokens) {
+      ureg_destroy(ureg);
        return NULL;
+   }

     p = pipe->create_fs_state(pipe, shader);


Reviewed-by: Brian Paul <bri...@vmware.com>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to