thac0/vim
1
0
Fork 0
forked from aniani/vim
Code Activity

patch 8.2.3822: leaking memory in map() and filter(), no string in Vim9

Browse Source 
Problem:    Leaking memory in map() and filter(), cannot use a string argument
            in Vim9 script.
Solution:   Fix the leak, adjust the argument check, also run the tests as
            Vim9 script. (Yegappan Lakshmanan, closes #9354)
This commit is contained in:
Bram Moolenaar
2021-12-16 08:21:09 +00:00
parent 19569ca6d8
commit 2d877599ee
8 changed files with 127 additions and 61 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
src/list.c
View File

@@ -2281,11 +2281,12 @@ filter_map(typval_T *argvars, typval_T *rettv, filtermap_T filtermap)
garray_T type_list;
// map() and filter() return the first argument, also on failure.
if (filtermap != FILTERMAP_MAPNEW)
if (filtermap != FILTERMAP_MAPNEW && argvars[0].v_type != VAR_STRING)
copy_tv(&argvars[0], rettv);
if (in_vim9script()
&& (check_for_list_or_dict_or_blob_arg(argvars, 0) == FAIL))
&& (check_for_list_or_dict_or_blob_or_string_arg(argvars, 0)
== FAIL))
return;
if (filtermap == FILTERMAP_MAP && in_vim9script())