summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorBram Moolenaar <Bram@vim.org>2022-08-14 21:28:32 +0100
committerBram Moolenaar <Bram@vim.org>2022-08-14 21:28:32 +0100
commitd1d8f6bacb489036d0fd479c9dd3c0102c988889 (patch)
tree4edf759dfb4d79facb2e07869bf2006792ee3b9f
parentc3a483fc3c65f649f9985bb88792a465ea18b0a2 (diff)
patch 9.0.0211: invalid memory access when compiling :lockvarv9.0.0211
Problem: Invalid memory access when compiling :lockvar. Solution: Don't read past the end of the line.
-rw-r--r--src/testdir/test_vim9_cmd.vim9
-rw-r--r--src/version.c2
-rw-r--r--src/vim9cmds.c9
3 files changed, 18 insertions, 2 deletions
diff --git a/src/testdir/test_vim9_cmd.vim b/src/testdir/test_vim9_cmd.vim
index fc1354e83d..79f19f0389 100644
--- a/src/testdir/test_vim9_cmd.vim
+++ b/src/testdir/test_vim9_cmd.vim
@@ -1737,6 +1737,15 @@ def Test_lockvar()
UnLockIt()
END
v9.CheckScriptFailure(lines, 'E46', 1)
+
+ lines =<< trim END
+ def _()
+ s:0([], s:0)
+ lockv
+ enddef
+ defcomp
+ END
+ v9.CheckScriptFailure(lines, 'E179', 2)
enddef
def Test_substitute_expr()
diff --git a/src/version.c b/src/version.c
index c58386b563..3f0bf5151a 100644
--- a/src/version.c
+++ b/src/version.c
@@ -736,6 +736,8 @@ static char *(features[]) =
static int included_patches[] =
{ /* Add new patch number below this line */
/**/
+ 211,
+/**/
210,
/**/
209,
diff --git a/src/vim9cmds.c b/src/vim9cmds.c
index ad32c32ff7..35a382138b 100644
--- a/src/vim9cmds.c
+++ b/src/vim9cmds.c
@@ -188,10 +188,17 @@ compile_lock_unlock(
size_t len;
char_u *buf;
isntype_T isn = ISN_EXEC;
+ char *cmd = eap->cmdidx == CMD_lockvar ? "lockvar" : "unlockvar";
if (cctx->ctx_skip == SKIP_YES)
return OK;
+ if (*p == NUL)
+ {
+ semsg(_(e_argument_required_for_str), cmd);
+ return FAIL;
+ }
+
// Cannot use :lockvar and :unlockvar on local variables.
if (p[1] != ':')
{
@@ -223,8 +230,6 @@ compile_lock_unlock(
ret = FAIL;
else
{
- char *cmd = eap->cmdidx == CMD_lockvar ? "lockvar" : "unlockvar";
-
if (deep < 0)
vim_snprintf((char *)buf, len, "%s! %s", cmd, p);
else