From 615ddd5342b50a6878a907062aa471740bd9a847 Mon Sep 17 00:00:00 2001 From: Bram Moolenaar Date: Wed, 17 Nov 2021 18:00:31 +0000 Subject: [PATCH] patch 8.2.3611: crash when using CTRL-W f without finding a file name Problem: Crash when using CTRL-W f without finding a file name. Solution: Bail out when the file name length is zero. --- src/findfile.c | 8 ++++++++ src/normal.c | 6 ++++-- src/testdir/test_visual.vim | 8 ++++++++ src/version.c | 2 ++ 4 files changed, 22 insertions(+), 2 deletions(-) diff --git a/src/findfile.c b/src/findfile.c index b79b1d1dbf4a2..b4aed048ce3c7 100644 --- a/src/findfile.c +++ b/src/findfile.c @@ -1735,6 +1735,9 @@ find_file_in_path_option( proc->pr_WindowPtr = (APTR)-1L; # endif + if (len == 0) + return NULL; + if (first == TRUE) { // copy file name into NameBuff, expanding environment variables @@ -2118,7 +2121,12 @@ find_file_name_in_path( int c; # if defined(FEAT_FIND_ID) && defined(FEAT_EVAL) char_u *tofree = NULL; +# endif + if (len == 0) + return NULL; + +# if defined(FEAT_FIND_ID) && defined(FEAT_EVAL) if ((options & FNAME_INCL) && *curbuf->b_p_inex != NUL) { tofree = eval_includeexpr(ptr, len); diff --git a/src/normal.c b/src/normal.c index 059baee90faab..aab336a5fe905 100644 --- a/src/normal.c +++ b/src/normal.c @@ -3881,8 +3881,10 @@ get_visual_text( *pp = ml_get_pos(&VIsual); *lenp = curwin->w_cursor.col - VIsual.col + 1; } - if (has_mbyte) - // Correct the length to include the whole last character. + if (**pp == NUL) + *lenp = 0; + if (has_mbyte && *lenp > 0) + // Correct the length to include all bytes of the last character. *lenp += (*mb_ptr2len)(*pp + (*lenp - 1)) - 1; } reset_VIsual_and_resel(); diff --git a/src/testdir/test_visual.vim b/src/testdir/test_visual.vim index 20eb4b43e8e88..0c89dbb0103bc 100644 --- a/src/testdir/test_visual.vim +++ b/src/testdir/test_visual.vim @@ -1265,6 +1265,14 @@ func Test_visual_block_with_virtualedit() call delete('XTest_block') endfunc +func Test_visual_block_ctrl_w_f() + " Emtpy block selected in new buffer should not result in an error. + au! BufNew foo sil norm f + edit foo + + au! BufNew +endfunc + func Test_visual_reselect_with_count() " this was causing an illegal memory access let lines =<< trim END diff --git a/src/version.c b/src/version.c index d301120079e69..4251b03ea5ad3 100644 --- a/src/version.c +++ b/src/version.c @@ -757,6 +757,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ +/**/ + 3611, /**/ 3610, /**/