Skip to content

Comments

Fix memory leak in abnormal branch of mlx_kem_dup function#29318

Closed
lan1120 wants to merge 1 commit intoopenssl:masterfrom
lan1120:mlx-kem
Closed

Fix memory leak in abnormal branch of mlx_kem_dup function#29318
lan1120 wants to merge 1 commit intoopenssl:masterfrom
lan1120:mlx-kem

Conversation

@lan1120
Copy link
Contributor

@lan1120 lan1120 commented Dec 5, 2025

Fix memory leak in abnormal branch of mlx_kem_dup function.

@t8m t8m added branch: master Applies to master branch approval: review pending This pull request needs review by a committer triaged: bug The issue/pr is/fixes a bug tests: exempted The PR is exempt from requirements for testing branch: 3.5 Applies to openssl-3.5 branch: 3.6 Applies to openssl-3.6 labels Dec 5, 2025
@github-actions github-actions bot added the severity: fips change The pull request changes FIPS provider sources label Dec 5, 2025
Copy link
Member

@beldmit beldmit left a comment

Choose a reason for hiding this comment

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

LGTM

@beldmit beldmit added approval: done This pull request has the required number of approvals and removed approval: review pending This pull request needs review by a committer labels Dec 5, 2025
@openssl-machine openssl-machine removed the approval: done This pull request has the required number of approvals label Dec 6, 2025
@openssl-machine
Copy link
Collaborator

This pull request is ready to merge

@openssl-machine openssl-machine added the approval: ready to merge The 24 hour grace period has passed, ready to merge label Dec 6, 2025
@t8m
Copy link
Member

t8m commented Dec 11, 2025

Merged to the master, 3.6 and 3.5 branches. Thank you for your contribution.

@t8m t8m closed this Dec 11, 2025
openssl-machine pushed a commit that referenced this pull request Dec 11, 2025
Reviewed-by: Dmitry Belyavskiy <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
(Merged from #29318)

(cherry picked from commit 013e117)
openssl-machine pushed a commit that referenced this pull request Dec 11, 2025
Reviewed-by: Dmitry Belyavskiy <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
(Merged from #29318)

(cherry picked from commit 013e117)
openssl-machine pushed a commit that referenced this pull request Dec 11, 2025
Reviewed-by: Dmitry Belyavskiy <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
(Merged from #29318)
cxx194832 pushed a commit to cxx194832/openssl that referenced this pull request Dec 12, 2025
Reviewed-by: Dmitry Belyavskiy <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
(Merged from openssl#29318)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approval: ready to merge The 24 hour grace period has passed, ready to merge branch: master Applies to master branch branch: 3.5 Applies to openssl-3.5 branch: 3.6 Applies to openssl-3.6 severity: fips change The pull request changes FIPS provider sources tests: exempted The PR is exempt from requirements for testing triaged: bug The issue/pr is/fixes a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants