Skip to content

ERR: replace remnant ECerr() and EVPerr() calls in crypto/#19302

Closed
DDvO wants to merge 1 commit intoopenssl:masterfrom
siemens:clean_remnant_ECerr_EVPerr
Closed

ERR: replace remnant ECerr() and EVPerr() calls in crypto/#19302
DDvO wants to merge 1 commit intoopenssl:masterfrom
siemens:clean_remnant_ECerr_EVPerr

Conversation

@DDvO
Copy link
Contributor

@DDvO DDvO commented Sep 29, 2022

except those throwing ERR_R_MALLOC_FAILURE, which are handled by #19301.

This has been taken over from #19072.

except those throwing ERR_R_MALLOC_FAILURE
@DDvO DDvO added branch: master Applies to master branch approval: review pending This pull request needs review by a committer approval: otc review pending triaged: cleanup The issue/pr deals with cleanup of comments/docs not altering code significantly labels Sep 29, 2022
@DDvO DDvO requested a review from levitte September 29, 2022 13:18
@github-actions github-actions bot added the severity: fips change The pull request changes FIPS provider sources label Sep 29, 2022
@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 29, 2022
@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 30, 2022
@openssl-machine
Copy link
Collaborator

This pull request is ready to merge

@DDvO
Copy link
Contributor Author

DDvO commented Sep 30, 2022

Merged - thanks for the swift handling

openssl-machine pushed a commit that referenced this pull request Sep 30, 2022
except those throwing ERR_R_MALLOC_FAILURE

Reviewed-by: Richard Levitte <levitte@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Paul Dale <pauli@openssl.org>
Reviewed-by: David von Oheimb <david.von.oheimb@siemens.com>
(Merged from #19302)
@DDvO DDvO closed this Sep 30, 2022
t8m pushed a commit to t8m/openssl that referenced this pull request Nov 21, 2022
except those throwing ERR_R_MALLOC_FAILURE

Reviewed-by: Richard Levitte <levitte@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Paul Dale <pauli@openssl.org>
Reviewed-by: David von Oheimb <david.von.oheimb@siemens.com>
(Merged from openssl#19302)

(cherry picked from commit bd07cc1)
openssl-machine pushed a commit that referenced this pull request Nov 23, 2022
except those throwing ERR_R_MALLOC_FAILURE

Reviewed-by: Richard Levitte <levitte@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Paul Dale <pauli@openssl.org>
Reviewed-by: David von Oheimb <david.von.oheimb@siemens.com>
(Merged from #19302)

(cherry picked from commit bd07cc1)
beldmit pushed a commit to beldmit/openssl that referenced this pull request Dec 26, 2022
except those throwing ERR_R_MALLOC_FAILURE

Reviewed-by: Richard Levitte <levitte@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Paul Dale <pauli@openssl.org>
Reviewed-by: David von Oheimb <david.von.oheimb@siemens.com>
(Merged from openssl#19302)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approval: ready to merge The 24 hour grace period has passed, ready to merge branch: master Applies to master branch severity: fips change The pull request changes FIPS provider sources triaged: cleanup The issue/pr deals with cleanup of comments/docs not altering code significantly

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants