From 2a984da18e4e9ccd79ef24557bd0c8ce0a1b9971 Mon Sep 17 00:00:00 2001 From: Erik Faye-Lund Date: Mon, 29 Mar 2021 12:36:28 +0200 Subject: [PATCH] compiler/glsl: drop rogue argument to _mesa_glsl_error This arugment is not present int the format-string, so we shouldn't pass it to _mesa_glsl_error either. Noticed by Coverity. Fixes: 02dc74fbd72 ("glsl: parse invocations layout qualifier for ARB_gpu_shader5") Reviewed-by: Eric Anholt Part-of: --- src/compiler/glsl/glsl_parser.yy | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/compiler/glsl/glsl_parser.yy b/src/compiler/glsl/glsl_parser.yy index 78bd3610990..13f5d49df6c 100644 --- a/src/compiler/glsl/glsl_parser.yy +++ b/src/compiler/glsl/glsl_parser.yy @@ -1863,7 +1863,7 @@ layout_qualifier_id: !state->EXT_geometry_shader_enable) { _mesa_glsl_error(& @3, state, "GL_ARB_gpu_shader5 invocations " - "qualifier specified", $3); + "qualifier specified"); } }