gh-101765: Fix refcount issues in list and unicode pickling #102265
Conversation
|
If you want to schedule another build, you need to add the |
|
Thanks @JelleZijlstra for the PR |
|
Sorry, @JelleZijlstra, I could not cleanly backport this to |
|
Sorry @JelleZijlstra, I had trouble checking out the |
|
Working on the backports |
|
GH-102268 is a backport of this pull request to the 3.11 branch. |
…ing (pythonGH-102265) Followup from pythonGH-101769.. (cherry picked from commit d71edbd) Co-authored-by: Jelle Zijlstra <jelle.zijlstra@gmail.com>
|
GH-102269 is a backport of this pull request to the 3.10 branch. |
…ing (pythonGH-102265) Followup from pythonGH-101769.. (cherry picked from commit d71edbd) Co-authored-by: Jelle Zijlstra <jelle.zijlstra@gmail.com>
| @@ -14794,8 +14794,10 @@ unicodeiter_reduce(unicodeiterobject *it, PyObject *Py_UNUSED(ignored)) | |||
| return Py_BuildValue("N(O)n", iter, it->it_seq, it->it_index); | |||
| } else { | |||
| PyObject *u = unicode_new_empty(); | |||
| if (u == NULL) | |||
| if (u == NULL) { | |||
| Py_DECREF(iter); | |||
There was a problem hiding this comment.
Shouldn't this be Py_XDECREF? There is no null check above for iter.
There was a problem hiding this comment.
Good point. But shouldn’t it be a null check and return above, if iter is null I think it would crash in the BuildValue before we even get here.
Followup from #101769.
__reduce__can segfault if accessing__builtins__.__dict__['iter']mutates the iter object #101765