BUG/MINOR: hlua/init: coroutine may not resume itself

It's not supported to call lua_resume with <L> and <from> designating
the same lua coroutine. It didn't cause visible bugs so far because
Lua 5.3 used to be more permissive about this, and moreover, yielding
is not involved during the hlua init state.

But this is wrong usage, and the doc clearly specifies that the <from>
argument can be NULL when there is no such coroutine, which is the case
here.

This should be backported in every stable versions.

diff --git a/src/hlua.c b/src/hlua.c
index b3c1b5da5b61349e7f5af125a423bfd9c329f488..e125eff828f971866ad9ddadcd83199fbf7c166d 100644 (file)
--- a/src/hlua.c
+++ b/src/hlua.c
@@ -12872,9 +12872,9 @@ int hlua_post_init_state(lua_State *L)
                hlua_unref(L, init->function_ref);
 
 #if defined(LUA_VERSION_NUM) && LUA_VERSION_NUM >= 504
-               ret = lua_resume(L, L, 0, &nres);
+               ret = lua_resume(L, NULL, 0, &nres);
 #else
-               ret = lua_resume(L, L, 0);
+               ret = lua_resume(L, NULL, 0);
 #endif
                kind = NULL;
                switch (ret) {