[PATCH] for mesa: Fill the padding between entrys in the sampler key

[Pierre Willenbrock <pierre@pirsoft.de>]

[-- Attachment #1: Type: text/plain, Size: 121 bytes --]

I thought i'd send this here first, in case someone has something to say
about it. Found by valgrind.

Regards,
  Pierre

[-- Attachment #2: fill-all-sampler-key.diff --]
[-- Type: text/plain, Size: 1162 bytes --]

commit 079d2ff410664a1c2c6dc4bf9982d84314734735
Author: Pierre Willenbrock <pierre@pirsoft.de>
Date:   Sun Apr 25 21:55:48 2010 +0200

    Fill the padding between entrys in the sampler key
    
    This struct is used to generate a hash, ignoring the entry boundaries.

diff --git a/src/mesa/drivers/dri/i965/brw_wm_sampler_state.c b/src/mesa/drivers/dri/i965/brw_wm_sampler_state.c
index d7650af..6504aaf 100644
--- a/src/mesa/drivers/dri/i965/brw_wm_sampler_state.c
+++ b/src/mesa/drivers/dri/i965/brw_wm_sampler_state.c
@@ -228,6 +228,8 @@ brw_wm_sampler_populate_key(struct brw_context *brw,
 {
    GLcontext *ctx = &brw->intel.ctx;
    int unit;
+   char *last_entry_end = ((char*)&key->sampler_count) + 
+      sizeof(key->sampler_count);
 
    key->sampler_count = 0;
 
@@ -240,7 +242,9 @@ brw_wm_sampler_populate_key(struct brw_context *brw,
 	 struct gl_texture_image *firstImage =
 	    texObj->Image[0][intelObj->firstLevel];
 
-	 memset(entry, 0, sizeof(*entry));
+	 memset(last_entry_end, 0, 
+		(char*)entry - last_entry_end + sizeof(*entry));
+	 last_entry_end = ((char*)entry) + sizeof(*entry);
 
          entry->tex_target = texObj->Target;
 

[-- Attachment #3: Type: text/plain, Size: 159 bytes --]

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx