mirror of
https://github.com/vim/vim.git
synced 2025-07-26 11:04:33 -04:00
patch 8.2.0817: not enough memory allocated when converting string
Problem: Not enough memory allocated when converting string with special character. Solution: Reserve space for modifier code. (closes #6130)
This commit is contained in:
parent
87202264f8
commit
f7271e8316
11
src/eval.c
11
src/eval.c
@ -3503,6 +3503,7 @@ get_string_tv(char_u **arg, typval_T *rettv, int evaluate)
|
|||||||
char_u *p;
|
char_u *p;
|
||||||
char_u *name;
|
char_u *name;
|
||||||
int extra = 0;
|
int extra = 0;
|
||||||
|
int len;
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* Find the end of the string, skipping backslashed characters.
|
* Find the end of the string, skipping backslashed characters.
|
||||||
@ -3513,9 +3514,10 @@ get_string_tv(char_u **arg, typval_T *rettv, int evaluate)
|
|||||||
{
|
{
|
||||||
++p;
|
++p;
|
||||||
// A "\<x>" form occupies at least 4 characters, and produces up
|
// A "\<x>" form occupies at least 4 characters, and produces up
|
||||||
// to 6 characters: reserve space for 2 extra
|
// to 9 characters (6 for the char and 3 for a modifier): reserve
|
||||||
|
// space for 5 extra.
|
||||||
if (*p == '<')
|
if (*p == '<')
|
||||||
extra += 2;
|
extra += 5;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -3536,7 +3538,8 @@ get_string_tv(char_u **arg, typval_T *rettv, int evaluate)
|
|||||||
* Copy the string into allocated memory, handling backslashed
|
* Copy the string into allocated memory, handling backslashed
|
||||||
* characters.
|
* characters.
|
||||||
*/
|
*/
|
||||||
name = alloc(p - *arg + extra);
|
len = (int)(p - *arg + extra);
|
||||||
|
name = alloc(len);
|
||||||
if (name == NULL)
|
if (name == NULL)
|
||||||
return FAIL;
|
return FAIL;
|
||||||
rettv->v_type = VAR_STRING;
|
rettv->v_type = VAR_STRING;
|
||||||
@ -3610,6 +3613,8 @@ get_string_tv(char_u **arg, typval_T *rettv, int evaluate)
|
|||||||
if (extra != 0)
|
if (extra != 0)
|
||||||
{
|
{
|
||||||
name += extra;
|
name += extra;
|
||||||
|
if (name >= rettv->vval.v_string + len)
|
||||||
|
iemsg("get_string_tv() used more space than allocated");
|
||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
// FALLTHROUGH
|
// FALLTHROUGH
|
||||||
|
@ -2278,6 +2278,8 @@ func Test_nr2char()
|
|||||||
set encoding=utf8
|
set encoding=utf8
|
||||||
call assert_equal('a', nr2char(97, 1))
|
call assert_equal('a', nr2char(97, 1))
|
||||||
call assert_equal('a', nr2char(97, 0))
|
call assert_equal('a', nr2char(97, 0))
|
||||||
|
|
||||||
|
call assert_equal("\x80\xfc\b\xf4\x80\xfeX\x80\xfeX\x80\xfeX", eval('"\<M-' .. nr2char(0x100000) .. '>"'))
|
||||||
endfunc
|
endfunc
|
||||||
|
|
||||||
" Test for screenattr(), screenchar() and screenchars() functions
|
" Test for screenattr(), screenchar() and screenchars() functions
|
||||||
|
@ -746,6 +746,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 */
|
||||||
|
/**/
|
||||||
|
817,
|
||||||
/**/
|
/**/
|
||||||
816,
|
816,
|
||||||
/**/
|
/**/
|
||||||
|
Loading…
x
Reference in New Issue
Block a user