From cab2767874dca019083aa371a1f3b45385dbd037 Mon Sep 17 00:00:00 2001 From: Bram Moolenaar Date: Thu, 9 Apr 2020 20:10:55 +0200 Subject: [PATCH] patch 8.2.0537: Vim9: no check for sandbox when setting v:var Problem: Vim9: no check for sandbox when setting v:var. Solution: Check for sandbox. --- src/evalvars.c | 11 +++++++++++ src/testdir/test_vim9_script.vim | 8 ++++++++ src/version.c | 2 ++ 3 files changed, 21 insertions(+) diff --git a/src/evalvars.c b/src/evalvars.c index 8af80ec0a..d32b76a87 100644 --- a/src/evalvars.c +++ b/src/evalvars.c @@ -1978,6 +1978,17 @@ set_vim_var_tv(int idx, typval_T *tv) clear_tv(tv); return FAIL; } + // VV_RO is also checked when compiling, but let's check here as well. + if (vimvars[idx].vv_flags & VV_RO) + { + semsg(_(e_readonlyvar), vimvars[idx].vv_name); + return FAIL; + } + if (sandbox && (vimvars[idx].vv_flags & VV_RO_SBX)) + { + semsg(_(e_readonlysbx), vimvars[idx].vv_name); + return FAIL; + } clear_tv(&vimvars[idx].vv_di.di_tv); vimvars[idx].vv_di.di_tv = *tv; return OK; diff --git a/src/testdir/test_vim9_script.vim b/src/testdir/test_vim9_script.vim index cab9f8630..4e0dc1b2c 100644 --- a/src/testdir/test_vim9_script.vim +++ b/src/testdir/test_vim9_script.vim @@ -170,6 +170,11 @@ def Test_assignment_default() assert_equal(5678, nr) enddef +def Mess(): string + v:foldstart = 123 + return 'xxx' +enddef + func Test_assignment_failure() call CheckDefFailure(['let var=234'], 'E1004:') call CheckDefFailure(['let var =234'], 'E1004:') @@ -207,6 +212,9 @@ func Test_assignment_failure() call CheckDefFailure(['let var: dict '], 'E1068:') call CheckDefFailure(['let var: dict