Skip to content

Comments

MacOS prior to 10.12 does not support random API correctly [branch OpenSSL_1_1_1-stable]#16587

Closed
lprimak wants to merge 1 commit intoopenssl:OpenSSL_1_1_1-stablefrom
flowlogix:p3
Closed

MacOS prior to 10.12 does not support random API correctly [branch OpenSSL_1_1_1-stable]#16587
lprimak wants to merge 1 commit intoopenssl:OpenSSL_1_1_1-stablefrom
flowlogix:p3

Conversation

@lprimak
Copy link
Contributor

@lprimak lprimak commented Sep 12, 2021 •
edited
Loading

Fixes #16517

@openssl-machine openssl-machine added the hold: cla required The contributor needs to submit a license agreement label Sep 12, 2021
@lprimak
Copy link
Contributor Author

lprimak commented Sep 12, 2021

ICLA sent

@lprimak lprimak marked this pull request as ready for review September 12, 2021 00:11
@lprimak lprimak closed this Sep 12, 2021
@lprimak lprimak reopened this Sep 12, 2021
@openssl-machine openssl-machine removed the hold: cla required The contributor needs to submit a license agreement label Sep 12, 2021
@paulidale paulidale added approval: review pending This pull request needs review by a committer branch: 1.1.1 Applies to OpenSSL_1_1_1-stable branch (EOL) triaged: bug The issue/pr is/fixes a bug labels Sep 12, 2021
@paulidale paulidale closed this Sep 12, 2021
@paulidale paulidale reopened this Sep 12, 2021
@slontis
Copy link
Member

slontis commented Sep 12, 2021

Isnt this a problem in master/3.0 also?

@lprimak
Copy link
Contributor Author

lprimak commented Sep 13, 2021

Maybe... I can get a patch for 3 as well.. lemme look

@lprimak lprimak mentioned this pull request Sep 13, 2021
Closed
@lprimak lprimak changed the title MacOS prior to 10.12 does not support random API correctly MacOS prior to 10.12 does not support random API correctly [branch OpenSSL_1_1_1-stable] Sep 13, 2021
@lprimak lprimak mentioned this pull request Sep 13, 2021
Closed
@lprimak
Copy link
Contributor Author

lprimak commented Sep 13, 2021

patches created for all 3 branches (linked above)

@t8m t8m 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 Sep 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 Sep 14, 2021
@openssl-machine
Copy link
Collaborator

openssl-machine commented Sep 14, 2021

This pull request is ready to merge

openssl-machine pushed a commit that referenced this pull request Sep 14, 2021
@lprimak @t8m
MacOS prior to 10.12 does not support random API correctly
2f3b120 
Fixes #16517

Reviewed-by: Paul Dale <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
(Merged from #16587)
@t8m
Copy link
Member

t8m commented Sep 14, 2021

Merged to 1.1.1 branch. Thank you for the contribution.

@t8m t8m closed this Sep 14, 2021
fhgwright pushed a commit to fhgwright/openssl that referenced this pull request Sep 30, 2021
@lprimak @fhgwright
MacOS prior to 10.12 does not support random API correctly
338e309 
Fixes openssl#16517

Reviewed-by: Paul Dale <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
(Merged from openssl#16587)
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