Patch 8.2.0135

10 views
Skip to first unread message

Bram Moolenaar

unread,
Jan 20, 2020, 3:13:02 PM1/20/20
to vim...@googlegroups.com

Patch 8.2.0135 (after 8.2.0133)
Problem: Bracketed paste can still cause invalid memory access. (Dominique
Pelle)
Solution: Check for NULL pointer.
Files: src/edit.c, src/testdir/test_search.vim


*** ../vim-8.2.0134/src/edit.c 2020-01-20 20:22:27.290330424 +0100
--- src/edit.c 2020-01-20 21:09:08.247912112 +0100
***************
*** 4942,4948 ****
int save_paste = p_paste;

// If the end code is too long we can't detect it, read everything.
! if (STRLEN(end) >= NUMBUFLEN)
end = NULL;
++no_mapping;
allow_keys = 0;
--- 4942,4948 ----
int save_paste = p_paste;

// If the end code is too long we can't detect it, read everything.
! if (end != NULL && STRLEN(end) >= NUMBUFLEN)
end = NULL;
++no_mapping;
allow_keys = 0;
*** ../vim-8.2.0134/src/testdir/test_search.vim 2020-01-20 20:22:27.290330424 +0100
--- src/testdir/test_search.vim 2020-01-20 21:08:36.160030970 +0100
***************
*** 1451,1456 ****
endfunc

func Test_search_special()
! " this was causing illegal memory access
exe "norm /\x80PS"
endfunc
--- 1451,1457 ----
endfunc

func Test_search_special()
! " this was causing illegal memory access and an endless loop
! set t_PE=
exe "norm /\x80PS"
endfunc
*** ../vim-8.2.0134/src/version.c 2020-01-20 20:41:38.857849537 +0100
--- src/version.c 2020-01-20 21:10:21.531638852 +0100
***************
*** 744,745 ****
--- 744,747 ----
{ /* Add new patch number below this line */
+ /**/
+ 135,
/**/

--
ARTHUR: Be quiet! I order you to shut up.
OLD WOMAN: Order, eh -- who does he think he is?
ARTHUR: I am your king!
OLD WOMAN: Well, I didn't vote for you.
"Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD

/// Bram Moolenaar -- Br...@Moolenaar.net -- http://www.Moolenaar.net \\\
/// sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\ an exciting new programming language -- http://www.Zimbu.org ///
\\\ help me help AIDS victims -- http://ICCF-Holland.org ///
Reply all
Reply to author
Forward
0 new messages