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.

Sign up for GitHub

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

Fix potential memory leak in test_bad_dtls #24464

Closed
wants to merge 1 commit into from
Closed

Fix potential memory leak in test_bad_dtls #24464

wants to merge 1 commit into from

Conversation

amiremohamadi
Copy link
Contributor

fix #24444

@t8m t8m added branch: master Merge to master branch approval: review pending This pull request needs review by a committer approval: otc review pending This pull request needs review by an OTC member triaged: bug The issue/pr is/fixes a bug branch: 3.0 Merge to openssl-3.0 branch branch: 3.1 Merge to openssl-3.1 tests: exempted The PR is exempt from requirements for testing branch: 3.2 Merge to openssl-3.2 branch: 3.3 Merge to openssl-3.3 labels May 22, 2024
@t8m t8m removed the approval: otc review pending This pull request needs review by an OTC member label May 22, 2024
@mattcaswell mattcaswell 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 May 23, 2024
@openssl-machine openssl-machine removed the approval: done This pull request has the required number of approvals label May 24, 2024
@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 May 24, 2024
@mattcaswell
Copy link
Member

Pushed to master, 3.3, 3.2, 3.1 and 3.0. Thanks for your contribution.

openssl-machine pushed a commit that referenced this pull request May 29, 2024
@amiremohamadi @mattcaswell
Fix potential memory leak in test_bad_dtls
abe05fd 
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Paul Dale <ppzgs1@gmail.com>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from #24464)
openssl-machine pushed a commit that referenced this pull request May 29, 2024
@amiremohamadi @mattcaswell
Fix potential memory leak in test_bad_dtls
94a672e 
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Paul Dale <ppzgs1@gmail.com>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from #24464)

(cherry picked from commit abe05fd)
openssl-machine pushed a commit that referenced this pull request May 29, 2024
@amiremohamadi @mattcaswell
Fix potential memory leak in test_bad_dtls
7d9e695 
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Paul Dale <ppzgs1@gmail.com>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from #24464)

(cherry picked from commit abe05fd)
openssl-machine pushed a commit that referenced this pull request May 29, 2024
@amiremohamadi @mattcaswell
Fix potential memory leak in test_bad_dtls
7b50a23 
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Paul Dale <ppzgs1@gmail.com>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from #24464)

(cherry picked from commit abe05fd)
openssl-machine pushed a commit that referenced this pull request May 29, 2024
@amiremohamadi @mattcaswell
Fix potential memory leak in test_bad_dtls
0bd7b9e 
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Paul Dale <ppzgs1@gmail.com>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from #24464)

(cherry picked from commit abe05fd)
jvdsn pushed a commit to jvdsn/openssl that referenced this pull request Jun 3, 2024
@amiremohamadi @jvdsn
Fix potential memory leak in test_bad_dtls
8cc43d4 
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Paul Dale <ppzgs1@gmail.com>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from openssl#24464)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@t8m t8m t8m approved these changes

@mattcaswell mattcaswell mattcaswell approved these changes

@paulidale paulidale paulidale approved these changes

Assignees
No one assigned
Labels
approval: ready to merge The 24 hour grace period has passed, ready to merge branch: master Merge to master branch branch: 3.0 Merge to openssl-3.0 branch branch: 3.1 Merge to openssl-3.1 branch: 3.2 Merge to openssl-3.2 branch: 3.3 Merge to openssl-3.3 tests: exempted The PR is exempt from requirements for testing triaged: bug The issue/pr is/fixes a bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

There is a memory leak defect at line 495 in the /openssl/test/bad_dtls_test.c file.
5 participants
@amiremohamadi @openssl-machine @mattcaswell @t8m @paulidale