[Python-checkins] [2.7] bpo-36291: Fix a possible reference leak in the json module (GH-12330)
Victor Stinner
webhook-mailer at python.org
Thu Mar 14 11:23:08 EDT 2019
https://github.com/python/cpython/commit/fb3336acfde3204fd01ce519ef24cc18a94dfa3f
commit: fb3336acfde3204fd01ce519ef24cc18a94dfa3f
branch: 2.7
author: stratakis <cstratak at redhat.com>
committer: Victor Stinner <vstinner at redhat.com>
date: 2019-03-14T16:22:46+01:00
summary:
[2.7] bpo-36291: Fix a possible reference leak in the json module (GH-12330)
Fix a reference leak in json if parsing a floating point number fails.
If PyOS_string_to_double() fails in _match_number_str():
decrement numstr ref counter.
files:
A Misc/NEWS.d/next/Library/2019-03-14-15-54-46.bpo-36291.UalHXP.rst
M Modules/_json.c
diff --git a/Misc/NEWS.d/next/Library/2019-03-14-15-54-46.bpo-36291.UalHXP.rst b/Misc/NEWS.d/next/Library/2019-03-14-15-54-46.bpo-36291.UalHXP.rst
new file mode 100644
index 000000000000..07d780cd7fd6
--- /dev/null
+++ b/Misc/NEWS.d/next/Library/2019-03-14-15-54-46.bpo-36291.UalHXP.rst
@@ -0,0 +1 @@
+Fix a possible reference leak in the json module.
diff --git a/Modules/_json.c b/Modules/_json.c
index 3a88882f0c98..050d37daa43c 100644
--- a/Modules/_json.c
+++ b/Modules/_json.c
@@ -1375,8 +1375,10 @@ _match_number_str(PyScannerObject *s, PyObject *pystr, Py_ssize_t start, Py_ssiz
else {
double d = PyOS_string_to_double(PyString_AS_STRING(numstr),
NULL, NULL);
- if (d == -1.0 && PyErr_Occurred())
+ if (d == -1.0 && PyErr_Occurred()) {
+ Py_DECREF(numstr);
return NULL;
+ }
rval = PyFloat_FromDouble(d);
}
}
More information about the Python-checkins
mailing list