diff options
author | Bram Moolenaar <Bram@vim.org> | 2020-10-03 13:41:53 +0200 |
---|---|---|
committer | Bram Moolenaar <Bram@vim.org> | 2020-10-03 13:41:53 +0200 |
commit | c1f0066b642fb62e37b300033223ff3ee4aafcd6 (patch) | |
tree | 013650efa41b9378bdc9166353aa994b37703c3a | |
parent | 016faaa1b2fe242022725b29e1353c42105d045c (diff) | |
download | vim-git-c1f0066b642fb62e37b300033223ff3ee4aafcd6.tar.gz |
patch 8.2.1789: Vim9: crash with invalid list constantv8.2.1789
Problem: Vim9: crash with invalid list constant. (Dhiraj Mishra)
Solution: Return FAIL when compiling the list fails. (closes #7066)
-rw-r--r-- | src/errors.h | 2 | ||||
-rw-r--r-- | src/testdir/test_vim9_expr.vim | 10 | ||||
-rw-r--r-- | src/version.c | 2 | ||||
-rw-r--r-- | src/vim9compile.c | 5 |
4 files changed, 18 insertions, 1 deletions
diff --git a/src/errors.h b/src/errors.h index bc6258caf..cddd6910f 100644 --- a/src/errors.h +++ b/src/errors.h @@ -276,4 +276,6 @@ EXTERN char e_final_requires_a_value[] INIT(= N_("E1125: Final requires a value")); EXTERN char e_cannot_use_let_in_vim9_script[] INIT(= N_("E1126: Cannot use :let in Vim9 script")); +EXTERN char e_missing_name_after_dot[] + INIT(= N_("E1127: Missing name after dot")); #endif diff --git a/src/testdir/test_vim9_expr.vim b/src/testdir/test_vim9_expr.vim index 0a0bcb41a..fa33089e8 100644 --- a/src/testdir/test_vim9_expr.vim +++ b/src/testdir/test_vim9_expr.vim @@ -1538,6 +1538,16 @@ def Test_expr7_list() CheckDefExecFailure(["var l: list<number> = ['x', 234]"], 'E1012:', 1) CheckDefExecFailure(["var l: list<string> = [234, 'x']"], 'E1012:', 1) CheckDefExecFailure(["var l: list<string> = ['x', 123]"], 'E1012:', 1) + + var lines =<< trim END + vim9script + var datalist: list<string> + def Main() + datalist += ['x'. + enddef + Main() + END + CheckScriptFailure(lines, 'E1127:') enddef def Test_expr7_list_vim9script() diff --git a/src/version.c b/src/version.c index ff1a97a3d..4280ac562 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 */ /**/ + 1789, +/**/ 1788, /**/ 1787, diff --git a/src/vim9compile.c b/src/vim9compile.c index 830e8bb9b..32def6c79 100644 --- a/src/vim9compile.c +++ b/src/vim9compile.c @@ -2549,7 +2549,7 @@ compile_list(char_u **arg, cctx_T *cctx) break; } if (compile_expr0(&p, cctx) == FAIL) - break; + return FAIL; ++count; if (*p == ',') { @@ -3333,7 +3333,10 @@ compile_subscript( *arg = p + 1; if (may_get_next_line(*arg, arg, cctx) == FAIL) + { + emsg(_(e_missing_name_after_dot)); return FAIL; + } // dictionary member: dict.name p = *arg; if (eval_isdictc(*p)) |