Skip to content

Comments

Fix speed, use OPENSSL_free instead of free#17132

Closed
dongbeiouba wants to merge 1 commit intoopenssl:OpenSSL_1_1_1-stablefrom
auvkone:fix111/speed_free
Closed

Fix speed, use OPENSSL_free instead of free#17132
dongbeiouba wants to merge 1 commit intoopenssl:OpenSSL_1_1_1-stablefrom
auvkone:fix111/speed_free

Conversation

@dongbeiouba
Copy link
Contributor

@dongbeiouba dongbeiouba commented Nov 25, 2021 •
edited
Loading

No description provided.

@beldmit
Copy link
Member

beldmit commented Nov 25, 2021

Not sure it's a relevant fix. fds is allocated via app_malloc, not OPENSSL_malloc.

@dongbeiouba
Copy link
Contributor Author

dongbeiouba commented Nov 25, 2021

Not sure it's a relevant fix. fds is allocated via app_malloc, not OPENSSL_malloc.

app_malloc invokes OPENSSL_malloc().

@t8m t8m added branch: 1.1.1 Applies to OpenSSL_1_1_1-stable branch (EOL) triaged: bug The issue/pr is/fixes a bug approval: review pending This pull request needs review by a committer labels Nov 25, 2021
@t8m
Copy link
Member

t8m commented Nov 25, 2021

OK for 1.1.1

@paulidale paulidale 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 Nov 25, 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 Nov 27, 2021
@openssl-machine
Copy link
Collaborator

openssl-machine commented Nov 27, 2021

This pull request is ready to merge

@paulidale
Copy link
Contributor

paulidale commented Nov 27, 2021

Merged, thanks for the fix.

@paulidale paulidale closed this Nov 27, 2021
openssl-machine pushed a commit that referenced this pull request Nov 27, 2021
@dongbeiouba @paulidale
Fix speed, use OPENSSL_free instead of free
37dc4f9 
Reviewed-by: Tomas Mraz <[email protected]>
Reviewed-by: Paul Dale <[email protected]>
(Merged from #17132)
@dongbeiouba dongbeiouba deleted the fix111/speed_free branch December 1, 2021 02:55
dongbeiouba added a commit to auvkone/BabaSSL that referenced this pull request Dec 1, 2021
@dongbeiouba
Fix speed, use OPENSSL_free instead of free
2111825 
Reviewed-by: Tomas Mraz <[email protected]>
Reviewed-by: Paul Dale <[email protected]>
(Merged from openssl/openssl#17132)
@dongbeiouba dongbeiouba mentioned this pull request Dec 1, 2021
Merged
dongbeiouba added a commit to Tongsuo-Project/Tongsuo that referenced this pull request Dec 3, 2021
@dongbeiouba
Fix speed, use OPENSSL_free instead of free
4c2b606 
Reviewed-by: Tomas Mraz <[email protected]>
Reviewed-by: Paul Dale <[email protected]>
(Merged from openssl/openssl#17132)
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