BUG/MINOR: vars: truncate the variable name in error reports about scope.
When a variable starts with the wrong scope, it is named without stripping
the extra characters that follow it, which usually are closing parenthesis.
Let's make sure we only report what is expected.
This may be backported to 2.4.
(cherry picked from commit 1402fef58aedbac671dd81b4ac93dc7008fcde50)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>
diff --git a/src/vars.c b/src/vars.c
index 156eb94..304945f 100644
--- a/src/vars.c
+++ b/src/vars.c
@@ -261,8 +261,8 @@
*scope = SCOPE_CHECK;
}
else {
- memprintf(err, "invalid variable name '%s'. A variable name must be start by its scope. "
- "The scope can be 'proc', 'sess', 'txn', 'req', 'res' or 'check'", name);
+ memprintf(err, "invalid variable name '%.*s'. A variable name must be start by its scope. "
+ "The scope can be 'proc', 'sess', 'txn', 'req', 'res' or 'check'", len, name);
return res;
}