From: Vinson Lee <vlee@vmware.com>
Date: Tue, 24 Aug 2010 08:16:46 +0000 (-0700)
Subject: glsl: Add assert to check variable_entry referenced_count after construction.
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=c8b26c63fdad84d311748a24972146b6a371ab31;p=mesa.git

glsl: Add assert to check variable_entry referenced_count after construction.

A variable_entry after construction should have its referenced_count
member set to 0. However, occassionally this isn't the case and
entry->referenced_count has been observed to be a garbage value. This
leads to crashes of several tests in the Piglit test suite.

This patch adds an assert to check that a variable_entry instance has
its referenced_count member initialized to 0 after construction.
---

diff --git a/src/glsl/ir_variable_refcount.cpp b/src/glsl/ir_variable_refcount.cpp
index 20c2f6602bf..66147279bef 100644
--- a/src/glsl/ir_variable_refcount.cpp
+++ b/src/glsl/ir_variable_refcount.cpp
@@ -45,6 +45,7 @@ ir_variable_refcount_visitor::get_variable_entry(ir_variable *var)
    }
 
    variable_entry *entry = new(mem_ctx) variable_entry(var);
+   assert(entry->referenced_count == 0);
    this->variable_list.push_tail(entry);
    return entry;
 }