diff options
author | Bram Moolenaar <Bram@vim.org> | 2020-03-20 21:15:51 +0100 |
---|---|---|
committer | Bram Moolenaar <Bram@vim.org> | 2020-03-20 21:15:51 +0100 |
commit | 0f1563ffee4397f5b379517c41b7c9a977fd2e22 (patch) | |
tree | 1a7d2bd202acb3f9f955d9283a6da2c90591da20 | |
parent | 9645e2d9fc8a591f1db30383caccf71d40e0810c (diff) | |
download | vim-git-0f1563ffee4397f5b379517c41b7c9a977fd2e22.tar.gz |
patch 8.2.0422: crash when passing popup window to win_splitmove()v8.2.0422
Problem: Crash when passing popup window to win_splitmove(). (john Devin)
Solution: Disallow moving a popup window. (closes #5816)
-rw-r--r-- | src/evalwindow.c | 3 | ||||
-rw-r--r-- | src/testdir/test_popupwin.vim | 12 | ||||
-rw-r--r-- | src/version.c | 2 |
3 files changed, 16 insertions, 1 deletions
diff --git a/src/evalwindow.c b/src/evalwindow.c index c7a9f7960..534a047e6 100644 --- a/src/evalwindow.c +++ b/src/evalwindow.c @@ -810,7 +810,8 @@ f_win_splitmove(typval_T *argvars, typval_T *rettv) targetwin = find_win_by_nr_or_id(&argvars[1]); if (wp == NULL || targetwin == NULL || wp == targetwin - || !win_valid(wp) || !win_valid(targetwin)) + || !win_valid(wp) || !win_valid(targetwin) + || win_valid_popup(wp) || win_valid_popup(targetwin)) { emsg(_(e_invalwindow)); rettv->vval.v_number = -1; diff --git a/src/testdir/test_popupwin.vim b/src/testdir/test_popupwin.vim index 9cd6e3229..478e9d49e 100644 --- a/src/testdir/test_popupwin.vim +++ b/src/testdir/test_popupwin.vim @@ -3307,4 +3307,16 @@ func Test_popupwin_atcursor_far_right() set signcolumn& endfunc +func Test_popupwin_splitmove() + vsplit + let win2 = win_getid() + let popup_winid = popup_dialog('hello', {}) + call assert_fails('call win_splitmove(popup_winid, win2)', 'E957:') + call assert_fails('call win_splitmove(win2, popup_winid)', 'E957:') + + call popup_clear() + bwipe +endfunc + + " vim: shiftwidth=2 sts=2 diff --git a/src/version.c b/src/version.c index 986a29feb..a6112cee9 100644 --- a/src/version.c +++ b/src/version.c @@ -739,6 +739,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 422, +/**/ 421, /**/ 420, |