diff options
| author | Serhiy Storchaka <storchaka@gmail.com> | 2016-12-11 12:43:18 (GMT) |
|---|---|---|
| committer | Serhiy Storchaka <storchaka@gmail.com> | 2016-12-11 12:43:18 (GMT) |
| commit | 8114f21668fe9775d2542d079c6396a745f4f094 (patch) | |
| tree | 62ef9087e76229ea0fd18d0a1052abd79c9ae2f3 /Python/errors.c | |
| parent | 0a7b8596c075b3d69e2fae3c71746415ba3bad39 (diff) | |
| parent | 8b58339eb2939da4df822f7ea457b44e120fad45 (diff) | |
| download | cpython-8114f21668fe9775d2542d079c6396a745f4f094.zip cpython-8114f21668fe9775d2542d079c6396a745f4f094.tar.gz cpython-8114f21668fe9775d2542d079c6396a745f4f094.tar.bz2 | |
Issue #28512: Fixed setting the offset attribute of SyntaxError by
PyErr_SyntaxLocationEx() and PyErr_SyntaxLocationObject().
Diffstat (limited to 'Python/errors.c')
| -rw-r--r-- | Python/errors.c | 12 |
1 files changed, 4 insertions, 8 deletions
diff --git a/Python/errors.c b/Python/errors.c index 7b25a22..fd6c42f 100644 --- a/Python/errors.c +++ b/Python/errors.c @@ -1059,16 +1059,15 @@ PyErr_SyntaxLocationObject(PyObject *filename, int lineno, int col_offset) PyErr_Clear(); Py_DECREF(tmp); } + tmp = NULL; if (col_offset >= 0) { tmp = PyLong_FromLong(col_offset); if (tmp == NULL) PyErr_Clear(); - else { - if (_PyObject_SetAttrId(v, &PyId_offset, tmp)) - PyErr_Clear(); - Py_DECREF(tmp); - } } + if (_PyObject_SetAttrId(v, &PyId_offset, tmp ? tmp : Py_None)) + PyErr_Clear(); + Py_XDECREF(tmp); if (filename != NULL) { if (_PyObject_SetAttrId(v, &PyId_filename, filename)) PyErr_Clear(); @@ -1080,9 +1079,6 @@ PyErr_SyntaxLocationObject(PyObject *filename, int lineno, int col_offset) Py_DECREF(tmp); } } - if (_PyObject_SetAttrId(v, &PyId_offset, Py_None)) { - PyErr_Clear(); - } if (exc != PyExc_SyntaxError) { if (!_PyObject_HasAttrId(v, &PyId_msg)) { tmp = PyObject_Str(v); |