Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SSL: Change SSLerr() to ERR_raise() #13450

Closed
wants to merge 1 commit into from
Closed

Conversation

@levitte
Copy link
Member

@levitte levitte commented Nov 19, 2020

This was probably due to a merge

Fixes #13449

This was probably due to a merge

Fixes #13449
@DDvO
DDvO approved these changes Nov 20, 2020
Copy link
Contributor

@DDvO DDvO left a comment

LGTM.
I suggest marking this urgent and merging it right away since this fixes CI breaks.

@levitte
Copy link
Member Author

@levitte levitte commented Nov 20, 2020

I suggest marking this urgent and merging it right away since this fixes CI breaks.

Thanks for the reminder, I had completely forgotten this

@levitte levitte added the urgent label Nov 20, 2020
@levitte
Copy link
Member Author

@levitte levitte commented Nov 20, 2020

Merged

276d6c6 SSL: Change SSLerr() to ERR_raise()

@levitte levitte closed this Nov 20, 2020
openssl-machine pushed a commit that referenced this pull request Nov 20, 2020
This was probably due to a merge

Fixes #13449

Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: David von Oheimb <david.von.oheimb@siemens.com>
(Merged from #13450)
@levitte levitte deleted the levitte:fix-13449 branch Nov 20, 2020
clin1234 added a commit to clin1234/openssl that referenced this pull request Nov 21, 2020
This was probably due to a merge

Fixes openssl#13449

Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: David von Oheimb <david.von.oheimb@siemens.com>
(Merged from openssl#13450)
@nelsonopenssl nelsonopenssl added this to Done in 3.0.0 estimator Dec 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Linked issues

Successfully merging this pull request may close these issues.

3 participants