]> granicus.if.org Git - vim/commitdiff
patch 8.2.4281: using freed memory with :lopen and :bwipe v8.2.4281
authorBram Moolenaar <Bram@vim.org>
Tue, 1 Feb 2022 13:54:17 +0000 (13:54 +0000)
committerBram Moolenaar <Bram@vim.org>
Tue, 1 Feb 2022 13:54:17 +0000 (13:54 +0000)
Problem:    Using freed memory with :lopen and :bwipe.
Solution:   Do not use a wiped out buffer.

src/buffer.c
src/testdir/test_quickfix.vim
src/version.c

index 24da8298478ff129e6c4ffccbc4a6f4abb4cec3b..81bdb31ca1f05b33f0f38ed3d2abf5c611c28497 100644 (file)
@@ -1706,6 +1706,7 @@ set_curbuf(buf_T *buf, int action)
 #endif
     bufref_T   newbufref;
     bufref_T   prevbufref;
+    int                valid;
 
     setpcmark();
     if ((cmdmod.cmod_flags & CMOD_KEEPALT) == 0)
@@ -1763,13 +1764,19 @@ set_curbuf(buf_T *buf, int action)
     // An autocommand may have deleted "buf", already entered it (e.g., when
     // it did ":bunload") or aborted the script processing.
     // If curwin->w_buffer is null, enter_buffer() will make it valid again
-    if ((buf_valid(buf) && buf != curbuf
+    valid = buf_valid(buf);
+    if ((valid && buf != curbuf
 #ifdef FEAT_EVAL
                && !aborting()
 #endif
        ) || curwin->w_buffer == NULL)
     {
-       enter_buffer(buf);
+       // If the buffer is not valid but curwin->w_buffer is NULL we must
+       // enter some buffer.  Using the last one is hopefully OK.
+       if (!valid)
+           enter_buffer(lastbuf);
+       else
+           enter_buffer(buf);
 #ifdef FEAT_SYN_HL
        if (old_tw != curbuf->b_p_tw)
            check_colorcolumn(curwin);
@@ -2288,8 +2295,7 @@ free_buf_options(
     clear_string_option(&buf->b_p_vsts);
     vim_free(buf->b_p_vsts_nopaste);
     buf->b_p_vsts_nopaste = NULL;
-    vim_free(buf->b_p_vsts_array);
-    buf->b_p_vsts_array = NULL;
+    VIM_CLEAR(buf->b_p_vsts_array);
     clear_string_option(&buf->b_p_vts);
     VIM_CLEAR(buf->b_p_vts_array);
 #endif
index fb6d21fc5cbd037f634337204d98c1f4890e8ad7..07fdb9644b505570193391ccf69e60bef1450bda 100644 (file)
@@ -979,6 +979,7 @@ func Test_locationlist_curwin_was_closed()
   call assert_fails('lrewind', 'E924:')
 
   augroup! testgroup
+  delfunc R
 endfunc
 
 func Test_locationlist_cross_tab_jump()
@@ -5835,4 +5836,20 @@ func Test_two_qf_windows()
   %bw!
 endfunc
 
+" Weird sequence of commands that caused entering a wiped-out buffer
+func Test_lopen_bwipe()
+  func R()
+    silent! tab lopen
+    e x
+    silent! lfile
+  endfunc
+
+  cal R()
+  cal R()
+  cal R()
+  bw!
+  delfunc R
+endfunc
+
+
 " vim: shiftwidth=2 sts=2 expandtab
index 978e869ec08f863e48643670c14f4ef97debf806..d8b3d4b7f50e17823191dfba27aca30c04ebd422 100644 (file)
@@ -746,6 +746,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    4281,
 /**/
     4280,
 /**/