[Python-checkins] cpython (2.7): Issue #20041: Fixed TypeError when frame.f_trace is set to None.

[serhiy.storchaka]

https://hg.python.org/cpython/rev/4d916be61d46
changeset:   101702:4d916be61d46
branch:      2.7
parent:      101695:7c20ea2bdb3f
user:        Serhiy Storchaka <storchaka at gmail.com>
date:        Sat Jun 04 20:30:43 2016 +0300
summary:
  Issue #20041: Fixed TypeError when frame.f_trace is set to None.
Patch by Xavier de Gaye.

files:
  Lib/test/test_sys_settrace.py |  9 +++++++++
  Misc/NEWS                     |  3 +++
  Objects/frameobject.c         |  8 +++-----
  3 files changed, 15 insertions(+), 5 deletions(-)


diff --git a/Lib/test/test_sys_settrace.py b/Lib/test/test_sys_settrace.py
--- a/Lib/test/test_sys_settrace.py
+++ b/Lib/test/test_sys_settrace.py
@@ -386,6 +386,15 @@
              (257, 'line'),
              (257, 'return')])
 
+    def test_17_none_f_trace(self):
+        # Issue 20041: fix TypeError when f_trace is set to None.
+        def func():
+            sys._getframe().f_trace = None
+            lineno = 2
+        self.run_and_compare(func,
+            [(0, 'call'),
+             (1, 'line')])
+
 
 class RaisingTraceFuncTestCase(unittest.TestCase):
     def trace(self, frame, event, arg):
diff --git a/Misc/NEWS b/Misc/NEWS
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -10,6 +10,9 @@
 Core and Builtins
 -----------------
 
+- Issue #20041: Fixed TypeError when frame.f_trace is set to None.
+  Patch by Xavier de Gaye.
+
 - Issue #25702: A --with-lto configure option has been added that will
   enable link time optimizations at build time during a make profile-opt.
   Some compilers and toolchains are known to not produce stable code when
diff --git a/Objects/frameobject.c b/Objects/frameobject.c
--- a/Objects/frameobject.c
+++ b/Objects/frameobject.c
@@ -364,15 +364,13 @@
 static int
 frame_settrace(PyFrameObject *f, PyObject* v, void *closure)
 {
-    PyObject* old_value;
-
     /* We rely on f_lineno being accurate when f_trace is set. */
     f->f_lineno = PyFrame_GetLineNumber(f);
 
-    old_value = f->f_trace;
+    if (v == Py_None)
+        v = NULL;
     Py_XINCREF(v);
-    f->f_trace = v;
-    Py_XDECREF(old_value);
+    Py_XSETREF(f->f_trace, v);
 
     return 0;
 }

-- 
Repository URL: https://hg.python.org/cpython