FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

[3.12] gh-127667: fix memory leaks in `hashlib` (GH-127668) by picnixz · Pull Request #130783 · python/cpython · GitHub

Repository navigation

[3.12] gh-127667: fix memory leaks in hashlib (GH-127668) - #130783

Merged
picnixz merged 1 commit into
python:3.12from
picnixz:bp/312/097846502b7f33cb327d512e2a396acf4f4de46e-hashlib-leaks-127668
Mar 3, 2025
Merged

picnixz merged 1 commit into
python:3.12from
picnixz:bp/312/097846502b7f33cb327d512e2a396acf4f4de46e-hashlib-leaks-127668

Conversation

picnixz commented Mar 3, 2025 •
edited
Loading

Copy link
Copy Markdown
Member
  • Correctly handle NULL values returned by EVP_MD_CTX_md.
  • Correctly free resources in error branches.
  • Consistently suppress _setException() return value when needed.
  • Collapse _setException() + return NULL into a single statement.

(cherry-picked from commit 0978465)

- Correctly handle `NULL` values returned by `EVP_MD_CTX_md`.
- Correctly free resources in error branches.
- Consistently suppress `_setException()` return value when needed.
- Collapse `_setException() + return NULL` into a single statement.
picnixz changed the title [3.12] gh-127667: fix memory leaks in hashlib (#127668) [3.12] gh-127667: fix memory leaks in hashlib (gh-127668) Mar 3, 2025
picnixz changed the title [3.12] gh-127667: fix memory leaks in hashlib (gh-127668) [3.12] gh-127667: fix memory leaks in hashlib (GH-127668) Mar 3, 2025
picnixz enabled auto-merge (squash) March 3, 2025 10:35
picnixz merged commit d25da89 into python:3.12 Mar 3, 2025
picnixz deleted the bp/312/097846502b7f33cb327d512e2a396acf4f4de46e-hashlib-leaks-127668 branch March 3, 2025 11:25
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

1 participant


Back | FazBrowse Home | New Git URL