mirrors/neovim
1
0
Fork 0
mirror of https://github.com/neovim/neovim.git synced 2025-09-08 12:28:18 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #18942 from neovim/backport-18933-to-release-0.7

Browse Source 
[Backport release-0.7] fix(buffer): disable buffer-updates before removing buffer from window
This commit is contained in:
zeertzjq
2022-06-13 07:44:20 +08:00
committed by GitHub
parent 1dba6cf8f0 05f6883ee2
commit 3cea4d6de9
3 changed files with 13 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
src/nvim/buffer.c
View File

@@ -578,6 +578,10 @@ bool close_buffer(win_T *win, buf_T *buf, int action, bool abort_if_last, bool i
return false; return false;
} }
// Disable buffer-updates for the current buffer.
// No need to check `unload_buf`: in that case the function returned above.
buf_updates_unload(buf, false);
if (win != NULL // Avoid bogus clang warning. if (win != NULL // Avoid bogus clang warning.
&& win_valid_any_tab(win) && win_valid_any_tab(win)
&& win->w_buffer == buf) { && win->w_buffer == buf) {
@@ -590,10 +594,6 @@ bool close_buffer(win_T *win, buf_T *buf, int action, bool abort_if_last, bool i
buf->b_nwindows--; buf->b_nwindows--;
} }
// Disable buffer-updates for the current buffer.
// No need to check `unload_buf`: in that case the function returned above.
buf_updates_unload(buf, false);
// Remove the buffer from the list. // Remove the buffer from the list.
if (wipe_buf) { if (wipe_buf) {
// Do not wipe out the buffer if it is used in a window. // Do not wipe out the buffer if it is used in a window.

2
src/nvim/window.c
View File

@@ -7207,7 +7207,7 @@ void win_findbuf(typval_T *argvars, list_T *list)
int bufnr = tv_get_number(&argvars[0]); int bufnr = tv_get_number(&argvars[0]);
FOR_ALL_TAB_WINDOWS(tp, wp) { FOR_ALL_TAB_WINDOWS(tp, wp) {
if (!wp->w_closing && wp->w_buffer->b_fnum == bufnr) { if (wp->w_buffer->b_fnum == bufnr) {
tv_list_append_number(list, wp->handle); tv_list_append_number(list, wp->handle);
} }
} }

11
test/functional/lua/buffer_updates_spec.lua
View File

@@ -252,9 +252,8 @@ describe('lua buffer event callbacks: on_lines', function()
eq(2, meths.win_get_cursor(0)[1]) eq(2, meths.win_get_cursor(0)[1])
end) end)
it('does not SEGFAULT when calling win_findbuf in on_detach', function() it('does not SEGFAULT when accessing window buffer info in on_detach #14998', function()
local code = [[
exec_lua[[
local buf = vim.api.nvim_create_buf(false, false) local buf = vim.api.nvim_create_buf(false, false)
vim.cmd"split" vim.cmd"split"
@@ -262,13 +261,19 @@ describe('lua buffer event callbacks: on_lines', function()
vim.api.nvim_buf_attach(buf, false, { vim.api.nvim_buf_attach(buf, false, {
on_detach = function(_, buf) on_detach = function(_, buf)
vim.fn.tabpagebuflist()
vim.fn.win_findbuf(buf) vim.fn.win_findbuf(buf)
end end
}) })
]] ]]
exec_lua(code)
command("q!") command("q!")
helpers.assert_alive() helpers.assert_alive()
exec_lua(code)
command("bd!")
helpers.assert_alive()
end) end)
it('#12718 lnume', function() it('#12718 lnume', function()