# HG changeset patch # User Bram Moolenaar # Date 1620910806 -7200 # Node ID 4aebea72c3972919060bfe5d4648f3df202670e0 # Parent 62ec7b94e2b1920363a1f553fcc326e1fb2522c0 patch 8.2.2848: crash whn calling partial Commit: https://github.com/vim/vim/commit/fe8ebdbe5c4e116311c0c0d5937b89ded5c92d01 Author: Dominique Pelle Date: Thu May 13 14:55:55 2021 +0200 patch 8.2.2848: crash whn calling partial Problem: Crash whn calling partial. Solution: Check for NULL pointer. (Dominique Pell?, closes https://github.com/vim/vim/issues/8202) diff --git a/src/eval.c b/src/eval.c --- a/src/eval.c +++ b/src/eval.c @@ -4284,10 +4284,13 @@ eval_index_inner( char_u * partial_name(partial_T *pt) { - if (pt->pt_name != NULL) - return pt->pt_name; - if (pt->pt_func != NULL) - return pt->pt_func->uf_name; + if (pt != NULL) + { + if (pt->pt_name != NULL) + return pt->pt_name; + if (pt->pt_func != NULL) + return pt->pt_func->uf_name; + } return (char_u *)""; } diff --git a/src/evalfunc.c b/src/evalfunc.c --- a/src/evalfunc.c +++ b/src/evalfunc.c @@ -1971,7 +1971,7 @@ internal_func_name(int idx) } /* - * Check the argument types for builting function "idx". + * Check the argument types for builtin function "idx". * Uses the list of types on the type stack: "types". * Return FAIL and gives an error message when a type is wrong. */ @@ -2475,8 +2475,8 @@ f_call(typval_T *argvars, typval_T *rett } else func = tv_get_string(&argvars[0]); - if (*func == NUL) - return; // type error or empty name + if (func == NULL || *func == NUL) + return; // type error, empty name or null function if (argvars[2].v_type != VAR_UNKNOWN) { @@ -2779,7 +2779,7 @@ f_cosh(typval_T *argvars, typval_T *rett /* * Set the cursor position. - * If 'charcol' is TRUE, then use the column number as a character offet. + * If 'charcol' is TRUE, then use the column number as a character offset. * Otherwise use the column number as a byte offset. */ static void diff --git a/src/testdir/test_functions.vim b/src/testdir/test_functions.vim --- a/src/testdir/test_functions.vim +++ b/src/testdir/test_functions.vim @@ -2150,6 +2150,10 @@ func Test_call() eval mydict.len->call([], mydict)->assert_equal(4) call assert_fails("call call('Mylen', [], 0)", 'E715:') call assert_fails('call foo', 'E107:') + + " This once caused a crash. + call call(test_null_function(), []) + call call(test_null_partial(), []) endfunc func Test_char2nr() diff --git a/src/testdir/test_listdict.vim b/src/testdir/test_listdict.vim --- a/src/testdir/test_listdict.vim +++ b/src/testdir/test_listdict.vim @@ -743,6 +743,7 @@ func Test_reduce() " should not crash call assert_fails('echo reduce([1], test_null_function())', 'E1132:') + call assert_fails('echo reduce([1], test_null_partial())', 'E1132:') endfunc " splitting a string to a List using split() diff --git a/src/version.c b/src/version.c --- a/src/version.c +++ b/src/version.c @@ -751,6 +751,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 2848, +/**/ 2847, /**/ 2846,