# HG changeset patch # User Bram Moolenaar # Date 1655490603 -7200 # Node ID e391590249a1d9642e1e86f6cba37b0676230fcb # Parent ee8ebed77cee75be3c410d55955110512c5aeceb patch 8.2.5117: crash when calling a Lua callback from a :def function Commit: https://github.com/vim/vim/commit/7d149f899d423b7bf2b90d7b11ebe3e560c462b9 Author: Bram Moolenaar Date: Fri Jun 17 19:23:34 2022 +0100 patch 8.2.5117: crash when calling a Lua callback from a :def function Problem: Crash when calling a Lua callback from a :def function. (Bohdan Makohin) Solution: Handle FC_CFUNC in call_user_func_check(). (closes #10587) diff --git a/src/testdir/test_lua.vim b/src/testdir/test_lua.vim --- a/src/testdir/test_lua.vim +++ b/src/testdir/test_lua.vim @@ -664,6 +664,17 @@ func Test_lua_blob() \ '[string "vim chunk"]:1: string expected, got table') endfunc +def Vim9Test(Callback: func()) + Callback() +enddef + +func Test_call_lua_func_from_vim9_func() + " this only tests that Vim doesn't crash + lua << EOF +vim.fn.Vim9Test(function () print('Hello') end) +EOF +endfunc + func Test_lua_funcref() function I(x) return a:x diff --git a/src/userfunc.c b/src/userfunc.c --- a/src/userfunc.c +++ b/src/userfunc.c @@ -3021,6 +3021,15 @@ call_user_func_check( { int error; +#ifdef FEAT_LUA + if (fp->uf_flags & FC_CFUNC) + { + cfunc_T cb = fp->uf_cb; + + return (*cb)(argcount, argvars, rettv, fp->uf_cb_state); + } +#endif + if (fp->uf_flags & FC_RANGE && funcexe->fe_doesrange != NULL) *funcexe->fe_doesrange = TRUE; error = check_user_func_argcount(fp, argcount); @@ -3584,14 +3593,6 @@ call_func( if (fp != NULL && (fp->uf_flags & FC_DELETED)) error = FCERR_DELETED; -#ifdef FEAT_LUA - else if (fp != NULL && (fp->uf_flags & FC_CFUNC)) - { - cfunc_T cb = fp->uf_cb; - - error = (*cb)(argcount, argvars, rettv, fp->uf_cb_state); - } -#endif else if (fp != NULL) { if (funcexe->fe_argv_func != NULL) diff --git a/src/version.c b/src/version.c --- a/src/version.c +++ b/src/version.c @@ -735,6 +735,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 5117, +/**/ 5116, /**/ 5115,