diff options
author | Neal Norwitz <nnorwitz@gmail.com> | 2003-01-13 20:13:12 (GMT) |
---|---|---|
committer | Neal Norwitz <nnorwitz@gmail.com> | 2003-01-13 20:13:12 (GMT) |
commit | 1a9975014f7ddc583a2428fe47d0e17261f98b46 (patch) | |
tree | febd9464101af88429c179aac60f378a6ee680dc /Objects | |
parent | a974b3939f8f6239018fd0de44530244812191da (diff) | |
download | cpython-1a9975014f7ddc583a2428fe47d0e17261f98b46.zip cpython-1a9975014f7ddc583a2428fe47d0e17261f98b46.tar.gz cpython-1a9975014f7ddc583a2428fe47d0e17261f98b46.tar.bz2 |
Fix SF bug #667147, Segmentation fault printing str subclass
Fix infinite recursion which occurred when printing an object
whose __str__() returned self.
Will backport
Diffstat (limited to 'Objects')
-rw-r--r-- | Objects/object.c | 19 |
1 files changed, 16 insertions, 3 deletions
diff --git a/Objects/object.c b/Objects/object.c index 3328643..e3234de 100644 --- a/Objects/object.c +++ b/Objects/object.c @@ -158,10 +158,15 @@ _PyObject_Del(PyObject *op) PyObject_FREE(op); } -int -PyObject_Print(PyObject *op, FILE *fp, int flags) +/* Implementation of PyObject_Print with recursion checking */ +static int +internal_print(PyObject *op, FILE *fp, int flags, int nesting) { int ret = 0; + if (nesting > 10) { + PyErr_SetString(PyExc_RuntimeError, "print recursion"); + return -1; + } if (PyErr_CheckSignals()) return -1; #ifdef USE_STACKCHECK @@ -187,7 +192,8 @@ PyObject_Print(PyObject *op, FILE *fp, int flags) if (s == NULL) ret = -1; else { - ret = PyObject_Print(s, fp, Py_PRINT_RAW); + ret = internal_print(s, fp, Py_PRINT_RAW, + nesting+1); } Py_XDECREF(s); } @@ -204,6 +210,13 @@ PyObject_Print(PyObject *op, FILE *fp, int flags) return ret; } +int +PyObject_Print(PyObject *op, FILE *fp, int flags) +{ + return internal_print(op, fp, flags, 0); +} + + /* For debugging convenience. See Misc/gdbinit for some useful gdb hooks */ void _PyObject_Dump(PyObject* op) { |