On 05/23/2013 11:54 PM, Vinson Lee wrote:
Fixes "Uninitialized pointer field" defect reported by Coverity.

Signed-off-by: Vinson Lee <v...@freedesktop.org>
---
  src/glsl/lower_named_interface_blocks.cpp | 3 ++-
  1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/glsl/lower_named_interface_blocks.cpp 
b/src/glsl/lower_named_interface_blocks.cpp
index eba667a..922cc02 100644
--- a/src/glsl/lower_named_interface_blocks.cpp
+++ b/src/glsl/lower_named_interface_blocks.cpp
@@ -72,7 +72,8 @@ public:
     hash_table *interface_namespace;

     flatten_named_interface_blocks_declarations(void *mem_ctx)
-      : mem_ctx(mem_ctx)
+      : mem_ctx(mem_ctx),
+        interface_namespace(NULL)
     {
     }

From my reading, this shouldn't lead to any actual issues, but initializing member variables is generally a good idea.

Reviewed-by: Kenneth Graunke <kenn...@whitecape.org>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to