mirror of
https://github.com/neovim/neovim.git
synced 2024-12-31 17:13:26 -07:00
vim-patch:8.0.0085
Problem: Using freed memory with recursive function call. (Dominique Pelle)
Solution: Make a copy of the function name.
8a01f969c1
This commit is contained in:
parent
c40093f47a
commit
710546c5e9
@ -4239,11 +4239,17 @@ static int eval7(
|
|||||||
// use its contents.
|
// use its contents.
|
||||||
s = deref_func_name((const char *)s, &len, &partial, !evaluate);
|
s = deref_func_name((const char *)s, &len, &partial, !evaluate);
|
||||||
|
|
||||||
|
// Need to make a copy, in case evaluating the arguments makes
|
||||||
|
// the name invalid.
|
||||||
|
s = xmemdupz(s, len);
|
||||||
|
|
||||||
// Invoke the function.
|
// Invoke the function.
|
||||||
ret = get_func_tv(s, len, rettv, arg,
|
ret = get_func_tv(s, len, rettv, arg,
|
||||||
curwin->w_cursor.lnum, curwin->w_cursor.lnum,
|
curwin->w_cursor.lnum, curwin->w_cursor.lnum,
|
||||||
&len, evaluate, partial, NULL);
|
&len, evaluate, partial, NULL);
|
||||||
|
|
||||||
|
xfree(s);
|
||||||
|
|
||||||
// If evaluate is false rettv->v_type was not set in
|
// If evaluate is false rettv->v_type was not set in
|
||||||
// get_func_tv, but it's needed in handle_subscript() to parse
|
// get_func_tv, but it's needed in handle_subscript() to parse
|
||||||
// what follows. So set it here.
|
// what follows. So set it here.
|
||||||
|
@ -1,32 +1,42 @@
|
|||||||
"Tests for nested functions
|
"Tests for nested functions
|
||||||
"
|
"
|
||||||
function! NestedFunc()
|
func NestedFunc()
|
||||||
fu! Func1()
|
func! Func1()
|
||||||
let g:text .= 'Func1 '
|
let g:text .= 'Func1 '
|
||||||
endfunction
|
endfunc
|
||||||
call Func1()
|
call Func1()
|
||||||
fu! s:func2()
|
func! s:func2()
|
||||||
let g:text .= 's:func2 '
|
let g:text .= 's:func2 '
|
||||||
endfunction
|
endfunc
|
||||||
call s:func2()
|
call s:func2()
|
||||||
fu! s:_func3()
|
func! s:_func3()
|
||||||
let g:text .= 's:_func3 '
|
let g:text .= 's:_func3 '
|
||||||
endfunction
|
endfunc
|
||||||
call s:_func3()
|
call s:_func3()
|
||||||
let fn = 'Func4'
|
let fn = 'Func4'
|
||||||
fu! {fn}()
|
func! {fn}()
|
||||||
let g:text .= 'Func4 '
|
let g:text .= 'Func4 '
|
||||||
endfunction
|
endfunc
|
||||||
call {fn}()
|
call {fn}()
|
||||||
let fn = 'func5'
|
let fn = 'func5'
|
||||||
fu! s:{fn}()
|
func! s:{fn}()
|
||||||
let g:text .= 's:func5'
|
let g:text .= 's:func5'
|
||||||
endfunction
|
endfunc
|
||||||
call s:{fn}()
|
call s:{fn}()
|
||||||
endfunction
|
endfunc
|
||||||
|
|
||||||
function! Test_nested_functions()
|
func Test_nested_functions()
|
||||||
let g:text = ''
|
let g:text = ''
|
||||||
call NestedFunc()
|
call NestedFunc()
|
||||||
call assert_equal('Func1 s:func2 s:_func3 Func4 s:func5', g:text)
|
call assert_equal('Func1 s:func2 s:_func3 Func4 s:func5', g:text)
|
||||||
endfunction
|
endfunction
|
||||||
|
|
||||||
|
func Test_nested_argument()
|
||||||
|
func g:X()
|
||||||
|
let g:Y = function('sort')
|
||||||
|
endfunc
|
||||||
|
let g:Y = function('sort')
|
||||||
|
echo g:Y([], g:X())
|
||||||
|
delfunc g:X
|
||||||
|
unlet g:Y
|
||||||
|
endfunc
|
||||||
|
@ -644,7 +644,7 @@ static const int included_patches[] = {
|
|||||||
// 88,
|
// 88,
|
||||||
// 87 NA
|
// 87 NA
|
||||||
// 86,
|
// 86,
|
||||||
// 85,
|
85,
|
||||||
84,
|
84,
|
||||||
83,
|
83,
|
||||||
// 82 NA
|
// 82 NA
|
||||||
|
Loading…
Reference in New Issue
Block a user