diff options
author | Bram Moolenaar <Bram@vim.org> | 2020-11-03 18:20:19 +0100 |
---|---|---|
committer | Bram Moolenaar <Bram@vim.org> | 2020-11-03 18:20:19 +0100 |
commit | 0d90e728fe089ff1bb34d6a17f5591a96b57f734 (patch) | |
tree | c4e8d1b172ee4660250ab06fa1d9781553e7f1c7 | |
parent | 6fd3a4ba2320c4a95f02daef5f2ac76d3105d013 (diff) | |
download | vim-git-8.2.1945.tar.gz |
patch 8.2.1945: crash when passing NULL function to reduce()v8.2.1945
Problem: Crash when passing NULL function to reduce().
Solution: Check for NULL pointer and give an error. (Dominique Pellé,
closes #7243)
-rw-r--r-- | src/errors.h | 2 | ||||
-rw-r--r-- | src/list.c | 7 | ||||
-rw-r--r-- | src/testdir/test_listdict.vim | 3 | ||||
-rw-r--r-- | src/version.c | 2 |
4 files changed, 12 insertions, 2 deletions
diff --git a/src/errors.h b/src/errors.h index 1843f3d6f..0e0377a0e 100644 --- a/src/errors.h +++ b/src/errors.h @@ -286,4 +286,6 @@ EXTERN char e_cannot_add_to_null_list[] INIT(= N_("E1130: Cannot add to null list")); EXTERN char e_cannot_add_to_null_blob[] INIT(= N_("E1131: Cannot add to null blob")); +EXTERN char e_missing_function_argument[] + INIT(= N_("E1132: Missing function argument")); #endif diff --git a/src/list.c b/src/list.c index 1da4f3d0c..beea6fae6 100644 --- a/src/list.c +++ b/src/list.c @@ -2552,8 +2552,11 @@ f_reduce(typval_T *argvars, typval_T *rettv) } else func_name = tv_get_string(&argvars[1]); - if (*func_name == NUL) - return; // type error or empty name + if (func_name == NULL || *func_name == NUL) + { + emsg(_(e_missing_function_argument)); + return; + } vim_memset(&funcexe, 0, sizeof(funcexe)); funcexe.evaluate = TRUE; diff --git a/src/testdir/test_listdict.vim b/src/testdir/test_listdict.vim index ade37f8c0..6a7fc4c68 100644 --- a/src/testdir/test_listdict.vim +++ b/src/testdir/test_listdict.vim @@ -740,6 +740,9 @@ func Test_reduce() call assert_equal(42, reduce(test_null_list(), function('add'), 42)) call assert_equal(42, reduce(test_null_blob(), function('add'), 42)) + + " should not crash + call assert_fails('echo reduce([1], test_null_function())', 'E1132:') endfunc " splitting a string to a List using split() diff --git a/src/version.c b/src/version.c index 714d038f8..8052de8d5 100644 --- 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 */ /**/ + 1945, +/**/ 1944, /**/ 1943, |