From 4cccf8c6b2c2b1af0afafef52bc87477ba7f817a Mon Sep 17 00:00:00 2001 From: Benoit Giannangeli Date: Fri, 12 May 2017 08:20:15 +0200 Subject: luaV_concat: 'total' should not be used inside the loop If there's a table in the concat sequence, we ended up with a toconcat array with undefined slots. --- tests/test-suite/events.js | 45 +++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 45 insertions(+) (limited to 'tests') diff --git a/tests/test-suite/events.js b/tests/test-suite/events.js index 8aa7aa7..04b4b1a 100644 --- a/tests/test-suite/events.js +++ b/tests/test-suite/events.js @@ -428,3 +428,48 @@ test("[test-suite] events: __eq between userdata", function (t) { }, "Lua program ran without error"); }); + + +test("[test-suite] events: concat", function (t) { + let luaCode = ` + t = {} + + t.__concat = function (a,b,c) + assert(c == nil) + if type(a) == 'table' then a = a.val end + if type(b) == 'table' then b = b.val end + if A then return a..b + else + return setmetatable({val=a..b}, t) + end + end + + c = {val="c"}; setmetatable(c, t) + d = {val="d"}; setmetatable(d, t) + + A = true + assert(c..d == 'cd') + assert(0 .."a".."b"..c..d.."e".."f"..(5+3).."g" == "0abcdef8g") + `, L; + + t.plan(2); + + t.doesNotThrow(function () { + + L = lauxlib.luaL_newstate(); + + lualib.luaL_openlibs(L); + + ltests.luaopen_tests(L); + + lauxlib.luaL_loadstring(L, lua.to_luastring(luaCode)); + + }, "Lua program loaded without error"); + + t.doesNotThrow(function () { + + lua.lua_call(L, 0, -1); + + }, "Lua program ran without error"); + +}); -- cgit v1.2.3-54-g00ecf