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

libcr51sign using old SHA hash functions #2

Open
williamspatrick opened this issue May 10, 2022 · 4 comments
Open

libcr51sign using old SHA hash functions #2

williamspatrick opened this issue May 10, 2022 · 4 comments

Comments

@williamspatrick
Copy link
Member

subprojects/libcr51sign/src/meson.build:    '-Wno-error=deprecated-declarations',

I noticed this being used as a hack to enable compile with OpenSSL3. OpenSSL3 is available everywhere in the org now (including in the Docker container), so we should probably remove the SSLv1 interfaces. I'm slightly concerned this is going to end up becoming a Yocto update blocker at some point in the not too distant future, so it would be best if this could be done in the next few months.

@brandonkimbk
Copy link
Contributor

Thanks for the heads up Patrick. I believe @FighterNan worked on that piece with the security folks, I'll see if he can own cleaning up that piece.

@FighterNan
Copy link
Contributor

Thanks, let me bring that up with security teams.

@williamspatrick
Copy link
Member Author

Thanks, let me bring that up with security teams.

Just to be sure there is no confusion. You don’t have to stop using SHA256 hash necessarily, but they have changed the primary API for doing hashes and the currently used API is deprecated. I did work across the rest of the codebase in about Sept/Oct last year to port it all to the OpenSSL3 APIs.

@FighterNan
Copy link
Contributor

Yes, I understand that. Reference the doc of SHA256_Init below.

"All of the functions described on this page except for SHA1(), SHA224(), SHA256(), SHA384() and SHA512() are deprecated. Applications should instead use EVP_DigestInit_ex(3), EVP_DigestUpdate(3) and EVP_DigestFinal_ex(3), or the quick one-shot function EVP_Q_digest(3). SHA1(), SHA224(), SHA256(), SHA384(), and SHA256() can continue to be used. They can also be replaced by, e.g.,"

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

No branches or pull requests

3 participants