[Mesa-dev] [PATCH 05/11] i965/fs: Check variable_storage return value in fs_visitor::visit

[Ian Romanick]

Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>

On 06/19/2014 05:24 AM, Juha-Pekka Heikkila wrote:
> check variable_storage() found the requested fs_reg.
> 
> Signed-off-by: Juha-Pekka Heikkila <juhapekka.heikkila at gmail.com>
> ---
>  src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/src/mesa/drivers/dri/i965/brw_fs_visitor.cpp b/src/mesa/drivers/dri/i965/brw_fs_visitor.cpp
> index 6352739..654f5fe 100644
> --- a/src/mesa/drivers/dri/i965/brw_fs_visitor.cpp
> +++ b/src/mesa/drivers/dri/i965/brw_fs_visitor.cpp
> @@ -155,6 +155,12 @@ void
>  fs_visitor::visit(ir_dereference_variable *ir)
>  {
>     fs_reg *reg = variable_storage(ir->var);
> +
> +   if (!reg) {
> +      fail("Failed to find variable storage for %s\n", ir->var->name);
> +      this->result = fs_reg(reg_null_d);
> +      return;
> +   }
>     this->result = *reg;
>  }
>  
>