Reviewed-by: Ian Romanick <ian.d.roman...@intel.com>
On 05/14/2014 10:55 AM, Juha-Pekka Heikkila wrote:
> Check return value from hash_table_find before using it as a pointer
>
> Signed-off-by: Juha-Pekka Heikkila <juhapekka.heikk...@gmail.com>
> ---
> src/glsl/loop_analysis.cpp | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/src/glsl/loop_analysis.cpp b/src/glsl/loop_analysis.cpp
> index d6a9ac7..78ac300 100644
> --- a/src/glsl/loop_analysis.cpp
> +++ b/src/glsl/loop_analysis.cpp
> @@ -589,8 +589,10 @@ get_basic_induction_increment(ir_assignment *ir,
> hash_table *var_hash)
> loop_variable *lv =
> (loop_variable *) hash_table_find(var_hash, inc_var);
>
> - if (!lv->is_loop_constant())
> - inc = NULL;
> + if (lv == NULL || !lv->is_loop_constant()) {
> + assert(lv != NULL);
> + inc = NULL;
> + }
> } else
> inc = NULL;
> }
>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
