Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@deweerdt
Copy link
Member

@deweerdt deweerdt commented Jun 6, 2018

coverity finding CID 1469832

coverity finding CID 1469832
@kazuho
Copy link
Member

kazuho commented Jun 12, 2018

Thank you for the PR. Do we need to add checks for other calls to the crypto library (e.g., HMAC_Init_ex, EVP_DecryptInit_ex)?

@deweerdt
Copy link
Member Author

@kazuho we might. The reason i did it for EVP_EncryptInit_ex is that existing code checks for it, and coverity notices that. I think it's good for code quality if checks are done in a consistent manner when possible.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants