mirror of
https://github.com/vim/vim.git
synced 2025-09-23 03:43:49 -04:00
patch 8.2.3244: Lua 5.3 print() with a long string crashes
Problem: Lua 5.3 print() with a long string crashes. Solution: Use a growarray instead of a Lua buffer. (Yegappan Lakshmanan, closes #8655)
This commit is contained in:
committed by
Bram Moolenaar
parent
83cd0156e0
commit
41114a2a27
23
src/if_lua.c
23
src/if_lua.c
@@ -1720,11 +1720,12 @@ static const luaL_Reg luaV_Window_mt[] = {
|
||||
static int
|
||||
luaV_print(lua_State *L)
|
||||
{
|
||||
int i, n = lua_gettop(L); // nargs
|
||||
const char *s;
|
||||
size_t l;
|
||||
luaL_Buffer b;
|
||||
luaL_buffinit(L, &b);
|
||||
int i, n = lua_gettop(L); // nargs
|
||||
const char *s;
|
||||
size_t l;
|
||||
garray_T msg_ga;
|
||||
|
||||
ga_init2(&msg_ga, 1, 128);
|
||||
lua_getglobal(L, "tostring");
|
||||
for (i = 1; i <= n; i++)
|
||||
{
|
||||
@@ -1735,13 +1736,19 @@ luaV_print(lua_State *L)
|
||||
if (s == NULL)
|
||||
return luaL_error(L, "cannot convert to string");
|
||||
if (i > 1)
|
||||
luaL_addchar(&b, ' '); // use space instead of tab
|
||||
luaV_addlstring(&b, s, l, 0);
|
||||
ga_append(&msg_ga, ' '); // use space instead of tab
|
||||
ga_concat_len(&msg_ga, (char_u *)s, l);
|
||||
lua_pop(L, 1);
|
||||
}
|
||||
luaL_pushresult(&b);
|
||||
// Replace any "\n" with "\0"
|
||||
for (i = 0; i < msg_ga.ga_len; i++)
|
||||
if (((char *)msg_ga.ga_data)[i] == '\n')
|
||||
((char *)msg_ga.ga_data)[i] = '\0';
|
||||
lua_pushlstring(L, msg_ga.ga_data, msg_ga.ga_len);
|
||||
if (!got_int)
|
||||
luaV_msg(L);
|
||||
|
||||
ga_clear(&msg_ga);
|
||||
return 0;
|
||||
}
|
||||
|
||||
|
Reference in New Issue
Block a user