[Piglit] [PATCH] ati_envmap_bumpmap-bump: Use more compatible sizeof operand type.

Vinson Lee vlee at freedesktop.org
Sat Jan 18 21:01:41 PST 2014


Fixes "Allocator sizeof operand mismatch" bug reported by Clang Static
Analyzer.

Signed-off-by: Vinson Lee <vlee at freedesktop.org>
---
 tests/spec/ati_envmap_bumpmap/bump.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/spec/ati_envmap_bumpmap/bump.c b/tests/spec/ati_envmap_bumpmap/bump.c
index ded1e5e..41c06cc 100644
--- a/tests/spec/ati_envmap_bumpmap/bump.c
+++ b/tests/spec/ati_envmap_bumpmap/bump.c
@@ -216,7 +216,7 @@ piglit_init(int argc, char **argv)
 		piglit_report_result(PIGLIT_FAIL);
 	}
 
-	units = malloc(num_units * sizeof(GLenum));
+	units = malloc(num_units * sizeof(int));
 	glGetTexBumpParameterivATI(GL_BUMP_TEX_UNITS_ATI, units);
 	found_unit_1 = GL_FALSE;
 	for (i = 0; i < num_units; i++) {
-- 
1.8.3.2



More information about the Piglit mailing list