From: Bram Moolenaar <Bram@vim.org>
Date: Tue, 15 Jun 2021 09:27:21 +0000 (+0200)
Subject: patch 8.2.3001: Vim9: memory leak when compilation fails
X-Git-Tag: v8.2.3001
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=caf1a2f296499bdcf030aca545a04da65ac210a1;p=vim

patch 8.2.3001: Vim9: memory leak when compilation fails

Problem:    Vim9: memory leak when compilation fails.
Solution:   Free the list of variable names.
---

diff --git a/src/version.c b/src/version.c
index 3e33c5d4c..3696c2e2a 100644
--- a/src/version.c
+++ b/src/version.c
@@ -750,6 +750,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    3001,
 /**/
     3000,
 /**/
diff --git a/src/vim9compile.c b/src/vim9compile.c
index 57e2311ff..838939921 100644
--- a/src/vim9compile.c
+++ b/src/vim9compile.c
@@ -9724,8 +9724,10 @@ erret:
 	dfunc_T	*dfunc = ((dfunc_T *)def_functions.ga_data)
 							 + ufunc->uf_dfunc_idx;
 
+	// Compiling aborted, free the generated instructions.
 	clear_instr_ga(instr);
 	VIM_CLEAR(dfunc->df_name);
+	ga_clear_strings(&dfunc->df_var_names);
 
 	// If using the last entry in the table and it was added above, we
 	// might as well remove it.