diff --git a/lvm.c b/lvm.c index 7023a04d..bbadf0a6 100644 --- a/lvm.c +++ b/lvm.c @@ -361,7 +361,12 @@ void luaV_finishset (lua_State *L, const TValue *t, TValue *key, } t = tm; /* else repeat assignment over 'tm' */ if (luaV_fastget(L, t, key, slot, luaH_get)) { - luaV_finishfastset(L, t, slot, val); + /* execute 'luaV_finishfastset', but preserving the original 't' + for the barrier. 't' and 'slot' can point to the same value, + and so the assignment can change 't' value */ + GCObject *h = gcvalue(t); + setobj2t(L, cast(TValue *,slot), val); + luaC_barrierback(L, h, val); return; /* done */ } /* else 'return luaV_finishset(L, t, key, val, slot)' (loop) */ diff --git a/testes/events.lua b/testes/events.lua index def13dc8..d0a48c66 100644 --- a/testes/events.lua +++ b/testes/events.lua @@ -382,6 +382,16 @@ do end +do -- bug in 5.4 + local parent = {} + parent.__newindex = parent + collectgarbage() + local child = setmetatable({}, parent) + child.__newindex = {x = "hello"} + collectgarbage("step") + assert(parent.__newindex.x == "hello") +end + -- concat metamethod x numbers (bug in 5.1.1) c = {}