diff options
| author | Zackery Spytz <zspytz@gmail.com> | 2018-10-06 06:44:25 (GMT) |
|---|---|---|
| committer | Serhiy Storchaka <storchaka@gmail.com> | 2018-10-06 06:44:25 (GMT) |
| commit | ae62f015240c9162773341a9922794e6b960779d (patch) | |
| tree | 24bfc29c50a3f0cdf9f98b14a58c29f6d19b5c43 | |
| parent | cd45385ffad8910293e5659cfe7ab036e70613b7 (diff) | |
| download | cpython-ae62f015240c9162773341a9922794e6b960779d.zip cpython-ae62f015240c9162773341a9922794e6b960779d.tar.gz cpython-ae62f015240c9162773341a9922794e6b960779d.tar.bz2 | |
bpo-34910: Ensure that PyObject_Print() always returns -1 on error. (GH-9733)
| -rw-r--r-- | Misc/NEWS.d/next/C API/2018-10-05-17-06-49.bpo-34910.tSFrls.rst | 2 | ||||
| -rw-r--r-- | Objects/object.c | 5 |
2 files changed, 5 insertions, 2 deletions
diff --git a/Misc/NEWS.d/next/C API/2018-10-05-17-06-49.bpo-34910.tSFrls.rst b/Misc/NEWS.d/next/C API/2018-10-05-17-06-49.bpo-34910.tSFrls.rst new file mode 100644 index 0000000..eff4755 --- /dev/null +++ b/Misc/NEWS.d/next/C API/2018-10-05-17-06-49.bpo-34910.tSFrls.rst @@ -0,0 +1,2 @@ +Ensure that :c:func:`PyObject_Print` always returns ``-1`` on error. Patch +by Zackery Spytz. diff --git a/Objects/object.c b/Objects/object.c index 607f047..ab1baa7 100644 --- a/Objects/object.c +++ b/Objects/object.c @@ -375,8 +375,9 @@ PyObject_Print(PyObject *op, FILE *fp, int flags) else if (PyUnicode_Check(s)) { PyObject *t; t = PyUnicode_AsEncodedString(s, "utf-8", "backslashreplace"); - if (t == NULL) - ret = 0; + if (t == NULL) { + ret = -1; + } else { fwrite(PyBytes_AS_STRING(t), 1, PyBytes_GET_SIZE(t), fp); |