diff options
author | Antoine Pitrou <solipsis@pitrou.net> | 2010-10-28 22:56:58 +0000 |
---|---|---|
committer | Antoine Pitrou <solipsis@pitrou.net> | 2010-10-28 22:56:58 +0000 |
commit | a0b3b96364f8ee0e88add47f1497bb2111878039 (patch) | |
tree | 379c6f1cfdf3a2b318524fa285e8815c2475d202 /Python/errors.c | |
parent | 0107769159cdb9adecaa0aaf0f387f79691b9ed6 (diff) | |
download | cpython-a0b3b96364f8ee0e88add47f1497bb2111878039.tar.gz |
Issue #5437: A preallocated MemoryError instance should not hold traceback
data (including local variables caught in the stack trace) alive infinitely.
Diffstat (limited to 'Python/errors.c')
-rw-r--r-- | Python/errors.c | 24 |
1 files changed, 1 insertions, 23 deletions
diff --git a/Python/errors.c b/Python/errors.c index 04906141d5..d5a6fae0b4 100644 --- a/Python/errors.c +++ b/Python/errors.c @@ -333,29 +333,7 @@ PyErr_BadArgument(void) PyObject * PyErr_NoMemory(void) { - if (PyErr_ExceptionMatches(PyExc_MemoryError)) - /* already current */ - return NULL; - - /* raise the pre-allocated instance if it still exists */ - if (PyExc_MemoryErrorInst) - { - /* Clear the previous traceback, otherwise it will be appended - * to the current one. - * - * The following statement is not likely to raise any error; - * if it does, we simply discard it. - */ - PyException_SetTraceback(PyExc_MemoryErrorInst, Py_None); - - PyErr_SetObject(PyExc_MemoryError, PyExc_MemoryErrorInst); - } - else - /* this will probably fail since there's no memory and hee, - hee, we have to instantiate this class - */ - PyErr_SetNone(PyExc_MemoryError); - + PyErr_SetNone(PyExc_MemoryError); return NULL; } |