# HG changeset patch # User Christian Brabandt # Date 1513597505 -3600 # Node ID 197a08152ad57491755442cfb921285126ee7cee # Parent 472e2cba236d72cf7be4afece73132eb58291507 patch 8.0.1402: crash with nasty autocommand commit https://github.com/vim/vim/commit/9bca805ec49eb0d2d0d0b2093f418ff425500169 Author: Bram Moolenaar Date: Mon Dec 18 12:37:55 2017 +0100 patch 8.0.1402: crash with nasty autocommand Problem: Crash with nasty autocommand. (gy741, Dominique Pelle) Solution: Check that the new current buffer isn't wiped out. (closes https://github.com/vim/vim/issues/2447) diff --git a/src/buffer.c b/src/buffer.c --- a/src/buffer.c +++ b/src/buffer.c @@ -1665,7 +1665,8 @@ set_curbuf(buf_T *buf, int action) #ifdef FEAT_SYN_HL long old_tw = curbuf->b_p_tw; #endif - bufref_T bufref; + bufref_T newbufref; + bufref_T prevbufref; setpcmark(); if (!cmdmod.keepalt) @@ -1675,18 +1676,22 @@ set_curbuf(buf_T *buf, int action) /* Don't restart Select mode after switching to another buffer. */ VIsual_reselect = FALSE; - /* close_windows() or apply_autocmds() may change curbuf */ + /* close_windows() or apply_autocmds() may change curbuf and wipe out "buf" + */ prevbuf = curbuf; - set_bufref(&bufref, prevbuf); + set_bufref(&prevbufref, prevbuf); + set_bufref(&newbufref, buf); #ifdef FEAT_AUTOCMD + /* Autocommands may delete the curren buffer and/or the buffer we wan to go + * to. In those cases don't close the buffer. */ if (!apply_autocmds(EVENT_BUFLEAVE, NULL, NULL, FALSE, curbuf) + || (bufref_valid(&prevbufref) + && bufref_valid(&newbufref) # ifdef FEAT_EVAL - || (bufref_valid(&bufref) && !aborting()) -# else - || bufref_valid(&bufref) + && !aborting() # endif - ) + )) #endif { #ifdef FEAT_SYN_HL @@ -1696,9 +1701,9 @@ set_curbuf(buf_T *buf, int action) if (unload) close_windows(prevbuf, FALSE); #if defined(FEAT_AUTOCMD) && defined(FEAT_EVAL) - if (bufref_valid(&bufref) && !aborting()) + if (bufref_valid(&prevbufref) && !aborting()) #else - if (bufref_valid(&bufref)) + if (bufref_valid(&prevbufref)) #endif { win_T *previouswin = curwin; diff --git a/src/testdir/test_autocmd.vim b/src/testdir/test_autocmd.vim --- a/src/testdir/test_autocmd.vim +++ b/src/testdir/test_autocmd.vim @@ -1163,3 +1163,11 @@ func Test_TextYankPost() unlet g:event bwipe! endfunc + +func Test_nocatch_wipe_all_buffers() + " Real nasty autocommand: wipe all buffers on any event. + au * * bwipe * + call assert_fails('next x', 'E93') + bwipe + au! +endfunc diff --git a/src/version.c b/src/version.c --- a/src/version.c +++ b/src/version.c @@ -772,6 +772,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 1402, +/**/ 1401, /**/ 1400,