wyj/neovim
1
0
Fork 0
mirror of https://github.com/neovim/neovim.git synced 2024-09-17 20:58:20 -04:00
Code Issues Packages Projects Releases Wiki Activity

fix(lua): memory leak when using invalid syntax with exists() (#26530)

Browse Source
This commit is contained in:
zeertzjq 2023-12-12 20:34:02 +08:00 committed by GitHub
parent 1d4a5cd185
commit b40170f7a3
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 24 additions and 36 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/nvim/lua/executor.c
View File

@ -2326,7 +2326,9 @@ bool nlua_func_exists(const char *lua_funcname)
Object result = NLUA_EXEC_STATIC("return type(loadstring(...)()) == 'function'", args, &err);
xfree(str);
api_clear_error(&err);
if (result.type != kObjectTypeBoolean) {
api_free_object(result);
return false;
}
return result.data.boolean;

37
test/functional/lua/vim_spec.lua
View File

@ -3351,32 +3351,23 @@ describe('vim.keymap', function()
eq(1, exec_lua[[return GlobalCount]])
end)
end)
it('exists() can check a lua function', function()
eq(true, exec_lua[[
describe('Vimscript function exists()', function()
it('can check a lua function', function()
eq(1, exec_lua[[
_G.test = function() print("hello") end
return vim.fn.exists('v:lua.test') == 1
return vim.fn.exists('v:lua.test')
]])
eq(true, exec_lua[[
return vim.fn.exists('v:lua.require("mpack").decode') == 1
]])
eq(true, exec_lua[[
return vim.fn.exists("v:lua.require('vim.lsp').start") == 1
]])
eq(true, exec_lua[[
return vim.fn.exists('v:lua.require"vim.lsp".start') == 1
]])
eq(true, exec_lua[[
return vim.fn.exists("v:lua.require'vim.lsp'.start") == 1
]])
eq(false, exec_lua[[
return vim.fn.exists("v:lua.require'vim.lsp'.unknown") == 1
]])
eq(1, funcs.exists('v:lua.require("mpack").decode'))
eq(1, funcs.exists("v:lua.require('mpack').decode"))
eq(1, funcs.exists('v:lua.require"mpack".decode'))
eq(1, funcs.exists("v:lua.require'mpack'.decode"))
eq(1, funcs.exists("v:lua.require('vim.lsp').start"))
eq(1, funcs.exists('v:lua.require"vim.lsp".start'))
eq(1, funcs.exists("v:lua.require'vim.lsp'.start"))
eq(0, funcs.exists("v:lua.require'vim.lsp'.unknown"))
eq(0, funcs.exists('v:lua.?'))
end)
end)

9
test/functional/treesitter/fold_spec.lua
View File

@ -639,9 +639,7 @@ t2]])
{1:│}t1 |
{1:│}^ |
{1:+}{2:+-- 2 lines: # h2·····················}|
{3:~ }|
{3:~ }|
{3:~ }|
{3:~ }|*3
{4:-- INSERT --} |
]]}
@ -656,10 +654,7 @@ t2]])
{1:-}^t1 |
{1:-}# h2 |
{1:│}t2 |
{3:~ }|
{3:~ }|
{3:~ }|
{3:~ }|
{3:~ }|*4
1 line less; before #2 {MATCH:.*}|
]]}
end)