Skip to content

Commit

Permalink
patch 9.1.1048: crash after scrolling and pasting in silent Ex mode
Browse files Browse the repository at this point in the history
Problem:  Crash after scrolling and pasting in silent Ex mode.
          (fizz-is-on-the-way)
Solution: Don't move cursor to line 0 when scrolling.
          (zeertzjq)

closes: #16506

Signed-off-by: zeertzjq <[email protected]>
Signed-off-by: Christian Brabandt <[email protected]>
  • Loading branch information
zeertzjq authored and chrisbra committed Jan 22, 2025
1 parent 613d0bc commit df098fe
Show file tree
Hide file tree
Showing 3 changed files with 23 additions and 1 deletion.
5 changes: 4 additions & 1 deletion src/move.c
Original file line number Diff line number Diff line change
Expand Up @@ -3285,8 +3285,11 @@ pagescroll(int dir, long count, int half)
{
// Place cursor at top or bottom of window.
validate_botline();
curwin->w_cursor.lnum = (dir == FORWARD ? curwin->w_topline
linenr_T lnum = (dir == FORWARD ? curwin->w_topline
: curwin->w_botline - 1);
// In silent Ex mode the value of w_botline - 1 may be 0,
// but cursor lnum needs to be at least 1.
curwin->w_cursor.lnum = MAX(lnum, 1);
}
}

Expand Down
17 changes: 17 additions & 0 deletions src/testdir/test_normal.vim
Original file line number Diff line number Diff line change
Expand Up @@ -1342,6 +1342,22 @@ func Test_scroll_in_ex_mode()
call delete('Xdone')
endfunc

func Test_scroll_and_paste_in_ex_mode()
" This used to crash because of moving cursor to line 0.
let lines =<< trim END
v/foo/vi|YY9PYQ
v/bar/vi|YY9PYQ
v/bar/exe line('.') == 1 ? "vi|Y\<C-B>9PYQ" : "vi|YQ"
call writefile(['done'], 'Xdone')
qa!
END
call writefile(lines, 'Xscript', 'D')
call assert_equal(1, RunVim([], [], '-u NONE -i NONE -n -X -Z -e -s -S Xscript'))
call assert_equal(['done'], readfile('Xdone'))

call delete('Xdone')
endfunc

" Test for the 'sidescroll' option
func Test_sidescroll_opt()
new
Expand Down Expand Up @@ -4293,4 +4309,5 @@ func Test_normal_go()

bwipe!
endfunc

" vim: shiftwidth=2 sts=2 expandtab nofoldenable
2 changes: 2 additions & 0 deletions src/version.c
Original file line number Diff line number Diff line change
Expand Up @@ -704,6 +704,8 @@ static char *(features[]) =

static int included_patches[] =
{ /* Add new patch number below this line */
/**/
1048,
/**/
1047,
/**/
Expand Down

0 comments on commit df098fe

Please sign in to comment.