diff options
| author | Kenneth Graunke <[email protected]> | 2013-09-16 23:41:57 -0700 |
|---|---|---|
| committer | Kenneth Graunke <[email protected]> | 2013-09-19 10:52:57 -0700 |
| commit | 9f64bb2312dece0bba02bcbaf0c48d27d2f69daf (patch) | |
| tree | bfa3e5b171987a09de43b569cdca6c055c771152 | |
| parent | 4e4b0799165ae7b482da7d9af656ed8878500de1 (diff) | |
i965: Fix brw_gs_prog_data_compare to actually check field members.
&a and &b are the address of the local stack variables, not the actual
structures. Instead of comparing the fields of a and b, we compared
...some stack memory.
Not a candidate for stable since GS code doesn't exist in 9.2.
Signed-off-by: Kenneth Graunke <[email protected]>
Reviewed-by: Chad Versace <[email protected]>
| -rw-r--r-- | src/mesa/drivers/dri/i965/brw_vec4_gs.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/mesa/drivers/dri/i965/brw_vec4_gs.c b/src/mesa/drivers/dri/i965/brw_vec4_gs.c index 30989c3faef..064e541624d 100644 --- a/src/mesa/drivers/dri/i965/brw_vec4_gs.c +++ b/src/mesa/drivers/dri/i965/brw_vec4_gs.c @@ -316,7 +316,7 @@ brw_gs_prog_data_compare(const void *in_a, const void *in_b, /* Compare the rest of the struct. */ const unsigned offset = sizeof(struct brw_vec4_prog_data); - if (memcmp(((char *) &a) + offset, ((char *) &b) + offset, + if (memcmp(((char *) a) + offset, ((char *) b) + offset, sizeof(struct brw_gs_prog_data) - offset)) { return false; } |