changeset 31875:244d4a84c546 v9.0.1270

patch 9.0.1270: crash when using search stat in narrow screen Commit: https://github.com/vim/vim/commit/a7d36b732070944aab614944075ec0b409311482 Author: zeertzjq <zeertzjq@outlook.com> Date: Tue Jan 31 21:13:38 2023 +0000 patch 9.0.1270: crash when using search stat in narrow screen Problem: Crash when using search stat in narrow screen. Solution: Check length of message. (closes https://github.com/vim/vim/issues/11921)
author Bram Moolenaar <Bram@vim.org>
date Tue, 31 Jan 2023 22:15:04 +0100
parents 7325519b56fc
children b27dae7424a9
files src/search.c src/testdir/test_search_stat.vim src/version.c
diffstat 3 files changed, 30 insertions(+), 1 deletions(-) [+]
line wrap: on
line diff
--- a/src/search.c
+++ b/src/search.c
@@ -3154,7 +3154,11 @@ cmdline_search_stat(
 	len += 2;
     }
 
-    mch_memmove(msgbuf + STRLEN(msgbuf) - len, t, len);
+    size_t msgbuf_len = STRLEN(msgbuf);
+    if (len > msgbuf_len)
+	len = msgbuf_len;
+    mch_memmove(msgbuf + msgbuf_len - len, t, len);
+
     if (dirc == '?' && stat.cur == maxcount + 1)
 	stat.cur = -1;
 
--- a/src/testdir/test_search_stat.vim
+++ b/src/testdir/test_search_stat.vim
@@ -270,6 +270,29 @@ func Test_searchcount_fails()
   call assert_fails('echo searchcount({"pos" : [1, 2, []]})', 'E745:')
 endfunc
 
+func Test_search_stat_narrow_screen()
+  " This used to crash Vim
+  let save_columns = &columns
+  try
+    let after =<< trim [CODE]
+      set laststatus=2
+      set columns=16
+      set shortmess-=S showcmd
+      call setline(1, 'abc')
+      call feedkeys("/abc\<CR>:quit!\<CR>")
+      autocmd VimLeavePre * call writefile(["done"], "Xdone")
+    [CODE]
+
+    if !RunVim([], after, '--clean')
+      return
+    endif
+    call assert_equal("done", readfile("Xdone")[0])
+    call delete('Xdone')
+  finally
+    let &columns = save_columns
+  endtry
+endfunc
+
 func Test_searchcount_in_statusline()
   CheckScreendump
 
--- a/src/version.c
+++ b/src/version.c
@@ -696,6 +696,8 @@ static char *(features[]) =
 static int included_patches[] =
 {   /* Add new patch number below this line */
 /**/
+    1270,
+/**/
     1269,
 /**/
     1268,