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

libcrypto.num: remove some unused EVP_MAC symbols #10421

Closed

Conversation

mspncp
Copy link
Contributor

@mspncp mspncp commented Nov 12, 2019

No description provided.

@mspncp mspncp added approval: otc review pending This pull request needs review by an OTC member branch: master Merge to master branch labels Nov 12, 2019
@levitte levitte added approval: done This pull request has the required number of approvals and removed approval: otc review pending This pull request needs review by an OTC member labels Nov 12, 2019
@mspncp
Copy link
Contributor Author

mspncp commented Nov 12, 2019

Oh, BTW: are there any more ephemeral symbols in this list, which need to be removed too?

~/src/openssl$ grep -n '\?.*NOEXIST' util/libcrypto.num 
4428:EVP_MAC_CTX_new_id                      ?	3_0_0	NOEXIST::FUNCTION:
4452:CRYPTO_siv128_new                       ?	3_0_0	NOEXIST::FUNCTION:SIV
4453:CRYPTO_siv128_init                      ?	3_0_0	NOEXIST::FUNCTION:SIV
4454:CRYPTO_siv128_copy_ctx                  ?	3_0_0	NOEXIST::FUNCTION:SIV
4455:CRYPTO_siv128_aad                       ?	3_0_0	NOEXIST::FUNCTION:SIV
4456:CRYPTO_siv128_encrypt                   ?	3_0_0	NOEXIST::FUNCTION:SIV
4457:CRYPTO_siv128_decrypt                   ?	3_0_0	NOEXIST::FUNCTION:SIV
4458:CRYPTO_siv128_finish                    ?	3_0_0	NOEXIST::FUNCTION:SIV
4459:CRYPTO_siv128_set_tag                   ?	3_0_0	NOEXIST::FUNCTION:SIV
4460:CRYPTO_siv128_get_tag                   ?	3_0_0	NOEXIST::FUNCTION:SIV
4461:CRYPTO_siv128_cleanup                   ?	3_0_0	NOEXIST::FUNCTION:SIV
4462:CRYPTO_siv128_speed                     ?	3_0_0	NOEXIST::FUNCTION:SIV
4660:EVP_PKEY_derive_init_ex                 ?	3_0_0	NOEXIST::FUNCTION:
4671:ERR_put_func_error                      ?	3_0_0	NOEXIST::FUNCTION:
4712:EVP_MAC_name                            ?	3_0_0	NOEXIST::FUNCTION:
4723:EVP_KDF_name                            ?	3_0_0	NOEXIST::FUNCTION:
4736:EVP_PKEY_sign_init_ex                   ?	3_0_0	NOEXIST::FUNCTION:
4738:EVP_PKEY_verify_init_ex                 ?	3_0_0	NOEXIST::FUNCTION:
4739:EVP_PKEY_verify_recover_init_ex         ?	3_0_0	NOEXIST::FUNCTION:

@mspncp mspncp 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 13, 2019
openssl-machine pushed a commit that referenced this pull request Nov 13, 2019
Reviewed-by: Richard Levitte <levitte@openssl.org>
(Merged from #10421)
@mspncp
Copy link
Contributor Author

mspncp commented Nov 13, 2019

Merged to master in a7372f5, thanks!

@mspncp mspncp closed this Nov 13, 2019
@mspncp mspncp deleted the pr-remove-unused-evp-mac-symbols branch November 13, 2019 15:45
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 Merge to master branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants