Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[2.7] bpo-36179: Fix ref leaks in _hashopenssl (GH-12158) #12166

Merged
merged 1 commit into from Mar 4, 2019
Merged

[2.7] bpo-36179: Fix ref leaks in _hashopenssl (GH-12158) #12166

merged 1 commit into from Mar 4, 2019

Conversation

tiran
Copy link
Member

@tiran tiran commented Mar 4, 2019 •
edited by bedevere-bot

Fix two unlikely reference leaks in _hashopenssl. The leaks only occur in
out-of-memory cases. Thanks to Charalampos Stratakis.

Signed-off-by: Christian Heimes christian@python.org

https://bugs.python.org/issue36179.
(cherry picked from commit b7bc283)

Co-authored-by: Christian Heimes christian@python.org

https://bugs.python.org/issue36179

@tiran
[2.7] bpo-36179: Fix ref leaks in _hashopenssl (pythonGH-12158)
13adbca 
Fix two unlikely reference leaks in _hashopenssl. The leaks only occur in
out-of-memory cases. Thanks to Charalampos Stratakis.

Signed-off-by: Christian Heimes <christian@python.org>

https://bugs.python.org/issue36179.
(cherry picked from commit b7bc283)

Co-authored-by: Christian Heimes <christian@python.org>
@bedevere-bot bedevere-bot mentioned this pull request Mar 4, 2019
Merged
vstinner
vstinner approved these changes Mar 4, 2019
View changes
Copy link
Member

@vstinner vstinner left a comment

LGTM.

@tiran tiran added the 🤖 automerge PR will be merged once it's been approved and all CI passed label Mar 4, 2019
@miss-islington miss-islington merged commit 84b5ac9 into python:2.7 Mar 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vstinner vstinner

Assignees
No one assigned
Labels
🤖 automerge PR will be merged once it's been approved and all CI passed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@tiran @vstinner @the-knights-who-say-ni @bedevere-bot @miss-islington