Mercurial > vim
diff src/misc2.c @ 28893:aa44d5842d6c v8.2.4969
patch 8.2.4969: changing text in Visual mode may cause invalid memory access
Commit: https://github.com/vim/vim/commit/7ce5b2b590256ce53d6af28c1d203fb3bc1d2d97
Author: Bram Moolenaar <Bram@vim.org>
Date: Mon May 16 19:40:59 2022 +0100
patch 8.2.4969: changing text in Visual mode may cause invalid memory access
Problem: Changing text in Visual mode may cause invalid memory access.
Solution: Check the Visual position after making a change.
| author | Bram Moolenaar <Bram@vim.org> |
|---|---|
| date | Mon, 16 May 2022 20:45:03 +0200 |
| parents | d0241e74bfdb |
| children | 057c26b5c33a |
line wrap: on
line diff
--- a/src/misc2.c +++ b/src/misc2.c @@ -622,6 +622,31 @@ check_cursor(void) check_cursor_col(); } +/* + * Check if VIsual position is valid, correct it if not. + * Can be called when in Visual mode and a change has been made. + */ + void +check_visual_pos(void) +{ + if (VIsual.lnum > curbuf->b_ml.ml_line_count) + { + VIsual.lnum = curbuf->b_ml.ml_line_count; + VIsual.col = 0; + VIsual.coladd = 0; + } + else + { + int len = (int)STRLEN(ml_get(VIsual.lnum)); + + if (VIsual.col > len) + { + VIsual.col = len; + VIsual.coladd = 0; + } + } +} + #if defined(FEAT_TEXTOBJ) || defined(PROTO) /* * Make sure curwin->w_cursor is not on the NUL at the end of the line. @@ -2416,7 +2441,7 @@ get_user_name(char_u *buf, int len) return OK; } -#if defined(EXITFREE) || defined(PROTOS) +#if defined(EXITFREE) || defined(PROTO) /* * Free the memory allocated by get_user_name() */