aniani/vim
0
0
Fork 1
mirror of https://github.com/vim/vim.git synced 2025-09-27 04:14:06 -04:00
Code Activity

patch 8.1.0112: no error when using bad arguments with searchpair()

Browse Source 
Problem:    No error when using bad arguments with searchpair().
Solution:   Add error messages.
This commit is contained in:
Bram Moolenaar
2018-06-24 19:01:59 +02:00
parent d1d037e901
commit 3dddb09c98
3 changed files with 23 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
src/evalfunc.c
View File

@@ -10152,7 +10152,8 @@ searchpair_cmn(typval_T *argvars, pos_T *match_pos)
long lnum_stop = 0; long lnum_stop = 0;
long time_limit = 0; long time_limit = 0;
/* Get the three pattern arguments: start, middle, end. */ /* Get the three pattern arguments: start, middle, end. Will result in an
* error if not a valid argument. */
spat = get_tv_string_chk(&argvars[0]); spat = get_tv_string_chk(&argvars[0]);
mpat = get_tv_string_buf_chk(&argvars[1], nbuf1); mpat = get_tv_string_buf_chk(&argvars[1], nbuf1);
epat = get_tv_string_buf_chk(&argvars[2], nbuf2); epat = get_tv_string_buf_chk(&argvars[2], nbuf2);
@@ -10189,20 +10190,27 @@ searchpair_cmn(typval_T *argvars, pos_T *match_pos)
&& skip->v_type != VAR_STRING) && skip->v_type != VAR_STRING)
{ {
/* Type error */ /* Type error */
EMSG2(_(e_invarg2), get_tv_string(&argvars[4]));
goto theend; goto theend;
} }
if (argvars[5].v_type != VAR_UNKNOWN) if (argvars[5].v_type != VAR_UNKNOWN)
{ {
lnum_stop = (long)get_tv_number_chk(&argvars[5], NULL); lnum_stop = (long)get_tv_number_chk(&argvars[5], NULL);
if (lnum_stop < 0) if (lnum_stop < 0)
{
EMSG2(_(e_invarg2), get_tv_string(&argvars[5]));
goto theend; goto theend;
}
#ifdef FEAT_RELTIME #ifdef FEAT_RELTIME
if (argvars[6].v_type != VAR_UNKNOWN) if (argvars[6].v_type != VAR_UNKNOWN)
{ {
time_limit = (long)get_tv_number_chk(&argvars[6], NULL); time_limit = (long)get_tv_number_chk(&argvars[6], NULL);
if (time_limit < 0) if (time_limit < 0)
{
EMSG2(_(e_invarg2), get_tv_string(&argvars[6]));
goto theend; goto theend;
} }
}
#endif #endif
} }
} }

12
src/testdir/test_search.vim
View File

@@ -297,6 +297,16 @@ func Test_searchpair()
q! q!
endfunc endfunc
func Test_searchpair_errors()
call assert_fails("call searchpair([0], 'middle', 'end', 'bW', 'skip', 99, 100)", 'E730: using List as a String')
call assert_fails("call searchpair('start', {-> 0}, 'end', 'bW', 'skip', 99, 100)", 'E729: using Funcref as a String')
call assert_fails("call searchpair('start', 'middle', {'one': 1}, 'bW', 'skip', 99, 100)", 'E731: using Dictionary as a String')
call assert_fails("call searchpair('start', 'middle', 'end', 'flags', 'skip', 99, 100)", 'E475: Invalid argument: flags')
call assert_fails("call searchpair('start', 'middle', 'end', 'bW', 0, 99, 100)", 'E475: Invalid argument: 0')
call assert_fails("call searchpair('start', 'middle', 'end', 'bW', 'func', -99, 100)", 'E475: Invalid argument: -99')
call assert_fails("call searchpair('start', 'middle', 'end', 'bW', 'func', 99, -100)", 'E475: Invalid argument: -100')
endfunc
func Test_searchpair_skip() func Test_searchpair_skip()
func Zero() func Zero()
return 0 return 0
@@ -311,8 +321,6 @@ func Test_searchpair_skip()
3 | call assert_equal(1, searchpair('{', '', '}', 'bWn', {-> 0})) 3 | call assert_equal(1, searchpair('{', '', '}', 'bWn', {-> 0}))
3 | call assert_equal(1, searchpair('{', '', '}', 'bWn', function('Zero'))) 3 | call assert_equal(1, searchpair('{', '', '}', 'bWn', function('Zero')))
3 | call assert_equal(1, searchpair('{', '', '}', 'bWn', function('Partial', [0]))) 3 | call assert_equal(1, searchpair('{', '', '}', 'bWn', function('Partial', [0])))
" invalid argument
3 | call assert_equal(0, searchpair('{', '', '}', 'bWn', 0))
bw! bw!
endfunc endfunc

2
src/version.c
View File

@@ -789,6 +789,8 @@ static char *(features[]) =
static int included_patches[] = static int included_patches[] =
{ /* Add new patch number below this line */ { /* Add new patch number below this line */
/**/
112,
/**/ /**/
111, 111,
/**/ /**/