From e89dbc4ec607bef8d819c7e4e7a7a0644a5b38ab Mon Sep 17 00:00:00 2001 From: Thierry Fournier Date: Mon, 19 Sep 2022 09:01:53 +0200 Subject: [PATCH] BUG/MINOR: hlua: Remove \n in Lua error message built with memprintf Because memprintf return an error to the caller and not on screen. the function which perform display of message on the right output is in charge of adding \n if it is necessary. This patch may be backported. (cherry picked from commit 70e38e91b41c3446cd6d7993dd71fe6100996fa3) Signed-off-by: Christopher Faulet --- src/hlua.c | 26 +++++++++++++------------- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/src/hlua.c b/src/hlua.c index 3e835837c..4404f9866 100644 --- a/src/hlua.c +++ b/src/hlua.c @@ -11182,7 +11182,7 @@ static int hlua_parse_maxmem(char **args, int section_type, struct proxy *curpx, char *error; if (*(args[1]) == 0) { - memprintf(err, "'%s' expects an integer argument (Lua memory size in MB).\n", args[0]); + memprintf(err, "'%s' expects an integer argument (Lua memory size in MB).", args[0]); return -1; } hlua_global_allocator.limit = strtoll(args[1], &error, 10) * 1024L * 1024L; @@ -11226,24 +11226,24 @@ static int hlua_load_state(char *filename, lua_State *L, char **err) case LUA_OK: break; case LUA_ERRRUN: - memprintf(err, "Lua runtime error: %s\n", lua_tostring(L, -1)); + memprintf(err, "Lua runtime error: %s", lua_tostring(L, -1)); lua_pop(L, 1); return -1; case LUA_ERRMEM: - memprintf(err, "Lua out of memory error\n"); + memprintf(err, "Lua out of memory error"); return -1; case LUA_ERRERR: - memprintf(err, "Lua message handler error: %s\n", lua_tostring(L, -1)); + memprintf(err, "Lua message handler error: %s", lua_tostring(L, -1)); lua_pop(L, 1); return -1; #if defined(LUA_VERSION_NUM) && LUA_VERSION_NUM <= 503 case LUA_ERRGCMM: - memprintf(err, "Lua garbage collector error: %s\n", lua_tostring(L, -1)); + memprintf(err, "Lua garbage collector error: %s", lua_tostring(L, -1)); lua_pop(L, 1); return -1; #endif default: - memprintf(err, "Lua unknown error: %s\n", lua_tostring(L, -1)); + memprintf(err, "Lua unknown error: %s", lua_tostring(L, -1)); lua_pop(L, 1); return -1; } @@ -11256,7 +11256,7 @@ static int hlua_load(char **args, int section_type, struct proxy *curpx, char **err) { if (*(args[1]) == 0) { - memprintf(err, "'%s' expects a file name as parameter.\n", args[0]); + memprintf(err, "'%s' expects a file name as parameter.", args[0]); return -1; } @@ -11273,7 +11273,7 @@ static int hlua_load_per_thread(char **args, int section_type, struct proxy *cur int len; if (*(args[1]) == 0) { - memprintf(err, "'%s' expects a file as parameter.\n", args[0]); + memprintf(err, "'%s' expects a file as parameter.", args[0]); return -1; } @@ -11520,7 +11520,7 @@ __LJMP static int hlua_ckch_set(lua_State *L) ret = lua_getfield(L, -1, "filename"); if (ret != LUA_TSTRING) { - memprintf(&err, "%sNo filename specified!\n", err ? err : ""); + memprintf(&err, "%sNo filename specified!", err ? err : ""); errcode |= ERR_ALERT | ERR_FATAL; goto end; } @@ -11530,7 +11530,7 @@ __LJMP static int hlua_ckch_set(lua_State *L) /* look for the filename in the tree */ old_ckchs = ckchs_lookup(filename); if (!old_ckchs) { - memprintf(&err, "%sCan't replace a certificate which is not referenced by the configuration!\n", err ? err : ""); + memprintf(&err, "%sCan't replace a certificate which is not referenced by the configuration!", err ? err : ""); errcode |= ERR_ALERT | ERR_FATAL; goto end; } @@ -11538,7 +11538,7 @@ __LJMP static int hlua_ckch_set(lua_State *L) new_ckchs = ckchs_dup(old_ckchs); if (!new_ckchs) { - memprintf(&err, "%sCannot allocate memory!\n", err ? err : ""); + memprintf(&err, "%sCannot allocate memory!", err ? err : ""); errcode |= ERR_ALERT | ERR_FATAL; goto end; } @@ -11567,14 +11567,14 @@ __LJMP static int hlua_ckch_set(lua_State *L) /* this is the default type, the field is not supported */ if (cert_ext == NULL) { - memprintf(&err, "%sUnsupported field '%s'\n", err ? err : "", field); + memprintf(&err, "%sUnsupported field '%s'", err ? err : "", field); errcode |= ERR_ALERT | ERR_FATAL; goto end; } /* appply the change on the duplicate */ if (cert_ext->load(filename, payload, ckch, &err) != 0) { - memprintf(&err, "%sCan't load the payload for '%s'\n", err ? err : "", cert_ext->ext); + memprintf(&err, "%sCan't load the payload for '%s'", err ? err : "", cert_ext->ext); errcode |= ERR_ALERT | ERR_FATAL; goto end; }