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

DOC: fix documentation of som EVP_MD_CTX functions #9996

Closed
wants to merge 2 commits into from

Conversation

levitte
Copy link
Member

@levitte levitte commented Sep 24, 2019

They were documented to take an EVP_MD pointer, when they really take
an EVP_MD_CTX pointer.

Fixes #9993

They were documented to take an EVP_MD pointer, when they really take
an EVP_MD_CTX pointer.

Fixes openssl#9993
@levitte levitte added the branch: master Merge to master branch label Sep 24, 2019
@levitte levitte changed the title DOC: fix documentation of som EVP_MD_CTX functions [masster only] DOC: fix documentation of som EVP_MD_CTX functions [master only] Sep 24, 2019
doc/man3/EVP_DigestInit.pod Outdated Show resolved Hide resolved
@levitte levitte added the branch: 1.1.1 Merge to OpenSSL_1_1_1-stable branch label Sep 24, 2019
@levitte levitte changed the title DOC: fix documentation of som EVP_MD_CTX functions [master only] DOC: fix documentation of som EVP_MD_CTX functions Sep 24, 2019
@levitte levitte removed the branch: 1.1.1 Merge to OpenSSL_1_1_1-stable branch label Sep 24, 2019
@levitte
Copy link
Member Author

levitte commented Sep 24, 2019

That also means there's no need for the separate PR for 1.1.1 any longer...

(I did anyway... it was approved, so)

levitte added a commit that referenced this pull request Sep 24, 2019
They were documented to take an EVP_MD pointer, when they really take
an EVP_MD_CTX pointer.

Fixes #9993

Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from #9996)
@levitte
Copy link
Member Author

levitte commented Sep 24, 2019

Merged.

7c2a981 DOC: fix documentation of som EVP_MD_CTX functions

@levitte levitte closed this Sep 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
branch: master Merge to master branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

EVP_MD_CTX_size parameters is wrong!
3 participants