diff options
| author | rubenwardy <rw@rubenwardy.com> | 2019-03-12 07:56:56 +0000 |
|---|---|---|
| committer | rubenwardy <rw@rubenwardy.com> | 2019-03-12 20:25:48 +0000 |
| commit | 444ec1e4128e68523f1bbf51aa03e8bf2006156c (patch) | |
| tree | 2b09dda5ea4135b0a3b38e563b7c2d016d94dc44 /src/script/cpp_api | |
| parent | 82739f4d7d634eaccdc8414382da5d55ae39734d (diff) | |
| download | minetest-444ec1e4128e68523f1bbf51aa03e8bf2006156c.tar.gz minetest-444ec1e4128e68523f1bbf51aa03e8bf2006156c.tar.bz2 minetest-444ec1e4128e68523f1bbf51aa03e8bf2006156c.zip | |
HPChange Reason: Fix push after free, and type being overwritten (#8359)
* HPChange Reason: Fix push after free, and type being overwritten
Fixes #8227 and #8344
Diffstat (limited to 'src/script/cpp_api')
| -rw-r--r-- | src/script/cpp_api/s_base.cpp | 14 |
1 files changed, 9 insertions, 5 deletions
diff --git a/src/script/cpp_api/s_base.cpp b/src/script/cpp_api/s_base.cpp index bf89f748c..a8ed902dd 100644 --- a/src/script/cpp_api/s_base.cpp +++ b/src/script/cpp_api/s_base.cpp @@ -384,14 +384,18 @@ void ScriptApiBase::objectrefGetOrCreate(lua_State *L, void ScriptApiBase::pushPlayerHPChangeReason(lua_State *L, const PlayerHPChangeReason &reason) { - if (reason.lua_reference >= 0) { + if (reason.hasLuaReference()) lua_rawgeti(L, LUA_REGISTRYINDEX, reason.lua_reference); - luaL_unref(L, LUA_REGISTRYINDEX, reason.lua_reference); - } else + else lua_newtable(L); - lua_pushstring(L, reason.getTypeAsString().c_str()); - lua_setfield(L, -2, "type"); + lua_getfield(L, -1, "type"); + bool has_type = (bool)lua_isstring(L, -1); + lua_pop(L, 1); + if (!has_type) { + lua_pushstring(L, reason.getTypeAsString().c_str()); + lua_setfield(L, -2, "type"); + } lua_pushstring(L, reason.from_mod ? "mod" : "engine"); lua_setfield(L, -2, "from"); |