diff options
author | Bram Moolenaar <Bram@vim.org> | 2021-10-04 23:13:13 +0100 |
---|---|---|
committer | Bram Moolenaar <Bram@vim.org> | 2021-10-04 23:13:13 +0100 |
commit | d3710cf01ef6ab1b2f233866ff01dab76686f642 (patch) | |
tree | 28e2058098833e3f613c0ecf75733ee4e0cc1627 | |
parent | 08d7b1c82866a61b61a55e55b6c190dba04e54ea (diff) |
patch 8.2.3476: renaming a buffer on startup may cause using freed memoryv8.2.3476
Problem: Renaming a buffer on startup may cause using freed memory.
Solution: Check if the buffer is used in a window. (closes #8955)
-rw-r--r-- | src/buffer.c | 12 | ||||
-rw-r--r-- | src/testdir/test_startup.vim | 14 | ||||
-rw-r--r-- | src/version.c | 2 |
3 files changed, 27 insertions, 1 deletions
diff --git a/src/buffer.c b/src/buffer.c index 5616487da9..bcbdf839dd 100644 --- a/src/buffer.c +++ b/src/buffer.c @@ -3399,7 +3399,17 @@ setfname( #endif if (obuf != NULL && obuf != buf) { - if (obuf->b_ml.ml_mfp != NULL) // it's loaded, fail + win_T *win; + tabpage_T *tab; + int in_use = FALSE; + + // during startup a window may use a buffer that is not loaded yet + FOR_ALL_TAB_WINDOWS(tab, win) + if (win->w_buffer == obuf) + in_use = TRUE; + + // it's loaded or used in a window, fail + if (obuf->b_ml.ml_mfp != NULL || in_use) { if (message) emsg(_("E95: Buffer with this name already exists")); diff --git a/src/testdir/test_startup.vim b/src/testdir/test_startup.vim index 6b64adf748..897492deb9 100644 --- a/src/testdir/test_startup.vim +++ b/src/testdir/test_startup.vim @@ -1296,4 +1296,18 @@ func Test_echo_true_in_cmd() call delete('Xresult') endfunc +func Test_rename_buffer_on_startup() + let lines =<< trim END + call writefile(['done'], 'Xresult') + qa! + END + call writefile(lines, 'Xscript') + if RunVim([], [], "--clean -e -s --cmd 'file x|new|file x' --cmd 'so Xscript'") + call assert_equal(['done'], readfile('Xresult')) + endif + call delete('Xscript') + call delete('Xresult') +endfunc + + " vim: shiftwidth=2 sts=2 expandtab diff --git a/src/version.c b/src/version.c index e6a4b1ad5d..95200c6660 100644 --- a/src/version.c +++ b/src/version.c @@ -758,6 +758,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 3476, +/**/ 3475, /**/ 3474, |