diff options
author | Bram Moolenaar <Bram@vim.org> | 2017-01-29 22:48:45 +0100 |
---|---|---|
committer | Bram Moolenaar <Bram@vim.org> | 2017-01-29 22:48:45 +0100 |
commit | d58f03b1c21b5b0242718f89da53ddd67b1eff6b (patch) | |
tree | 8d26ad06a50967f6e14eb24e2556e38bae7091ae | |
parent | 6fe2eb43d2527cc8a3450456a60639e87f16d32d (diff) |
patch 8.0.0268: may get ml_get error when :luado deletes linesv8.0.0268
Problem: May get ml_get error when :luado deletes lines or switches to
another buffer. (Nikolai Pavlov, issue #1421)
Solution: Check the buffer and line every time.
-rw-r--r-- | src/Makefile | 1 | ||||
-rw-r--r-- | src/if_lua.c | 9 | ||||
-rw-r--r-- | src/testdir/Make_all.mak | 3 | ||||
-rw-r--r-- | src/testdir/test_lua.vim | 22 | ||||
-rw-r--r-- | src/version.c | 2 |
5 files changed, 36 insertions, 1 deletions
diff --git a/src/Makefile b/src/Makefile index 76170ec36b..f26a1bf33c 100644 --- a/src/Makefile +++ b/src/Makefile @@ -2147,6 +2147,7 @@ test_arglist \ test_langmap \ test_largefile \ test_lispwords \ + test_lua \ test_man \ test_mapping \ test_marks \ diff --git a/src/if_lua.c b/src/if_lua.c index b77a3cd5dd..ad8ee1bf0c 100644 --- a/src/if_lua.c +++ b/src/if_lua.c @@ -1716,6 +1716,8 @@ ex_luado(exarg_T *eap) const char *s = (const char *) eap->arg; luaL_Buffer b; size_t len; + buf_T *was_curbuf = curbuf; + if (lua_init() == FAIL) return; if (u_save(eap->line1 - 1, eap->line2 + 1) == FAIL) { @@ -1739,6 +1741,10 @@ ex_luado(exarg_T *eap) lua_replace(L, -2); /* function -> body */ for (l = eap->line1; l <= eap->line2; l++) { + /* Check the line number, the command my have deleted lines. */ + if (l > curbuf->b_ml.ml_line_count) + break; + lua_pushvalue(L, -1); /* function */ luaV_pushline(L, curbuf, l); /* current line as arg */ lua_pushinteger(L, l); /* current line number as arg */ @@ -1747,6 +1753,9 @@ ex_luado(exarg_T *eap) luaV_emsg(L); break; } + /* Catch the command switching to another buffer. */ + if (curbuf != was_curbuf) + break; if (lua_isstring(L, -1)) /* update line? */ { #ifdef HAVE_SANDBOX diff --git a/src/testdir/Make_all.mak b/src/testdir/Make_all.mak index c01634cc0c..bb646ee1b0 100644 --- a/src/testdir/Make_all.mak +++ b/src/testdir/Make_all.mak @@ -164,6 +164,7 @@ NEW_TESTS = test_arglist.res \ test_job_fails.res \ test_json.res \ test_langmap.res \ + test_lua.res \ test_man.res \ test_marks.res \ test_matchadd_conceal.res \ @@ -172,8 +173,8 @@ NEW_TESTS = test_arglist.res \ test_nested_function.res \ test_netbeans.res \ test_normal.res \ - test_paste.res \ test_packadd.res \ + test_paste.res \ test_perl.res \ test_profile.res \ test_python2.res \ diff --git a/src/testdir/test_lua.vim b/src/testdir/test_lua.vim new file mode 100644 index 0000000000..944ad903fe --- /dev/null +++ b/src/testdir/test_lua.vim @@ -0,0 +1,22 @@ +" Tests for Lua. +" TODO: move tests from test85.in here. + +if !has('lua') + finish +endif + +func Test_luado() + new + call setline(1, ['one', 'two', 'three']) + luado vim.command("%d_") + bwipe! + + " Check switching to another buffer does not trigger ml_get error. + new + let wincount = winnr('$') + call setline(1, ['one', 'two', 'three']) + luado vim.command("new") + call assert_equal(wincount + 1, winnr('$')) + bwipe! + bwipe! +endfunc diff --git a/src/version.c b/src/version.c index 0b12b2a0b3..9788d6527b 100644 --- a/src/version.c +++ b/src/version.c @@ -765,6 +765,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 268, +/**/ 267, /**/ 266, |