Mercurial > vim

diff src/userfunc.c @ 21566:1e3e5058041c v8.2.1333

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
patch 8.2.1333: Vim9: memory leak when using nested global function Commit: https://github.com/vim/vim/commit/af8edbb8dc551aaf2105a4d77b3d8495da239ef0 Author: Bram Moolenaar <Bram@vim.org> Date: Sat Aug 1 00:03:09 2020 +0200 patch 8.2.1333: Vim9: memory leak when using nested global function Problem: Vim9: memory leak when using nested global function. Solution: Swap from and to when copying the lines.
author Bram Moolenaar <Bram@vim.org>
date Sat, 01 Aug 2020 00:15:04 +0200
parents 30a997217524
children 5470c36ed7e6
line wrap: on
line diff
--- a/src/userfunc.c
+++ b/src/userfunc.c
@@ -1188,10 +1188,10 @@ copy_func(char_u *lambda, char_u *global
 	fp->uf_flags = (ufunc->uf_flags & ~FC_VIM9) | FC_COPY;
 	fp->uf_def_status = ufunc->uf_def_status;
 	fp->uf_dfunc_idx = ufunc->uf_dfunc_idx;
-	if (ga_copy_strings(&fp->uf_args, &ufunc->uf_args) == FAIL
-		|| ga_copy_strings(&fp->uf_def_args, &ufunc->uf_def_args)
+	if (ga_copy_strings(&ufunc->uf_args, &fp->uf_args) == FAIL
+		|| ga_copy_strings(&ufunc->uf_def_args, &fp->uf_def_args)
 									== FAIL
-		|| ga_copy_strings(&fp->uf_lines, &ufunc->uf_lines) == FAIL)
+		|| ga_copy_strings(&ufunc->uf_lines, &fp->uf_lines) == FAIL)
 	    goto failed;
 
 	fp->uf_name_exp = ufunc->uf_name_exp == NULL ? NULL