Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
patch 8.2.5037: cursor position may be invalid after "0;" range
Problem:    Cursor position may be invalid after "0;" range.
Solution:   Check the cursor position when it was set by ";" in the range.
  • Loading branch information
brammool committed May 28, 2022
1 parent 305abc6 commit 4d97a56
Show file tree
Hide file tree
Showing 3 changed files with 27 additions and 7 deletions.
24 changes: 17 additions & 7 deletions src/ex_docmd.c
Expand Up @@ -3280,6 +3280,8 @@ parse_cmd_address(exarg_T *eap, char **errormsg, int silent)
{
int address_count = 1;
linenr_T lnum;
int need_check_cursor = FALSE;
int ret = FAIL;

// Repeat for all ',' or ';' separated addresses.
for (;;)
Expand All @@ -3290,7 +3292,7 @@ parse_cmd_address(exarg_T *eap, char **errormsg, int silent)
lnum = get_address(eap, &eap->cmd, eap->addr_type, eap->skip, silent,
eap->addr_count == 0, address_count++);
if (eap->cmd == NULL) // error detected
return FAIL;
goto theend;
if (lnum == MAXLNUM)
{
if (*eap->cmd == '%') // '%' - all lines
Expand Down Expand Up @@ -3335,14 +3337,14 @@ parse_cmd_address(exarg_T *eap, char **errormsg, int silent)
// there is no Vim command which uses '%' and
// ADDR_WINDOWS or ADDR_TABS
*errormsg = _(e_invalid_range);
return FAIL;
goto theend;
}
break;
case ADDR_TABS_RELATIVE:
case ADDR_UNSIGNED:
case ADDR_QUICKFIX:
*errormsg = _(e_invalid_range);
return FAIL;
goto theend;
case ADDR_ARGUMENTS:
if (ARGCOUNT == 0)
eap->line1 = eap->line2 = 0;
Expand Down Expand Up @@ -3374,19 +3376,19 @@ parse_cmd_address(exarg_T *eap, char **errormsg, int silent)
if (eap->addr_type != ADDR_LINES)
{
*errormsg = _(e_invalid_range);
return FAIL;
goto theend;
}

++eap->cmd;
if (!eap->skip)
{
fp = getmark('<', FALSE);
if (check_mark(fp) == FAIL)
return FAIL;
goto theend;
eap->line1 = fp->lnum;
fp = getmark('>', FALSE);
if (check_mark(fp) == FAIL)
return FAIL;
goto theend;
eap->line2 = fp->lnum;
++eap->addr_count;
}
Expand All @@ -3401,10 +3403,13 @@ parse_cmd_address(exarg_T *eap, char **errormsg, int silent)
if (!eap->skip)
{
curwin->w_cursor.lnum = eap->line2;

// Don't leave the cursor on an illegal line or column, but do
// accept zero as address, so 0;/PATTERN/ works correctly.
// Check the cursor position before returning.
if (eap->line2 > 0)
check_cursor();
need_check_cursor = TRUE;
}
}
else if (*eap->cmd != ',')
Expand All @@ -3420,7 +3425,12 @@ parse_cmd_address(exarg_T *eap, char **errormsg, int silent)
if (lnum == MAXLNUM)
eap->addr_count = 0;
}
return OK;
ret = OK;

theend:
if (need_check_cursor)
check_cursor();
return ret;
}

/*
Expand Down
8 changes: 8 additions & 0 deletions src/testdir/test_excmd.vim
Expand Up @@ -717,5 +717,13 @@ func Test_address_line_overflow()
bwipe!
endfunc

" This was leaving the cursor in line zero
func Test_using_zero_in_range()
new
norm o00
silent! 0;s/\%')
bwipe!
endfunc


" vim: shiftwidth=2 sts=2 expandtab
2 changes: 2 additions & 0 deletions src/version.c
Expand Up @@ -734,6 +734,8 @@ static char *(features[]) =

static int included_patches[] =
{ /* Add new patch number below this line */
/**/
5037,
/**/
5036,
/**/
Expand Down

0 comments on commit 4d97a56

Please sign in to comment.