From: Brian <brian@yutani.localnet.net>
Date: Tue, 17 Apr 2007 16:19:47 +0000 (-0600)
Subject: In bind_inputs() set VB->EdgeFlag to NULL if it's not needed.
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=d2d86a3f0b38716196ea2b3ffa4cbbd0420de1b3;p=mesa.git

In bind_inputs() set VB->EdgeFlag to NULL if it's not needed.

Otherwise, the clip/interp code was finding VB->EdgeFlag to be non-null and
reading/writing it when the memory may have been freed earlier in free_space().
This fixes several VTK segfaults/failures reported by Brad King @ Kitware.
---

diff --git a/src/mesa/tnl/t_draw.c b/src/mesa/tnl/t_draw.c
index c97cf5f7b21..5b2b2ae5495 100644
--- a/src/mesa/tnl/t_draw.c
+++ b/src/mesa/tnl/t_draw.c
@@ -251,7 +251,10 @@ static void bind_inputs( GLcontext *ctx,
 					   VB->AttribPtr[_TNL_ATTRIB_EDGEFLAG],
 					   VB->Count );
    }
-
+   else {
+      /* the data previously pointed to by EdgeFlag may have been freed */
+      VB->EdgeFlag = NULL;
+   }
 }