mesa: Branch 'master'

Brian Paul brianp at kemper.freedesktop.org
Tue Apr 17 16:19:51 UTC 2007


 src/mesa/tnl/t_draw.c |    5 ++++-
 1 files changed, 4 insertions(+), 1 deletion(-)

New commits:
diff-tree d2d86a3f0b38716196ea2b3ffa4cbbd0420de1b3 (from 9cbcf7c696372c680c87cf60c2e290e78e9e14c8)
Author: Brian <brian at yutani.localnet.net>
Date:   Tue Apr 17 10:19:47 2007 -0600

    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 c97cf5f..5b2b2ae 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;
+   }
 }
 
 



More information about the mesa-commit mailing list