From: Bram Moolenaar <Bram@vim.org>
Date: Sat, 18 Mar 2017 22:11:04 +0000 (+0100)
Subject: patch 8.0.0483: illegal memory access when using :all
X-Git-Tag: v8.0.0483
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=f79225ed4f81bc579bb3360ad2eb06adc8058153;p=vim

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".
---

diff --git a/src/testdir/test_window_cmd.vim b/src/testdir/test_window_cmd.vim
index 569a78a0e..6c9ddc289 100644
--- 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
diff --git a/src/version.c b/src/version.c
index 828450d17..bb4cadcd9 100644
--- a/src/version.c
+++ b/src/version.c
@@ -764,6 +764,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    483,
 /**/
     482,
 /**/
diff --git a/src/window.c b/src/window.c
index 06f97de27..d0dff69e0 100644
--- 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);