diff options
| author | Kenneth Graunke <kenneth@whitecape.org> | 2011-08-30 12:34:13 -0700 |
|---|---|---|
| committer | Eric Anholt <eric@anholt.net> | 2011-09-02 12:17:06 -0700 |
| commit | ee41383ab31f6ef5f1d18961de78371d4f52065b (patch) | |
| tree | 4b830fef0de52927bda98f6070bad2e1de5470c9 | |
| parent | 7fdbedac14637b5e8990cb639bd4ddae28c46994 (diff) | |
i965/vs: Fix NULL pointer dereference in pre-Gen6 push constant loading.
According to the comment, we need to load /some/ push constants on
pre-Gen6 hardware or the GPU will hang. The existing code set these
bogus parameters to NULL pointers; unfortunately, the code in
brw_curbe.c that loads them dereferences those pointers. So, change
them to be pointers to an actual floating point value of 0.0.
Signed-off-by: Kenneth Graunke <kenneth@whitecape.org>
Reviewed-by: Eric Anholt <eric@anholt.net>
| -rw-r--r-- | src/mesa/drivers/dri/i965/brw_vec4_emit.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/mesa/drivers/dri/i965/brw_vec4_emit.cpp b/src/mesa/drivers/dri/i965/brw_vec4_emit.cpp index 067f1c98af..2d1c878706 100644 --- a/src/mesa/drivers/dri/i965/brw_vec4_emit.cpp +++ b/src/mesa/drivers/dri/i965/brw_vec4_emit.cpp @@ -117,8 +117,8 @@ vec4_visitor::setup_uniforms(int reg) for (unsigned int i = 0; i < 4; i++) { unsigned int slot = this->uniforms * 4 + i; - - c->prog_data.param[slot] = NULL; + static float zero = 0.0; + c->prog_data.param[slot] = &zero; } this->uniforms++; |