mirror of
https://github.com/luanti-org/luanti.git
synced 2025-06-27 16:36:03 +00:00
Make sure we get a stacktrace for as many lua errors as possible
This commit is contained in:
parent
d753d352f1
commit
362ef5f6ce
6 changed files with 29 additions and 29 deletions
|
@ -54,7 +54,7 @@ int ScriptApiDetached::detached_inventory_AllowMove(
|
|||
if(lua_pcall(L, 7, 1, errorhandler))
|
||||
scriptError();
|
||||
if(!lua_isnumber(L, -1))
|
||||
throw LuaError(NULL, "allow_move should return a number");
|
||||
throw LuaError(L, "allow_move should return a number");
|
||||
int ret = luaL_checkinteger(L, -1);
|
||||
lua_pop(L, 2); // Pop integer and error handler
|
||||
return ret;
|
||||
|
@ -86,7 +86,7 @@ int ScriptApiDetached::detached_inventory_AllowPut(
|
|||
if(lua_pcall(L, 5, 1, errorhandler))
|
||||
scriptError();
|
||||
if(!lua_isnumber(L, -1))
|
||||
throw LuaError(NULL, "allow_put should return a number");
|
||||
throw LuaError(L, "allow_put should return a number");
|
||||
int ret = luaL_checkinteger(L, -1);
|
||||
lua_pop(L, 2); // Pop integer and error handler
|
||||
return ret;
|
||||
|
@ -118,7 +118,7 @@ int ScriptApiDetached::detached_inventory_AllowTake(
|
|||
if(lua_pcall(L, 5, 1, errorhandler))
|
||||
scriptError();
|
||||
if(!lua_isnumber(L, -1))
|
||||
throw LuaError(NULL, "allow_take should return a number");
|
||||
throw LuaError(L, "allow_take should return a number");
|
||||
int ret = luaL_checkinteger(L, -1);
|
||||
lua_pop(L, 2); // Pop integer and error handler
|
||||
return ret;
|
||||
|
|
|
@ -61,7 +61,7 @@ int ScriptApiNodemeta::nodemeta_inventory_AllowMove(v3s16 p,
|
|||
scriptError();
|
||||
lua_remove(L, errorhandler); // Remove error handler
|
||||
if(!lua_isnumber(L, -1))
|
||||
throw LuaError(NULL, "allow_metadata_inventory_move should"
|
||||
throw LuaError(L, "allow_metadata_inventory_move should"
|
||||
" return a number, guilty node: " + nodename);
|
||||
int num = luaL_checkinteger(L, -1);
|
||||
lua_pop(L, 1); // Pop integer
|
||||
|
@ -100,7 +100,7 @@ int ScriptApiNodemeta::nodemeta_inventory_AllowPut(v3s16 p,
|
|||
scriptError();
|
||||
lua_remove(L, errorhandler); // Remove error handler
|
||||
if(!lua_isnumber(L, -1))
|
||||
throw LuaError(NULL, "allow_metadata_inventory_put should"
|
||||
throw LuaError(L, "allow_metadata_inventory_put should"
|
||||
" return a number, guilty node: " + nodename);
|
||||
int num = luaL_checkinteger(L, -1);
|
||||
lua_pop(L, 1); // Pop integer
|
||||
|
@ -139,7 +139,7 @@ int ScriptApiNodemeta::nodemeta_inventory_AllowTake(v3s16 p,
|
|||
scriptError();
|
||||
lua_remove(L, errorhandler); // Remove error handler
|
||||
if(!lua_isnumber(L, -1))
|
||||
throw LuaError(NULL, "allow_metadata_inventory_take should"
|
||||
throw LuaError(L, "allow_metadata_inventory_take should"
|
||||
" return a number, guilty node: " + nodename);
|
||||
int num = luaL_checkinteger(L, -1);
|
||||
lua_pop(L, 1); // Pop integer
|
||||
|
|
|
@ -33,7 +33,7 @@ bool ScriptApiServer::getAuth(const std::string &playername,
|
|||
getAuthHandler();
|
||||
lua_getfield(L, -1, "get_auth");
|
||||
if(lua_type(L, -1) != LUA_TFUNCTION)
|
||||
throw LuaError(NULL, "Authentication handler missing get_auth");
|
||||
throw LuaError(L, "Authentication handler missing get_auth");
|
||||
lua_pushstring(L, playername.c_str());
|
||||
if(lua_pcall(L, 1, 1, errorhandler))
|
||||
scriptError();
|
||||
|
@ -48,13 +48,13 @@ bool ScriptApiServer::getAuth(const std::string &playername,
|
|||
std::string password;
|
||||
bool found = getstringfield(L, -1, "password", password);
|
||||
if(!found)
|
||||
throw LuaError(NULL, "Authentication handler didn't return password");
|
||||
throw LuaError(L, "Authentication handler didn't return password");
|
||||
if(dst_password)
|
||||
*dst_password = password;
|
||||
|
||||
lua_getfield(L, -1, "privileges");
|
||||
if(!lua_istable(L, -1))
|
||||
throw LuaError(NULL, "Authentication handler didn't return privilege table");
|
||||
throw LuaError(L, "Authentication handler didn't return privilege table");
|
||||
if(dst_privs)
|
||||
readPrivileges(-1, *dst_privs);
|
||||
lua_pop(L, 1);
|
||||
|
@ -74,7 +74,7 @@ void ScriptApiServer::getAuthHandler()
|
|||
}
|
||||
lua_remove(L, -2); // Remove minetest
|
||||
if(lua_type(L, -1) != LUA_TTABLE)
|
||||
throw LuaError(NULL, "Authentication handler table not valid");
|
||||
throw LuaError(L, "Authentication handler table not valid");
|
||||
}
|
||||
|
||||
void ScriptApiServer::readPrivileges(int index, std::set<std::string> &result)
|
||||
|
@ -108,7 +108,7 @@ void ScriptApiServer::createAuth(const std::string &playername,
|
|||
lua_getfield(L, -1, "create_auth");
|
||||
lua_remove(L, -2); // Remove auth handler
|
||||
if(lua_type(L, -1) != LUA_TFUNCTION)
|
||||
throw LuaError(NULL, "Authentication handler missing create_auth");
|
||||
throw LuaError(L, "Authentication handler missing create_auth");
|
||||
lua_pushstring(L, playername.c_str());
|
||||
lua_pushstring(L, password.c_str());
|
||||
if(lua_pcall(L, 2, 0, errorhandler))
|
||||
|
@ -128,7 +128,7 @@ bool ScriptApiServer::setPassword(const std::string &playername,
|
|||
lua_getfield(L, -1, "set_password");
|
||||
lua_remove(L, -2); // Remove auth handler
|
||||
if(lua_type(L, -1) != LUA_TFUNCTION)
|
||||
throw LuaError(NULL, "Authentication handler missing set_password");
|
||||
throw LuaError(L, "Authentication handler missing set_password");
|
||||
lua_pushstring(L, playername.c_str());
|
||||
lua_pushstring(L, password.c_str());
|
||||
if(lua_pcall(L, 2, 1, errorhandler))
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue