From 8e9a24a127c4ef8833fdc3986623f96c7d04210f Mon Sep 17 00:00:00 2001 From: Bram Moolenaar Date: Tue, 19 Mar 2019 22:22:55 +0100 Subject: patch 8.1.1021: pyeval() and py3eval() leak memory Problem: pyeval() and py3eval() leak memory. Solution: Do not increase the reference count twice. (Ozaki Kiichi, closes #4129) --- src/if_python.c | 24 +++++------------------- 1 file changed, 5 insertions(+), 19 deletions(-) (limited to 'src/if_python.c') diff --git a/src/if_python.c b/src/if_python.c index c0a3fe105..9f9c257ee 100644 --- a/src/if_python.c +++ b/src/if_python.c @@ -1555,30 +1555,16 @@ FunctionGetattr(PyObject *self, char *name) } void -do_pyeval (char_u *str, typval_T *rettv) +do_pyeval(char_u *str, typval_T *rettv) { DoPyCommand((char *) str, (rangeinitializer) init_range_eval, (runner) run_eval, (void *) rettv); - switch (rettv->v_type) + if (rettv->v_type == VAR_UNKNOWN) { - case VAR_DICT: ++rettv->vval.v_dict->dv_refcount; break; - case VAR_LIST: ++rettv->vval.v_list->lv_refcount; break; - case VAR_FUNC: func_ref(rettv->vval.v_string); break; - case VAR_PARTIAL: ++rettv->vval.v_partial->pt_refcount; break; - case VAR_UNKNOWN: - rettv->v_type = VAR_NUMBER; - rettv->vval.v_number = 0; - break; - case VAR_NUMBER: - case VAR_STRING: - case VAR_FLOAT: - case VAR_SPECIAL: - case VAR_JOB: - case VAR_CHANNEL: - case VAR_BLOB: - break; + rettv->v_type = VAR_NUMBER; + rettv->vval.v_number = 0; } } @@ -1594,7 +1580,7 @@ Py_GetProgramName(void) #endif /* Python 1.4 */ int -set_ref_in_python (int copyID) +set_ref_in_python(int copyID) { return set_ref_in_py(copyID); } -- cgit v1.2.1