From: Petr Vaněk <arkamar@atlas.cz>
Date: Tue, 2 Dec 2025 21:06:16 +0000 (+0100)
Subject: [Fix] Use 0 instead of LUA_OK for lua_pcall result comparison
X-Git-Tag: 3.14.2~42^2
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=refs%2Fpull%2F5774%2Fhead;p=thirdparty%2Frspamd.git

[Fix] Use 0 instead of LUA_OK for lua_pcall result comparison

Lua 5.1 does not define LUA_OK, so the result of lua_pcall is compared
to 0 instead, which is consistent with the rest of the codebase.

Link: https://bugs.gentoo.org/967009
---

diff --git a/src/rspamadm/confighelp.c b/src/rspamadm/confighelp.c
index 04c8845a16..68617f86d2 100644
--- a/src/rspamadm/confighelp.c
+++ b/src/rspamadm/confighelp.c
@@ -207,7 +207,7 @@ rspamadm_confighelp_load_plugins_doc(struct rspamd_config *cfg)
 	lua_getglobal(L, "require");
 	lua_pushstring(L, "rspamadm.confighelp_plugins");
 
-	if (lua_pcall(L, 1, 1, 0) != LUA_OK) {
+	if (lua_pcall(L, 1, 1, 0) != 0) {
 		rspamd_fprintf(stderr, "cannot load confighelp_plugins module: %s\n",
 					   lua_tostring(L, -1));
 		lua_pop(L, 1);
@@ -221,7 +221,7 @@ rspamadm_confighelp_load_plugins_doc(struct rspamd_config *cfg)
 		return NULL;
 	}
 
-	if (lua_pcall(L, 0, 1, 0) != LUA_OK) {
+	if (lua_pcall(L, 0, 1, 0) != 0) {
 		rspamd_fprintf(stderr, "cannot execute confighelp_plugins function: %s\n",
 					   lua_tostring(L, -1));
 		lua_pop(L, 1);