Partial fix to issue #1731706: memory leak in Tkapp_Call when calling

from a thread different than the one that created the Tcl interpreter.

diff --git a/Misc/NEWS b/Misc/NEWS
index c84325796ee76f32fbc581d5727915c0b633f765..3c86cf230578b6cc37374408927fc343e0e181fc 100644 (file)
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -149,6 +149,10 @@ Core and Builtins
 Library
 -------
 
+- Partial fix to issue #1731706: memory leak in Tkapp_Call when calling
+  from a thread different than the one that created the Tcl interpreter.
+  Patch by Robert Hancock.
+
 - Issue #1520877: Now distutils.sysconfig reads $AR from the 
   environment/Makefile. Patch by Douglas Greiman.
 

diff --git a/Modules/_tkinter.c b/Modules/_tkinter.c
index 8b56bcd0dc38bef072d7120e3383207744ac9f9b..79fa6327ae3ad983b6bb111bfef039919e42ea87 100644 (file)
--- a/Modules/_tkinter.c
+++ b/Modules/_tkinter.c
@@ -1256,7 +1256,9 @@ Tkapp_CallProc(Tkapp_CallEvent *e, int flags)
                *(e->res) = Tkapp_CallResult(e->self);
        }
        LEAVE_PYTHON
-  done:
+
+       Tkapp_CallDeallocArgs(objv, objStore, objc);
+done:
        /* Wake up calling thread. */
        Tcl_MutexLock(&call_mutex);
        Tcl_ConditionNotify(&e->done);