Skip to content

Comments

test/ssl_old_test.c: Fix potential leak#16806

Closed
PeiweiHu wants to merge 1 commit intoopenssl:masterfrom
PeiweiHu:PeiweiHu-patch_1012
Closed

test/ssl_old_test.c: Fix potential leak#16806
PeiweiHu wants to merge 1 commit intoopenssl:masterfrom
PeiweiHu:PeiweiHu-patch_1012

Conversation

@PeiweiHu
Copy link
Contributor

@PeiweiHu PeiweiHu commented Oct 12, 2021

Checklist
  • documentation is added or updated
  • tests are added or updated

@t8m t8m added branch: 1.1.1 Applies to OpenSSL_1_1_1-stable branch (EOL) branch: 3.0 Applies to openssl-3.0 branch branch: master Applies to master branch triaged: bug The issue/pr is/fixes a bug approval: review pending This pull request needs review by a committer labels Oct 12, 2021
@kroeckx kroeckx 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 Oct 13, 2021
@openssl-machine openssl-machine added approval: ready to merge The 24 hour grace period has passed, ready to merge and removed approval: done This pull request has the required number of approvals labels Oct 14, 2021
@openssl-machine
Copy link
Collaborator

openssl-machine commented Oct 14, 2021

This pull request is ready to merge

openssl-machine pushed a commit that referenced this pull request Oct 15, 2021
@PeiweiHu @t8m
test/ssl_old_test.c: Fix potential leak
34563be 
Reviewed-by: Kurt Roeckx <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
(Merged from #16806)
openssl-machine pushed a commit that referenced this pull request Oct 15, 2021
@PeiweiHu @t8m
test/ssl_old_test.c: Fix potential leak
f9c4823 
Reviewed-by: Kurt Roeckx <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
(Merged from #16806)

(cherry picked from commit 34563be)
openssl-machine pushed a commit that referenced this pull request Oct 15, 2021
@PeiweiHu @t8m
test/ssl_old_test.c: Fix potential leak
2f8b804 
Reviewed-by: Kurt Roeckx <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
(Merged from #16806)

(cherry picked from commit 34563be)
@t8m
Copy link
Member

t8m commented Oct 15, 2021

Merged to master, 3.0 and 1.1.1 branches. Thank you for the contribution.

@t8m t8m closed this Oct 15, 2021
dstebila pushed a commit to open-quantum-safe/openssl that referenced this pull request Mar 15, 2022
@baentsch @levitte
@dongbeiouba @t8m @mattcaswell @NanXiao @yursha @mohd-akram @dcoombs @tmshort @paulidale @niclasr @slontis @tomato42 @beldmit @iguessthislldo @devnexen @bonniegong @kaduk @botovq @tiran @jpboivin @DDvO @p-steuer @goodpaperman @larsimmisch-adobe @luyahan @davidben @ueno @ischwarze @bbbrumley @ifranzki @romen @bernd-edlinger @uudiin @zhaozg @Dani-Hub @ellert @lprimak @mspncp @PeiweiHu @x2018 @allanjude @KanPlus @roussosalex @pkubaj @jmgdjgpp @Thomas1664 @JiangJias @faramir-dev @hlandau
1.1.1n merge (#361