changeset 11193:75ccc8a15a51 v8.0.0483

patch 8.0.0483: illegal memory access when using :all commit https://github.com/vim/vim/commit/f79225ed4f81bc579bb3360ad2eb06adc8058153 Author: Bram Moolenaar <Bram@vim.org> Date: Sat Mar 18 23:11:04 2017 +0100 patch 8.0.0483: illegal memory access when using :all Problem: Illegal memory access when using :all. (Dominique Pelle) Solution: Adjust the cursor position right after setting "curwin".
author Christian Brabandt <cb@256bit.org>
date Sat, 18 Mar 2017 23:15:04 +0100
parents 988b6e9fd9d4
children a6434de9107c
files src/testdir/test_window_cmd.vim src/version.c src/window.c
diffstat 3 files changed, 18 insertions(+), 0 deletions(-) [+]
line wrap: on
line diff
--- a/src/testdir/test_window_cmd.vim
+++ b/src/testdir/test_window_cmd.vim
@@ -67,4 +67,16 @@ function Test_window_cmd_wincmd_gf()
   augroup! test_window_cmd_wincmd_gf
 endfunc
 
+func Test_next_split_all()
+  " This was causing an illegal memory access.
+  n x
+  norm axxx
+  split
+  split
+  s/x
+  s/x
+  all
+  bwipe!
+endfunc
+
 " vim: shiftwidth=2 sts=2 expandtab
--- 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 */
 /**/
+    483,
+/**/
     482,
 /**/
     481,
--- a/src/window.c
+++ b/src/window.c
@@ -2450,6 +2450,10 @@ win_close(win_T *win, int free_buf)
 #endif
 	curbuf = curwin->w_buffer;
 	close_curwin = TRUE;
+
+	/* The cursor position may be invalid if the buffer changed after last
+	 * using the window. */
+	check_cursor();
     }
     if (p_ea && (*p_ead == 'b' || *p_ead == dir))
 	win_equal(curwin, TRUE, dir);