summaryrefslogtreecommitdiffstats
path: root/src/mesa/drivers/dri/swrast
diff options
context:
space:
mode:
authorEric Anholt <[email protected]>2013-11-18 09:55:00 -0800
committerEric Anholt <[email protected]>2013-11-20 16:12:46 -0800
commit81ff29e30c573fcc134bf717670015ab4654ebcd (patch)
tree0ba448cc998e12a38b07b93e2716a7ad00e4a67b /src/mesa/drivers/dri/swrast
parent5fe49d99f250be1aa6080286ff15862ac4d4c903 (diff)
mesa: Fix setup of LocalParams array.
i965 passed piglit, but swrast and gallium both segfaulted without this. i965 happened to work because it never ran _mesa_load_state_parameters() on the new program before the test called glProgramLocalParameter(), which was allocating a LocalParams array for the fallback path. v2: Since v1 threw away old localparams data, leaked old LocalParams memory, only fixed fragment programs, and I was dubious of my previous invariants already (nothing but program_parse.y will generate LocalParams, and only that one path of program_parse.y will), just late-allocate localparams at the other point of dereferencing them. This adds overhead to _mesa_load_state_parameter, which is uncomfortable, but I'm pretty sure that giant switch statement is super slow already. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=71734 Tested-by: Michel Dänzer <[email protected]>
Diffstat (limited to 'src/mesa/drivers/dri/swrast')
0 files changed, 0 insertions, 0 deletions