]> granicus.if.org Git - vim/commitdiff
patch 8.2.2663: Vim9: leaking memory when inline function has an error v8.2.2663
authorBram Moolenaar <Bram@vim.org>
Sat, 27 Mar 2021 14:40:11 +0000 (15:40 +0100)
committerBram Moolenaar <Bram@vim.org>
Sat, 27 Mar 2021 14:40:11 +0000 (15:40 +0100)
Problem:    Vim9: leaking memory when inline function has an error.
Solution:   Free the partially allocated function.

src/userfunc.c
src/version.c

index 57669419d29b81b6c9ef7f4b549111165f6e1534..ba7e0c3da2be4ba84533af9f9ab630c5556131ac 100644 (file)
@@ -35,6 +35,8 @@ static char *e_funcref = N_("E718: Funcref required");
 static char *e_nofunc = N_("E130: Unknown function: %s");
 
 static void funccal_unref(funccall_T *fc, ufunc_T *fp, int force);
+static void func_clear(ufunc_T *fp, int force);
+static int func_free(ufunc_T *fp, int force);
 
     void
 func_init()
@@ -946,7 +948,7 @@ lambda_function_body(
 {
     int                evaluate = evalarg != NULL
                                      && (evalarg->eval_flags & EVAL_EVALUATE);
-    ufunc_T    *ufunc;
+    ufunc_T    *ufunc = NULL;
     exarg_T    eap;
     garray_T   newlines;
     char_u     *cmdline = NULL;
@@ -1000,10 +1002,7 @@ lambda_function_body(
        goto erret;
     set_ufunc_name(ufunc, name);
     if (hash_add(&func_hashtab, UF2HIKEY(ufunc)) == FAIL)
-    {
-       vim_free(ufunc);
        goto erret;
-    }
     ufunc->uf_refcount = 1;
     ufunc->uf_args = *newargs;
     newargs->ga_data = NULL;
@@ -1045,6 +1044,7 @@ lambda_function_body(
 
     rettv->vval.v_partial = pt;
     rettv->v_type = VAR_PARTIAL;
+    ufunc = NULL;
     ret = OK;
 
 erret:
@@ -1054,6 +1054,11 @@ erret:
     ga_clear_strings(&newlines);
     ga_clear_strings(newargs);
     ga_clear_strings(default_args);
+    if (ufunc != NULL)
+    {
+       func_clear(ufunc, TRUE);
+       func_free(ufunc, TRUE);
+    }
     return ret;
 }
 
index 60fe36f0f6f190b86e4a4698887c5f776856737a..2b63d6d91006a629d79dac26b7d968c328bc73f2 100644 (file)
@@ -750,6 +750,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    2663,
 /**/
     2662,
 /**/