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

4ed5ad79 · Xiang Zhang · GitHub · ba75af71 · 4ed5ad79 · 4ed5ad79
Commit 4ed5ad79 authored Jul 19, 2017 by Xiang Zhang Committed by GitHub Jul 19, 2017
Showing with 2 additions and 2 deletions

Misc/NEWS.d/next/Library/2017-07-18-23-47-51.bpo-30961.064jz0.rst ...S.d/next/Library/2017-07-18-23-47-51.bpo-30961.064jz0.rst +1 -0

Modules/_tracemalloc.c Modules/_tracemalloc.c +1 -2

No files found.
--- 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
+Fix decrementing a borrowed reference in tracemalloc.
--- a/Modules/_tracemalloc.c
+++ b/Modules/_tracemalloc.c
@@ -1752,7 +1752,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())
@@ -1763,7 +1763,6 @@ _PyTraceMalloc_Init(void)
        }

        nframe = parse_sys_xoptions(value);
-        Py_DECREF(value);
        if (nframe < 0) {
            Py_FatalError("-X tracemalloc=NFRAME: invalid number of frames");
        }