Skip to content

Conversation

miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Jan 11, 2019

Fix memory leaks in asyncio ProactorEventLoop on overlapped operation
failures.

Changes:

  • Implement the tp_traverse slot in the _overlapped.Overlapped type
    to help to break reference cycles and identify referrers in the
    garbage collector.
  • Always clear overlapped on failure: not only set type to
    TYPE_NOT_STARTED, but release also resources.
    (cherry picked from commit 5485085)

Co-authored-by: Victor Stinner vstinner@redhat.com

https://bugs.python.org/issue32710

Fix memory leaks in asyncio ProactorEventLoop on overlapped operation
failures.

Changes:

* Implement the tp_traverse slot in the _overlapped.Overlapped type
  to help to break reference cycles and identify referrers in the
  garbage collector.
* Always clear overlapped on failure: not only set type to
  TYPE_NOT_STARTED, but release also resources.
(cherry picked from commit 5485085)

Co-authored-by: Victor Stinner <vstinner@redhat.com>
Copy link
Member

@vstinner vstinner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, good bot.

@miss-islington
Copy link
Contributor Author

@vstinner: Status check is done, and it's a success ✅ .

@miss-islington miss-islington merged commit 059997d into python:3.7 Jan 11, 2019
@miss-islington miss-islington deleted the backport-5485085-3.7 branch January 11, 2019 14:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants