On 02/02/2013 12:41 AM, Vinson Lee wrote:
Fixes uninitialized pointer field defect reported by Coverity.

Signed-off-by: Vinson Lee<v...@freedesktop.org>
---
  src/glsl/ast.h | 12 ++++++++----
  1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/src/glsl/ast.h b/src/glsl/ast.h
index 1a28963..fcc6b45 100644
--- a/src/glsl/ast.h
+++ b/src/glsl/ast.h
@@ -547,11 +547,15 @@ public:

  class ast_parameter_declarator : public ast_node {
  public:
-   ast_parameter_declarator()
+   ast_parameter_declarator() :
+      type(NULL),
+      identifier(NULL),
+      is_array(false),
+      array_size(NULL),
+      formal_parameter(false),
+      is_void(false)
     {
-      this->identifier = NULL;
-      this->is_array = false;
-      this->array_size = 0;
+      /* empty */
     }

     virtual void print(void) const;

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