From f79225ed4f81bc579bb3360ad2eb06adc8058153 Mon Sep 17 00:00:00 2001 From: Bram Moolenaar Date: Sat, 18 Mar 2017 23:11:04 +0100 Subject: 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". --- src/testdir/test_window_cmd.vim | 12 ++++++++++++ src/version.c | 2 ++ src/window.c | 4 ++++ 3 files changed, 18 insertions(+) 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); -- cgit v1.2.1