Skip to content

Comments

check the return value of BN_new() and BN_dup()#16948

Closed
x2018 wants to merge 1 commit intoopenssl:masterfrom
x2018:check_rv
Closed

check the return value of BN_new() and BN_dup()#16948
x2018 wants to merge 1 commit intoopenssl:masterfrom
x2018:check_rv

Conversation

@x2018
Copy link
Contributor

@x2018 x2018 commented Nov 1, 2021

Checklist

  • 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 Nov 1, 2021
@paulidale paulidale added the approval: done This pull request has the required number of approvals label Nov 2, 2021
@t8m t8m removed the approval: review pending This pull request needs review by a committer label Nov 2, 2021
@openssl-machine openssl-machine removed the approval: done This pull request has the required number of approvals label Nov 3, 2021
@openssl-machine
Copy link
Collaborator

openssl-machine commented Nov 3, 2021

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 Nov 3, 2021
openssl-machine pushed a commit that referenced this pull request Nov 3, 2021
@x2018 @t8m
check the return value of BN_new() and BN_dup()
d99004f 
Reviewed-by: Paul Dale <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
(Merged from #16948)
openssl-machine pushed a commit that referenced this pull request Nov 3, 2021
@x2018 @t8m
check the return value of BN_new() and BN_dup()
3f67195 
Reviewed-by: Paul Dale <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
(Merged from #16948)

(cherry picked from commit d99004f)
openssl-machine pushed a commit that referenced this pull request Nov 3, 2021
@x2018 @t8m
check the return value of BN_new() and BN_dup()
afbea17 
Reviewed-by: Paul Dale <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
(Merged from #16948)

(cherry picked from commit d99004f)
@t8m
Copy link
Member

t8m commented Nov 3, 2021

Merged to master, 3.0, and 1.1.1 branches.

@t8m t8m closed this Nov 3, 2021
@t8m
Copy link
Member

t8m commented Nov 3, 2021

Thank you for the contribution.

baentsch added a commit to open-quantum-safe/openssl that referenced this pull request Jun 23, 2022
@baentsch @luyahan
@devnexen @davidben @t8m @paulidale @mattcaswell @levitte @ueno @ischwarze @bbbrumley @tmshort @ifranzki @romen @bernd-edlinger @uudiin @zhaozg @Dani-Hub @ellert @lprimak @mspncp @beldmit @PeiweiHu @x2018 @allanjude @DDvO @dongbeiouba @KanPlus @roussosalex @pkubaj @jmgdjgpp @Thomas1664 @JiangJias @faramir-dev @hlandau @juergenchrist @reaperhulk @basavesh @frasertweedale
Upstream 111p merge (#375