glsl: Initialize coordinate to NULL in ir_texture constructor.
authorKenneth Graunke <kenneth@whitecape.org>
Mon, 9 Jul 2012 19:36:49 +0000 (12:36 -0700)
committerKenneth Graunke <kenneth@whitecape.org>
Thu, 12 Jul 2012 17:19:38 +0000 (10:19 -0700)
I ran into this while trying to create a TXS query, which doesn't have a
coordinate.  Since it didn't get initialized to NULL, a bunch of
visitors tried to access it and crashed.

Most of the time, this won't be a problem, but it's just a good idea.

Signed-off-by: Kenneth Graunke <kenneth@whitecape.org>
Reviewed-by: Eric Anholt <eric@anholt.net>
src/glsl/ir.h

index b54e2f2e0c1ea854cc9bc9d3326e0185069cd1d6..9bbf3b7efae31202ed14c84b0cab7e4478833aa7 100644 (file)
@@ -1341,7 +1341,8 @@ enum ir_texture_opcode {
 class ir_texture : public ir_rvalue {
 public:
    ir_texture(enum ir_texture_opcode op)
-      : op(op), projector(NULL), shadow_comparitor(NULL), offset(NULL)
+      : op(op), coordinate(NULL), projector(NULL), shadow_comparitor(NULL),
+        offset(NULL)
    {
       this->ir_type = ir_type_texture;
    }