From dad649b66c8bd3f10fbf96cedea3ab7133ade758 Mon Sep 17 00:00:00 2001 From: Ian Romanick Date: Mon, 14 Sep 2015 11:59:22 -0700 Subject: [PATCH] t_dd_dmatmp: Use addition instead of subtraction in loop bounds This is used everywhere else in this file because it avoids problems when count is zero (due to trimming). No piglit regressions on i915 (G33) or radeon (Radeon 7500). Signed-off-by: Ian Romanick Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=38109 Reviewed-by: Brian Paul Cc: Marius Predut Cc: "10.6 11.0" (cherry picked from commit 25543d8ec506ef32599af6f5e0dd735e01b39909) --- src/mesa/tnl_dd/t_dd_dmatmp.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mesa/tnl_dd/t_dd_dmatmp.h b/src/mesa/tnl_dd/t_dd_dmatmp.h index dea95460c74..ef2991a2aed 100644 --- a/src/mesa/tnl_dd/t_dd_dmatmp.h +++ b/src/mesa/tnl_dd/t_dd_dmatmp.h @@ -623,7 +623,7 @@ static void TAG(render_quads_verts)( struct gl_context *ctx, INIT(GL_TRIANGLES); - for (j = 0; j < count-3; j += 4) { + for (j = 0; j + 3 < count; j += 4) { void *tmp = ALLOC_VERTS( 6 ); /* Send v0, v1, v3 */