From 0d68ab3f481ab6c8ecb0d83762b4cd9fb9208356 Mon Sep 17 00:00:00 2001 From: "Gregory P. Smith" Date: Tue, 27 Nov 2012 10:19:29 -0800 Subject: Plug a leak in timemodule. The module dictionary is saved during initialization. If the interpreter is shut down and reinitialized (embedded CPython), the old module dictionary was not dec-refed during the next import of the time extension module. Contributed by Torsten Marek of Google. --- Misc/NEWS | 3 +++ Modules/timemodule.c | 9 ++++++--- 2 files changed, 9 insertions(+), 3 deletions(-) diff --git a/Misc/NEWS b/Misc/NEWS index 8c13755..abe5f51 100644 --- a/Misc/NEWS +++ b/Misc/NEWS @@ -507,6 +507,9 @@ Library Extension Modules ----------------- +- Fix the leak of a dict in the time module when used in an embedded + interpreter that is repeatedly initialized and shutdown and reinitialized. + - Issue #12268: File readline, readlines and read or readall methods no longer lose data when an underlying read system call is interrupted within an io module object. IOError is no longer raised due to a read diff --git a/Modules/timemodule.c b/Modules/timemodule.c index 397cf8c..13be691 100644 --- a/Modules/timemodule.c +++ b/Modules/timemodule.c @@ -96,7 +96,7 @@ static int floatsleep(double); static double floattime(void); /* For Y2K check */ -static PyObject *moddict; +static PyObject *moddict = NULL; /* Exposed in timefuncs.h. */ time_t @@ -858,6 +858,11 @@ inittime(void) /* Accept 2-digit dates unless PYTHONY2K is set and non-empty */ p = Py_GETENV("PYTHONY2K"); PyModule_AddIntConstant(m, "accept2dyear", (long) (!p || !*p)); + /* If an embedded interpreter is shutdown and reinitialized the old + moddict was not decrefed on shutdown and the next import of this + module leads to a leak. Conditionally decref here to prevent that. + */ + Py_XDECREF(moddict); /* Squirrel away the module's dictionary for the y2k check */ moddict = PyModule_GetDict(m); Py_INCREF(moddict); @@ -1050,5 +1055,3 @@ floatsleep(double secs) return 0; } - - -- cgit v0.12