Module: Mesa Branch: master Commit: 494e4b8976a48e37e3cb51338ad69e085ea5061a URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=494e4b8976a48e37e3cb51338ad69e085ea5061a
Author: Nicolai Hähnle <[email protected]> Date: Fri May 13 01:56:44 2016 -0500 st_glsl_to_tgsi: don't read potentially uninitialized buffer variable Found by -fsanitize=undefined. Note that this should be a harmless issue in practice because the inst->op check always dominates anyway. Reviewed-by: Brian Paul <[email protected]> Reviewed-by: Ilia Mirkin <[email protected]> --- src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp b/src/mesa/state_tracker/st_glsl_to_tgsi.cpp index b7eaa13..8c6fe79 100644 --- a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp +++ b/src/mesa/state_tracker/st_glsl_to_tgsi.cpp @@ -3334,7 +3334,7 @@ glsl_to_tgsi_visitor::visit_ssbo_intrinsic(ir_call *ir) inst = (glsl_to_tgsi_instruction *)inst->get_prev(); if (inst->op == TGSI_OPCODE_UADD) inst = (glsl_to_tgsi_instruction *)inst->get_prev(); - } while (inst && inst->buffer.file == PROGRAM_UNDEFINED && inst->op == op); + } while (inst && inst->op == op && inst->buffer.file == PROGRAM_UNDEFINED); } void _______________________________________________ mesa-commit mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-commit
