diff options
author | Paul Berry <[email protected]> | 2013-10-15 09:43:22 -0700 |
---|---|---|
committer | Paul Berry <[email protected]> | 2013-10-17 11:51:06 -0700 |
commit | e2d1eaa32a83204646bcccc029fdd31dbaee6d5e (patch) | |
tree | 825f628a176630924f3cb291feacc91066d09093 /src/glsl/opt_constant_propagation.cpp | |
parent | 79e835a71202f1a6e112266b199e833d952da606 (diff) |
glsl: In update_max_array_access, fix interface instance check.
In commit f878d20 (glsl: Update ir_variable::max_ifc_array_access
properly), I accidentally used the wrong kind of check to determine
whether the variable being accessed was an interface instance (I used
var->get_interface_type() != NULL when I should have used
var->is_interface_instance()). As a result, if an unnamed interface
block contained a struct which contained an array,
update_max_array_access() would mistakenly interpret the struct as a
named interface block and try to dereference a null
var->max_ifc_array_access.
This patch corrects the check, fixing the null dereference.
Fixes piglit test interface-block-struct-nesting.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=70368
Reviewed-by: Matt Turner <[email protected]>
Diffstat (limited to 'src/glsl/opt_constant_propagation.cpp')
0 files changed, 0 insertions, 0 deletions