From df68f7ec96c0c9d0e8f50643b96ec9062b7aa658 Mon Sep 17 00:00:00 2001 From: Valentine Krasnobaeva Date: Mon, 30 Sep 2024 15:29:47 +0200 Subject: [PATCH] BUG/MINOR: cfgparse-global: fix allowed args number for setenv Keywords setenv and presetenv take 2 arguments: variable name and value. So, the total number, that should be passed to alertif_too_many_args is 2 ("setenv ") instead of 3. For alertif_too_many_args the first argument index is 0. This should be backported in all stable versions. --- src/cfgparse-global.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/cfgparse-global.c b/src/cfgparse-global.c index 3a59824cb..0c8a71d0b 100644 --- a/src/cfgparse-global.c +++ b/src/cfgparse-global.c @@ -798,7 +798,7 @@ int cfg_parse_global(const char *file, int linenum, char **args, int kwm) #endif /* ! USE_CPU_AFFINITY */ } else if (strcmp(args[0], "setenv") == 0 || strcmp(args[0], "presetenv") == 0) { - if (alertif_too_many_args(3, file, linenum, args, &err_code)) + if (alertif_too_many_args(2, file, linenum, args, &err_code)) goto out; if (*(args[2]) == 0) {