bpo-30961: Fix decrementing a borrowed reference in tracemalloc. (#2747) (#2749)

zhangyangyu · web-flow · commit cb3f024f3ac9 · 2017-07-19T00:51:58.000+08:00
(cherry picked from commit 4ed5ad7)
diff --git a/Misc/NEWS.d/next/Library/2017-07-18-23-47-51.bpo-30961.064jz0.rst b/Misc/NEWS.d/next/Library/2017-07-18-23-47-51.bpo-30961.064jz0.rst
@@ -0,0 +1 @@
+Fix decrementing a borrowed reference in tracemalloc.
diff --git a/Modules/_tracemalloc.c b/Modules/_tracemalloc.c
@@ -1408,7 +1408,7 @@ _PyTraceMalloc_Init(void)
         if (key == NULL)
             return -1;
 
-        value = PyDict_GetItemWithError(xoptions, key);
+        value = PyDict_GetItemWithError(xoptions, key); /* borrowed */
         Py_DECREF(key);
         if (value == NULL) {
             if (PyErr_Occurred())
@@ -1419,7 +1419,6 @@ _PyTraceMalloc_Init(void)
         }
 
         nframe = parse_sys_xoptions(value);
-        Py_DECREF(value);
         if (nframe < 0) {
             Py_FatalError("-X tracemalloc=NFRAME: invalid number of frames");
         }

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	`+Fix decrementing a borrowed reference in tracemalloc.`
Original file line number	Diff line number	Diff line change
`@@ -1408,7 +1408,7 @@ _PyTraceMalloc_Init(void)`
`1408`	`1408`	`if (key == NULL)`
`1409`	`1409`	`return -1;`
`1410`	`1410`
`1411`		`- value = PyDict_GetItemWithError(xoptions, key);`
	`1411`	`+ value = PyDict_GetItemWithError(xoptions, key); /* borrowed */`
`1412`	`1412`	`Py_DECREF(key);`
`1413`	`1413`	`if (value == NULL) {`
`1414`	`1414`	`if (PyErr_Occurred())`
`@@ -1419,7 +1419,6 @@ _PyTraceMalloc_Init(void)`
`1419`	`1419`	`}`
`1420`	`1420`
`1421`	`1421`	`nframe = parse_sys_xoptions(value);`
`1422`		`- Py_DECREF(value);`
`1423`	`1422`	`if (nframe < 0) {`
`1424`	`1423`	`Py_FatalError("-X tracemalloc=NFRAME: invalid number of frames");`
`1425`	`1424`	`}`