mesa: Fix setup of LocalParams array.
authorEric Anholt <eric@anholt.net>
Mon, 18 Nov 2013 17:55:00 +0000 (09:55 -0800)
committerEric Anholt <eric@anholt.net>
Thu, 21 Nov 2013 00:12:46 +0000 (16:12 -0800)
commit81ff29e30c573fcc134bf717670015ab4654ebcd
tree0ba448cc998e12a38b07b93e2716a7ad00e4a67b
parent5fe49d99f250be1aa6080286ff15862ac4d4c903
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 <michel.daenzer@amd.com>
src/mesa/program/prog_statevars.c
src/mesa/program/program_parse.y