Commit f4b3167f authored by Benjamin Peterson's avatar Benjamin Peterson

fix possible overflow in encode_basestring_ascii (closes #23369)

parent 68a74091
from collections import OrderedDict from collections import OrderedDict
from test.test_json import PyTest, CTest from test.test_json import PyTest, CTest
from test.support import bigaddrspacetest
CASES = [ CASES = [
...@@ -41,4 +42,10 @@ class TestEncodeBasestringAscii: ...@@ -41,4 +42,10 @@ class TestEncodeBasestringAscii:
class TestPyEncodeBasestringAscii(TestEncodeBasestringAscii, PyTest): pass class TestPyEncodeBasestringAscii(TestEncodeBasestringAscii, PyTest): pass
class TestCEncodeBasestringAscii(TestEncodeBasestringAscii, CTest): pass class TestCEncodeBasestringAscii(TestEncodeBasestringAscii, CTest):
@bigaddrspacetest
def test_overflow(self):
s = "\uffff"*((2**32)//6 + 1)
with self.assertRaises(OverflowError):
self.json.encoder.encode_basestring_ascii(s)
...@@ -13,6 +13,12 @@ Core and Builtins ...@@ -13,6 +13,12 @@ Core and Builtins
- Issue #23055: Fixed a buffer overflow in PyUnicode_FromFormatV. Analysis - Issue #23055: Fixed a buffer overflow in PyUnicode_FromFormatV. Analysis
and fix by Guido Vranken. and fix by Guido Vranken.
Library
-------
- Issue #23369: Fixed possible integer overflow in
_json.encode_basestring_ascii.
What's New in Python 3.3.6? What's New in Python 3.3.6?
=========================== ===========================
......
...@@ -216,17 +216,24 @@ ascii_escape_unicode(PyObject *pystr) ...@@ -216,17 +216,24 @@ ascii_escape_unicode(PyObject *pystr)
/* Compute the output size */ /* Compute the output size */
for (i = 0, output_size = 2; i < input_chars; i++) { for (i = 0, output_size = 2; i < input_chars; i++) {
Py_UCS4 c = PyUnicode_READ(kind, input, i); Py_UCS4 c = PyUnicode_READ(kind, input, i);
if (S_CHAR(c)) Py_ssize_t d;
output_size++; if (S_CHAR(c)) {
d = 1;
}
else { else {
switch(c) { switch(c) {
case '\\': case '"': case '\b': case '\f': case '\\': case '"': case '\b': case '\f':
case '\n': case '\r': case '\t': case '\n': case '\r': case '\t':
output_size += 2; break; d = 2; break;
default: default:
output_size += c >= 0x10000 ? 12 : 6; d = c >= 0x10000 ? 12 : 6;
} }
} }
if (output_size > PY_SSIZE_T_MAX - d) {
PyErr_SetString(PyExc_OverflowError, "string is too long to escape");
return NULL;
}
output_size += d;
} }
rval = PyUnicode_New(output_size, 127); rval = PyUnicode_New(output_size, 127);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment